[ADD] swap parameter

Author: emjay0921

spp_base_common/__manifest__.py

@@ -33,6 +33,7 @@
     "assets": {
         "web.assets_backend": [
             "spp_base_common/static/src/scss/navbar.scss",
+            "spp_base_common/static/src/js/list_controller.js",
         ],
         "web._assets_primary_variables": [
             "spp_base_common/static/src/scss/colors.scss",

spp_base_common/readme/ARCHIVING_FIX.md

@@ -0,0 +1,19 @@
+## Archiving Notification Fix
+
+### Issue
+The global Odoo archiving method had swapped notification parameters, showing messages like:
+"50,000 records have been archived out of 20,000 selected"
+
+This is illogical as you cannot archive more records than were selected.
+
+### Solution
+This module overrides the `_toggleArchiveState` method in the legacy ListController to fix the swapped parameters in the notification message.
+
+The fix swaps the parameters from `(total, resIds.length)` to `(resIds.length, total)` to correctly display:
+"Of the [number_selected] records selected, only the first [number_archived] have been archived/unarchived."
+
+### Technical Details
+- **File**: `static/src/js/list_controller.js`
+- **Method**: `_toggleArchiveState`
+- **Original Location**: `addons/web/static/src/legacy/js/views/list/list_controller.js` (line 658-659)
+

spp_base_common/static/src/js/list_controller.js

@@ -0,0 +1,42 @@
+/** @odoo-module **/
+
+import { ListController } from "web.ListController";
+import { _t } from "web.core";
+import { session } from "@web/session";
+
+/**
+ * Fix for swapped notification message parameters when archiving/unarchiving records.
+ * 
+ * Original bug: Shows "Of the 20,000 records selected, only the first 50,000 have been archived"
+ * This is illogical - you can't archive more records than selected.
+ * 
+ * Fixed to show: "Of the 50,000 records selected, only the first 20,000 have been archived"
+ * This makes sense - you selected 50,000 but only 20,000 were processed due to active_ids_limit.
+ */
+ListController.include({
+    /**
+     * Toggle the archive state of the selected records.
+     * 
+     * @override
+     * @param {boolean} archive - whether to archive or unarchive
+     * @returns {Promise}
+     */
+    _toggleArchiveState: async function (archive) {
+        const resIds = await this.getSelectedIdsWithDomain();
+        const notif = this.isDomainSelected;
+        await this._archive(resIds, archive);
+        const total = this.model.get(this.handle, {raw: true}).count;
+        
+        // Fixed: Swapped parameters from (total, resIds.length) to (resIds.length, total)
+        // Original Odoo bug had them backwards in addons/web/static/src/legacy/js/views/list/list_controller.js:658-659
+        if (notif && resIds.length === session.active_ids_limit && resIds.length < total) {
+            const msg = _.str.sprintf(
+                _t("Of the %d records selected, only the first %d have been archived/unarchived."),
+                resIds.length,  // Fixed: total records selected
+                total           // Fixed: number actually archived/unarchived  
+            );
+            this.displayNotification({ title: _t('Warning'), message: msg });
+        }
+    },
+});
+