Minimal clean-up of deprecated features

Author: DanielCortild

functions/Actions/CampsEvents.py

@@ -41,30 +41,30 @@ def _generate_event_body(self, camp_info):
             'sendUpdates': 'all'
         }
 
-    def _manage_picture_folders(self, camp_info):
-        """Creates Google Drive folders for camp pictures if they don't exist."""
-        # ... unchanged ...
-        # (same as original)
-        # Create holiday-specific folder if needed
-        if not camp_info.get("picture_grand_parent_id"):
-            folder_id = self.google.create_camp_pictures_folder(f"{camp_info.get('holiday_name')}")
-            self.sf.sf.Picklist__c.update(camp_info["holiday_id"], {"Google_Drive_Pictures_ID__c": folder_id})
-            camp_info["picture_grand_parent_id"] = folder_id
-
-        # Create week-specific folder if needed
-        if not camp_info.get("picture_parent_id"):
-            folder_id = self.google.create_camp_pictures_folder(
-                f"{camp_info.get('picture_parent_name')}", parent=camp_info["picture_grand_parent_id"]
-            )
-            self.sf.sf.Picklist__c.update(camp_info["week_id"], {"Google_Drive_Pictures_ID__c": folder_id})
-            camp_info["picture_parent_id"] = folder_id
-
-        # Create camp-specific folder if needed
-        if not camp_info.get("pictures_id"):
-            folder_id = self.google.create_camp_pictures_folder(
-                f"{camp_info.get('pictures_name')}", parent=camp_info["picture_parent_id"]
-            )
-            self.sf.sf.Opportunity.update(camp_info["id"], {"Google_Drive_Pictures__c": folder_id})
+    # def _manage_picture_folders(self, camp_info):
+    #     """Creates Google Drive folders for camp pictures if they don't exist."""
+    #     # ... unchanged ...
+    #     # (same as original)
+    #     # Create holiday-specific folder if needed
+    #     if not camp_info.get("picture_grand_parent_id"):
+    #         folder_id = self.google.create_camp_pictures_folder(f"{camp_info.get('holiday_name')}")
+    #         self.sf.sf.Picklist__c.update(camp_info["holiday_id"], {"Google_Drive_Pictures_ID__c": folder_id})
+    #         camp_info["picture_grand_parent_id"] = folder_id
+    #
+    #     # Create week-specific folder if needed
+    #     if not camp_info.get("picture_parent_id"):
+    #         folder_id = self.google.create_camp_pictures_folder(
+    #             f"{camp_info.get('picture_parent_name')}", parent=camp_info["picture_grand_parent_id"]
+    #         )
+    #         self.sf.sf.Picklist__c.update(camp_info["week_id"], {"Google_Drive_Pictures_ID__c": folder_id})
+    #         camp_info["picture_parent_id"] = folder_id
+    #
+    #     # Create camp-specific folder if needed
+    #     if not camp_info.get("pictures_id"):
+    #         folder_id = self.google.create_camp_pictures_folder(
+    #             f"{camp_info.get('pictures_name')}", parent=camp_info["picture_parent_id"]
+    #         )
+    #         self.sf.sf.Opportunity.update(camp_info["id"], {"Google_Drive_Pictures__c": folder_id})
 
     def _add_camp_event_to_batch(self, camp_info):
         """Handles creating or updating a single camp event via batch request."""
@@ -192,7 +192,7 @@ def process_camps(self, camps):
         for camp_info in camps:
             try:
                 self._add_camp_event_to_batch(camp_info)
-                self._manage_picture_folders(camp_info)
+                # self._manage_picture_folders(camp_info)
                 print(f'[INFO] Queued tasks for {camp_info.get("code")}')
             except Exception as e:
                 print(f'[ERROR] Failed to queue tasks for {camp_info.get("code")}: {e}')

functions/Actions/CampsEvents2.py

@@ -102,23 +102,23 @@ def callback3(request_id, response, exception):
         # Update the Google Event
         batch1.add(google.calendar.events().update(calendarId=google.CALENDAR_CAMPS_ID, eventId=camp_info.get("event_id"), body=event, sendUpdates="all"), callback=callback1)
 
