fix(spp_programs): remove unused variable and invalid noqa comment

emjay0921 · emjay0921 · commit dd6385185523 · 2026-03-18T13:15:32.000+08:00
diff --git a/spp_programs/models/managers/deduplication_manager.py b/spp_programs/models/managers/deduplication_manager.py
@@ -556,12 +556,6 @@ def _check_duplicate_by_individual(self, beneficiaries):
         individual_ids = [x.partner_id.id for x in duplicated_phone_ids]
         individual_ids = list(dict.fromkeys(individual_ids))
         _logger.debug("Individual IDS with Duplicated Phone Numbers: %s", individual_ids)
-        individual_program_membership = self.env["spp.program.membership"].search(
-            [
-                ("partner_id", "in", individual_ids),
-                ("program_id", "=", self.program_id.id),
-            ]
-        )
 
         all_duplicated_memberships = self.env["spp.program.membership"]
 
diff --git a/spp_programs/models/programs.py b/spp_programs/models/programs.py
@@ -721,7 +721,7 @@ def check_managers_limit(self):
                 combined_message = ", ".join(error_messages)
                 raise UserError(
                     f"Only one manager can be configured under {combined_message}."
-                    f"Please delete any new manager(s) before saving your changes."  # noqa: B950
+                    f"Please delete any new manager(s) before saving your changes."
                 )
 
     def _pre_enrollment_hook(self, partner):

Original file line number	Diff line number	Diff line change
`@@ -721,7 +721,7 @@ def check_managers_limit(self):`
`721`	`721`	`combined_message = ", ".join(error_messages)`
`722`	`722`	`raise UserError(`
`723`	`723`	`f"Only one manager can be configured under {combined_message}."`
`724`		`- f"Please delete any new manager(s) before saving your changes." # noqa: B950`
	`724`	`+ f"Please delete any new manager(s) before saving your changes."`
`725`	`725`	`)`
`726`	`726`
`727`	`727`	`def _pre_enrollment_hook(self, partner):`