From bd59a8debf30e0e72b2da8c2f006410a97e3d2cd Mon Sep 17 00:00:00 2001 From: "Paul Gauthier (aider)" Date: Wed, 30 Oct 2024 12:25:18 -0700 Subject: [PATCH] refactor: extract analytics disable logic into dedicated method --- aider/analytics.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/aider/analytics.py b/aider/analytics.py index 6c982ab20..80dadee6b 100644 --- a/aider/analytics.py +++ b/aider/analytics.py @@ -29,18 +29,20 @@ class Analytics: self.get_or_create_uuid() if not enable or self.permanently_disable or permanently_disable: - #ai refactor this into a method called disable()! - self.mp = None - self.ph = None - if permanently_disable and not self.permanently_disable: - self.permanently_disable = True - self.save_data() + self.disable(permanently_disable) return if self.user_id and not self.permanently_disable: self.mp = Mixpanel(mixpanel_project_token) self.ph = Posthog(project_api_key=posthog_project_api_key, host=posthog_host) + def disable(self, permanently_disable): + self.mp = None + self.ph = None + if permanently_disable and not self.permanently_disable: + self.permanently_disable = True + self.save_data() + def get_data_file_path(self): data_file = Path.home() / ".aider" / "analytics.json" data_file.parent.mkdir(parents=True, exist_ok=True)