From 53357314d4ccfc647bc850034312f8a34596576e Mon Sep 17 00:00:00 2001
From: Debora Crescenzo <dcrescenzo@sipwise.com>
Date: Mon, 16 Feb 2026 16:10:59 +0000
Subject: [PATCH] MT#64432 Refactor updateSharedValue and getList

* fix updateSharedValue to be less flacky: it was using
  the reference of row, mutated, to make the backend request.
  It now use a value stored in a variable.
* amend getList to make sure we pass the default page (1)
  when looking for all rows with api v1.

Change-Id: Ia8c4fcb1547c16a6a3e862f864fbc5aecda3e065
(cherry picked from commit 9db53bc3cec1177f0a1854402fa2c96742ca0fdc)
---
 src/api/common.js                 | 1 +
 src/store/subscriber-phonebook.js | 5 +++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/api/common.js b/src/api/common.js
index 72f577c8..8de81533 100644
--- a/src/api/common.js
+++ b/src/api/common.js
@@ -101,6 +101,7 @@ export async function getList (options) {
     }, options)
     if (requestConfig.all === true) {
         requestConfig.params.rows = LIST_ALL_ROWS
+        requestConfig.params.page = LIST_DEFAULT_PAGE
     }
     if (requestConfig.resource !== undefined) {
         requestConfig.path = `api/${requestConfig.resource}/`
diff --git a/src/store/subscriber-phonebook.js b/src/store/subscriber-phonebook.js
index e4a4c98c..eee2fdce 100644
--- a/src/store/subscriber-phonebook.js
+++ b/src/store/subscriber-phonebook.js
@@ -85,8 +85,9 @@ export default {
             await updateEntry(data)
         },
         async updateSharedValue (context, row) {
-            context.commit('setSharedValue', { id: row.id, value: !row.shared })
-            await setSharedValue(row.subscriber_id, row.id, row.shared)
+            const newValue = !row.shared
+            context.commit('setSharedValue', { id: row.id, value: newValue })
+            await setSharedValue(row.id, newValue)
         }
     }
 }