-def __camp_pictures(google, sf, camp_info):
-    # If holiday specific folder does not exist, create it
-    if camp_info.get("picture_grand_parent_id") == None:
-        created_folder_id = google.create_camp_pictures_folder(f"{camp_info.get('holiday_name')}")
-        sf.sf.Picklist__c.update(camp_info["holiday_id"], {"Google_Drive_Pictures_ID__c": created_folder_id})
-        camp_info["picture_grand_parent_id"] = created_folder_id
-
-    # If week specific folder does not exist, create it
-    if camp_info.get("picture_parent_id") == None:
-        created_folder_id = google.create_camp_pictures_folder(f"{camp_info.get('picture_parent_name')}", parent=camp_info["picture_grand_parent_id"])
-        sf.sf.Picklist__c.update(camp_info["week_id"], {"Google_Drive_Pictures_ID__c": created_folder_id})
-        camp_info["picture_parent_id"] = created_folder_id
-
-    # If camp specific folder does not exist, create it
-    if camp_info["pictures_id"] == None:
-        created_folder_id = google.create_camp_pictures_folder(f"{camp_info.get('pictures_name')}", parent=camp_info["picture_parent_id"])
-        sf.sf.Opportunity.update(camp_info["id"], {"Google_Drive_Pictures__c": created_folder_id})
+# def __camp_pictures(google, sf, camp_info):
+#     # If holiday specific folder does not exist, create it
+#     if camp_info.get("picture_grand_parent_id") == None:
+#         created_folder_id = google.create_camp_pictures_folder(f"{camp_info.get('holiday_name')}")
+#         sf.sf.Picklist__c.update(camp_info["holiday_id"], {"Google_Drive_Pictures_ID__c": created_folder_id})
+#         camp_info["picture_grand_parent_id"] = created_folder_id
+#
+#     # If week specific folder does not exist, create it
+#     if camp_info.get("picture_parent_id") == None:
+#         created_folder_id = google.create_camp_pictures_folder(f"{camp_info.get('picture_parent_name')}", parent=camp_info["picture_grand_parent_id"])
+#         sf.sf.Picklist__c.update(camp_info["week_id"], {"Google_Drive_Pictures_ID__c": created_folder_id})
+#         camp_info["picture_parent_id"] = created_folder_id
+#
+#     # If camp specific folder does not exist, create it
+#     if camp_info["pictures_id"] == None:
+#         created_folder_id = google.create_camp_pictures_folder(f"{camp_info.get('pictures_name')}", parent=camp_info["picture_parent_id"])
+#         sf.sf.Opportunity.update(camp_info["id"], {"Google_Drive_Pictures__c": created_folder_id})
 
 def update_and_create_camps_per_week(google, sf, week_codes):
     # Get the camp details for the specified weeks
@@ -136,7 +136,7 @@ def update_and_create_camps_per_week(google, sf, week_codes):
     for camp_info in camps:
         try:
             __camp_event(google, sf, camp_info, batch1, batch2, batch3)
-            __camp_pictures(google, sf, camp_info)
+            # __camp_pictures(google, sf, camp_info)
             print(f'[SUCCESS] Event created for {camp_info.get("code")}')
         except Exception as e:
             raise ValueError(f'[ERROR] {e}')
@@ -165,7 +165,7 @@ def update_and_create_camps(google, sf, camp_id):
     for camp_info in camps:
         try:
             __camp_event(google, sf, camp_info, batch1, batch2, batch3)
-            __camp_pictures(google, sf, camp_info)
+            # __camp_pictures(google, sf, camp_info)
             print(f'[SUCCESS] Event created for {camp_info.get("code")}')
         except Exception as e:
             raise ValueError(f'[ERROR] {e}')

functions/Actions/UpdatePaymentsSheet.py

@@ -1,6 +1,4 @@
 from .EventsToText import generate_confirmation_text, get_teacher_dict
 from .CampsForm import get_camps_form
-from .UpdatePaymentsSheet import update_payment_sheet
 from .CampsEvents import update_and_create_camps_per_week
 from .CampsEvents import update_and_create_camps
-# from .ClassesEvents import update_and_create_classes_per_week