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 9db53bc3ce)
(cherry picked from commit fd4e43ef8e)

src/api/common.js

@@ -94,6 +94,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}/`

src/api/subscriber-phonebook.js

@@ -32,16 +32,16 @@ export async function getEntryById (subscriberId, id) {
     })
 }
 
-export async function getPhonebook (options) {
+export async function getPhonebook (data) {
     return getList({
-        path: `api/v2/subscribers/${options.subscriber_id}/phonebook`,
-        params: options
+        path: `api/v2/subscribers/${data.subscriber_id}/phonebook`,
+        params: data
     })
 }
 
-export function setSharedValue (subscriberId, phonebookId, value) {
+export function setSharedValue (data, value) {
     return patchReplace({
-        path: `api/v2/subscribers/${subscriberId}/phonebook/${phonebookId}`,
+        path: `api/v2/subscribers/${data.subscriber_id}/phonebook/${data.id}`,
         fieldPath: 'shared',
         value
     })

src/store/subscriber-phonebook.js

@@ -83,8 +83,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, newValue)
         }
     }
 }