Changeset 862198c in tailbone

Timestamp:

08/03/2022 11:13:43 AM (6 months ago)

Author:

Lance Edgar <lance@…>

Branches:

master

Children:

4ff0450

Parents:

3726a26

Message:

Improve "touch" logic for employees

also use app handler for default touch logic

Location:

tailbone/views

Files:

• employees.py (modified) (1 diff)
• master.py (modified) (2 diffs)

tailbone/views/employees.py

@@ -305 +305 @@
         return HTML.tag('ul', c=items)
 
+    def touch_instance(self, employee):
+        app = self.get_rattail_app()
+        employment = app.get_employment_handler()
+        employment.touch_employee(self.Session(), employee)
+
     def get_version_child_classes(self):
         return [

tailbone/views/master.py

@@ -1160 +1160 @@
         Perform actual "touch" logic for the given object.
         """
-        change = model.Change()
-        change.class_name = obj.__class__.__name__
-        change.instance_uuid = obj.uuid
-        change = self.Session.merge(change)
-        change.deleted = False
+        app = self.get_rattail_app()
+        app.touch_object(self.Session(), obj)
 
     def versions(self):
@@ -4796 +4793 @@
             config.add_tailbone_permission(permission_prefix, '{}.touch'.format(permission_prefix),
                                            "\"Touch\" a {} to trigger datasync for it".format(model_title))
-            config.add_route('{}.touch'.format(route_prefix), '{}/touch'.format(instance_url_prefix))
+            config.add_route('{}.touch'.format(route_prefix),
+                             '{}/touch'.format(instance_url_prefix),
+                             # TODO: should add this restriction after the old
+                             # jquery theme is no longer in use
+                             #request_method='POST'
+            )
             config.add_view(cls, attr='touch', route_name='{}.touch'.format(route_prefix),
                             permission='{}.touch'.format(permission_prefix))