[FIX] swap parameter

Author: emjay0921

spp_base_common/__manifest__.py

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

spp_base_common/readme/ARCHIVING_FIX.md

@@ -0,0 +1,36 @@
+## Bugfix: Archive Notification Message
+
+### Issue
+Odoo core has a bug in the archive notification where the parameters are swapped, showing an illogical message like:
+> "Of the 20,000 records selected, only the first 50,000 have been archived/unarchived."
+
+This is impossible since you cannot archive more records than were selected.
+
+### Root Cause
+**Location**: `odoo/addons/web/static/src/legacy/js/views/list/list_controller.js:658-659`
+
+**Bug**: The sprintf parameters are in the wrong order:
+```javascript
+const msg = _.str.sprintf(
+    _t("Of the %d records selected, only the first %d have been archived/unarchived."),
+    total, resIds.length  // ← WRONG ORDER
+);
+```
+
+### Fix
+This module patches the `ListController._toggleArchiveState` method to swap the parameters:
+```javascript
+const msg = _.str.sprintf(
+    _t("Of the %d records selected, only the first %d have been archived/unarchived."),
+    resIds.length, total  // ← CORRECT ORDER  
+);
+```
+
+### Result
+After the fix, the notification correctly shows:
+> "Of the 50,000 records selected, only the first 20,000 have been archived/unarchived."
+
+### References
+- Odoo Repository: https://github.com/odoo/odoo/tree/17.0/odoo/addons/base
+- Fixed in: `spp_base_common/static/src/legacy/js/views/list/list_controller_fix.js`
+

spp_base_common/readme/BUGFIX_archive_notification.md

@@ -0,0 +1,41 @@
+odoo.define('spp_base_common.ListControllerFix', function (require) {
+'use strict';
+
+/**
+ * Patch for Odoo bug: swapped parameters in archive notification message.
+ * 
+ * Bug location: odoo/addons/web/static/src/legacy/js/views/list/list_controller.js:658-659
+ * Bug report: https://github.com/odoo/odoo/issues/xxxxx (if applicable)
+ * 
+ * Original shows: "Of the 20,000 records selected, only the first 50,000 have been archived"
+ * Fixed to show: "Of the 50,000 records selected, only the first 20,000 have been archived"
+ */
+
+var ListController = require('web.ListController');
+var session = require('web.session');
+var core = require('web.core');
+
+var _t = core._t;
+
+ListController.include({
+    /**
+     * @override
+     */
+    _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;
+        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: swapped - total selected (larger)
+                total           // Fixed: swapped - actually processed (smaller)  
+            );
+            this.displayNotification({ title: _t('Warning'), message: msg });
+        }
+    },
+});
+
+});
+