Only change the time a beer was added to a tap if the beer changed

tap_list_providers/parsers/digitalpour.py

@@ -62,7 +62,6 @@ def handle_venue(self, venue: Venue) -> datetime.datetime:
                 tap = taps[tap_info["tap_number"]]
             except KeyError:
                 tap = Tap(venue=venue, tap_number=tap_info["tap_number"])
-            tap.time_added = tap_info["added"]
             tap.time_updated = tap_info["updated"]
             if tap.time_updated and tap.time_updated > self.update_date:
                 LOG.debug("Updating venue timestamp to %s", tap.time_updated)
@@ -123,6 +122,9 @@ def handle_venue(self, venue: Venue) -> datetime.datetime:
                 venue=venue,
                 **parsed_beer,
             )
+            if beer and tap.beer_id != beer.id:
+                tap.time_added = tap_info["added"]
+
             # 4. assign the beer to the tap
             tap.beer = beer
             tap.save()

tap_list_providers/parsers/stemandstein.py

@@ -13,6 +13,7 @@
 from dateutil.parser import parse
 from django.db.models import Q
 from django.core.exceptions import ImproperlyConfigured, AppRegistryNotReady
+from django.utils.timezone import now
 
 # boilerplate code necessary for launching outside manage.py
 try:
@@ -204,6 +205,7 @@ def fill_in_beer_details(self, beer):
         return time_tapped
 
     def handle_venue(self, venue):
+        timestamp = now()
         self.venue = venue
         self.fetch_root_html()
         beers_found = self.parse_root_html()
@@ -223,6 +225,9 @@ def handle_venue(self, venue):
                     venue=self.venue,
                     tap_number=tap_number,
                 )
+            if beer and tap.beer_id != beer.id:
+                tap.time_added = timestamp
+            tap.time_updated = timestamp
             tap.beer = beer
             tap.time_added = time_tapped
             tap.save()

tap_list_providers/parsers/taphunter.py

@@ -75,7 +75,6 @@ def handle_venue(self, venue):
                 tap = taps[tap_number]
             except KeyError:
                 tap = Tap(venue=venue, tap_number=tap_number)
-            tap.time_added = tap_info["added"]
             tap.time_updated = tap_info["updated"]
             parsed_time = parse(tap_info["updated"])
             if parsed_time > latest_timestamp:
@@ -127,6 +126,9 @@ def handle_venue(self, venue):
                 venue=venue,
                 **parsed_beer,
             )
+            if beer and tap.beer_id != beer.id:
+                tap.time_added = tap_info["added"]
+
             # 4. assign the beer to the tap
             tap.beer = beer
             tap.save()

tap_list_providers/parsers/taplist_io.py

@@ -64,10 +64,12 @@ def handle_venue(self, venue):
                 current_tap.time_updated = timestamp
                 current_tap.save()
                 continue
-            current_tap.time_added = tap_dict.pop("time_added")
+            time_added = tap_dict.pop("time_added")
             mfg_dict = tap_dict.pop("manufacturer")
             manufacturer = self.get_manufacturer(**mfg_dict)
             beer = self.get_beer(manufacturer=manufacturer, venue=venue, **tap_dict)
+            if beer and current_tap.beer_id != beer.id:
+                current_tap.time_added = time_added
             current_tap.beer = beer
             if current_tap.time_updated != timestamp:
                 current_tap.time_updated = timestamp

tap_list_providers/parsers/untappd.py

@@ -127,6 +127,9 @@ def handle_venue(self, venue):
                 pricing=tap_info["pricing"],
                 **tap_info["beer"],
             )
+            if beer and tap.beer_id != beer.id and tap_info["added"]:
+                # only change the time added if the beer on tap changed
+                tap.time_added = tap_info["added"]
             # 4. assign the beer to the tap
             tap.beer = beer
             tap.save()