fix: set deduplication_status in bulk SQL insert, remove inaccurate docstring return

Include deduplication_status='new' in the program membership raw SQL
INSERT to match ORM default. Remove misleading :return: docstring from
_add_beneficiaries (method returns None).

Author: gonzalesedwin1123

spp_programs/models/managers/cycle_manager_base.py

@@ -838,7 +838,6 @@ def _add_beneficiaries(self, cycle, beneficiaries, state="draft", do_count=False
         :param beneficiaries: List of partner IDs
         :param state: String state to be set to beneficiary
         :param do_count: Boolean - set to False to not run compute functions
-        :return: Integer - count of inserted members
         """
         today = fields.Date.today()
         vals_list = [

spp_programs/models/program_membership.py

@@ -414,13 +414,14 @@ def _bulk_insert_on_conflict(self, vals_list, chunk_size=1000):
             for v in batch:
                 state = v.get("state", "draft")
                 enrollment_date = now if state == "enrolled" else None
-                values.append("(%s, %s, %s, %s, %s, %s, now(), now())")
+                values.append("(%s, %s, %s, %s, %s, %s, %s, now(), now())")
                 params.extend(
                     [
                         v["partner_id"],
                         v["program_id"],
                         state,
                         enrollment_date,
+                        v.get("deduplication_status", "new"),
                         uid,
                         uid,
                     ]
@@ -429,6 +430,7 @@ def _bulk_insert_on_conflict(self, vals_list, chunk_size=1000):
             sql = """
                 INSERT INTO spp_program_membership
                     (partner_id, program_id, state, enrollment_date,
+                     deduplication_status,
                      create_uid, write_uid, create_date, write_date)
                 VALUES {}
                 ON CONFLICT (partner_id, program_id) DO NOTHING