From 93c393d8ebb093f80504eb3d95ef1d018a5302c1 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Tue, 25 Mar 2025 09:08:45 -0300
Subject: [PATCH 01/11] Update WhatsNew.vue to enhance user feedback and
 accessibility features

---
 src/components/WhatsNew.vue | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/components/WhatsNew.vue b/src/components/WhatsNew.vue
index 8f486c0f..1a5a1523 100644
--- a/src/components/WhatsNew.vue
+++ b/src/components/WhatsNew.vue
@@ -14,25 +14,25 @@
       <p class="font-weight-bold">Changing your assessment won’t delete your explanation</p>
       <ul>
         <li>
-        Based on user feedback, iSoQ will no longer delete the text you have written in an explanation box if you change your choice of level of concern or level of confidence.  
+          Based on user feedback, iSoQ will no longer delete the text you have written in an explanation box if you change your choice of level of concern or level of confidence.
         </li>
       </ul>
       <p class="font-weight-bold">Users with “view only” rights now see Worksheets</p>
       <ul>
         <li>
-       When a user is invited to a project but assigned “view only” rights they can now see (but not edit) all pages including project properties, My Data section, iSoQ table and Worksheets with the Evidence Profile. 
+          When a user is invited to a project but assigned “view only” rights they can now see (but not edit) all pages including project properties, My Data section, iSoQ table and Worksheets with the Evidence Profile.
         </li>
       </ul>
       <p class="font-weight-bold">Accessibility: Increase and decrease text size</p>
       <ul>
         <li>
-       You will now find a +A and -A in the top right corner of the screen so that you can adjust the size of the font throughout iSoQ.
+          You will now find a +A and -A in the top right corner of the screen so that you can adjust the size of the font throughout iSoQ.
         </li>
       </ul>
        <p class="font-weight-bold">Prompt to check your references, success starts here!</p>
       <ul>
         <li>
-       When you import your references in Step 1 of the MyData section of iSoQ, your imported studies are now displayed, thus prompting you to look them over before manually moving to Step 2. It is important to make sure that your references have imported correctly and are complete. You cannot currently edit your references within iSoQ. 
+          When you import your references in Step 1 of the MyData section of iSoQ, your imported studies are now displayed, thus prompting you to look them over before manually moving to Step 2. It is important to make sure that your references have imported correctly and are complete. You cannot currently edit your references within iSoQ.
         </li>
       </ul>
 
@@ -40,16 +40,15 @@
       <p class="font-weight-bold">Narrower margins = easier viewing of content</p>
       <ul>
         <li>
-        The margins have been reduced throughout the platform to maximize what you are able to see in tables and modals. 
+            The margins have been reduced throughout the platform to maximize what you are able to see in tables and modals.
         </li>
       </ul>
-          <p class="font-weight-bold">New section in help tab – iSoQ training</p>
+      <p class="font-weight-bold">New section in help tab – iSoQ training</p>
       <ul>
         <li>
-        We have added links to iSoQ training webinars on the Help page
+          We have added links to iSoQ training webinars on the Help page
         </li>
       </ul>
-      
       <h3>May 2024</h3>
       <!-- acá escribes tu comentario -->
       <p class="font-weight-bold">Facilitating access to information on iSoQ development </p>

From 55fb31b413fad0e125538757b5755918d5b12781 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Tue, 25 Mar 2025 15:21:37 -0300
Subject: [PATCH 02/11] remove unused code

---
 src/components/project/viewProject.vue | 238 -------------------------
 1 file changed, 238 deletions(-)

diff --git a/src/components/project/viewProject.vue b/src/components/project/viewProject.vue
index 170e8ae7..956c7256 100644
--- a/src/components/project/viewProject.vue
+++ b/src/components/project/viewProject.vue
@@ -372,211 +372,6 @@
             </b-card>
           </b-col>
           <b-col cols="12" class="toDoc">
-            <!--
-            <template v-if="mode==='edit'">
-              <b-table
-                :selectable="(mode==='view')?true:false"
-                select-mode="multi"
-                selected-variant="warning"
-                bordered
-                head-variant="light"
-                id="findings"
-                ref="findings"
-                sort-by="sort"
-                :fields="(list_categories.options.length)?fields.with_categories:fields.without_categories"
-                :items="lists"
-                show-empty
-                :busy="table_settings.isBusy"
-                :current-page="table_settings.currentPage"
-                :filter="table_settings.filter"
-                @filtered="onFiltered"
-                :filter-included-fields="table_settings.filterOn">
-                <template v-slot:head(sort)="data">
-                  <span v-b-tooltip.hover title="Automatic numbering of summarised review findings">{{ data.label }}</span>
-                </template>
-                <template v-slot:head(name)="data">
-                  <span v-b-tooltip.hover title="Summaries of each review finding produced by the review team">{{ data.label }}</span>
-                </template>
-                <template v-slot:head(category_name)="data">
-                  {{data.label}}
-                  <b-dropdown
-                    id="dropdown-categories"
-                    text=""
-                    class="finding-filter"
-                    :no-caret="false"
-                    size="sm">
-                    <b-dropdown-item
-                    v-for="(category, index) of list_categories.options"
-                    :key="index"
-                    @click="tableFilter(category.text, 1)" :active="isFilterActive(category.text)">{{ category.text }}</b-dropdown-item>
-                  </b-dropdown>
-                  <span v-if="ui.project.showFilterOne" class="text-danger remove-opt" @click="cleanTableFilter">&times;</span>
-                </template>
-                <template v-slot:head(cerqual_option)="data">
-                  <span v-b-tooltip.hover title="Assessment of the extent to which a review finding is a reasonable representation of the phenomenon of interest">{{ data.label }}</span>
-                  <b-dropdown
-                    id="dropdown-cerqual-option"
-                    text=""
-                    class="finding-filter"
-                    :no-caret="false"
-                    size="sm">
-                    <b-dropdown-item @click="tableFilter('hc', 2)" :active="isFilterActive('hc')">High confidence</b-dropdown-item>
-                    <b-dropdown-item @click="tableFilter('mc', 2)" :active="isFilterActive('mc')">Moderate confidence</b-dropdown-item>
-                    <b-dropdown-item @click="tableFilter('lc', 2)" :active="isFilterActive('lc')">Low confidence</b-dropdown-item>
-                    <b-dropdown-item @click="tableFilter('vc', 2)" :active="isFilterActive('vc')">Very low confidence</b-dropdown-item>
-                    <b-dropdown-divider></b-dropdown-divider>
-                    <b-dropdown-item @click="tableFilter('completed', 2)" :active="isFilterActive('completed')">Assessments completed</b-dropdown-item>
-                    <b-dropdown-item @click="tableFilter('unfinished', 2)" :active="isFilterActive('unfinished')">Assessments not completed</b-dropdown-item>
-                  </b-dropdown>
-                  <span v-if="ui.project.showFilterTwo" class="text-danger remove-opt" @click="cleanTableFilter">&times;</span>
-                </template>
-                <template v-slot:head(cerqual_explanation)="data">
-                  <span v-b-tooltip.hover title="Statement explaining concerns with any of the GRADE-CERQual components that justifies the level of confidence chosen">{{ data.label }}</span>
-                  <b-dropdown
-                    id="dropdown-cerqual-explanation"
-                    text=""
-                    class="finding-filter"
-                    :no-caret="false"
-                    size="sm">
-                    <b-dropdown-item @click="tableFilter('with_explanation', 3)" :active="isFilterActive('with_explanation')">Completed</b-dropdown-item>
-                    <b-dropdown-item @click="tableFilter('without_explanation', 3)" :active="isFilterActive('without_explanation')">Not completed</b-dropdown-item>
-                  </b-dropdown>
-                  <span v-if="ui.project.showFilterThree" class="text-danger remove-opt" @click="cleanTableFilter">&times;</span>
-                </template>
-                <template v-slot:head(ref_list)="data">
-                  <span v-b-tooltip.hover title="Studies that contribute to each review finding">{{ data.label }}</span>
-                </template>
-                
-                <template v-slot:cell(sort)="data">
-                  {{(Object.prototype.hasOwnProperty.call(data.item, 'sort')) ? data.item.sort : data.index + 1}}
-                </template>
-                <template v-slot:cell(name)="data">
-                  <a :id="`a-${data.item.id}`"></a>
-                  <template v-if="mode === 'edit' && checkPermissions()">
-                    <b-row
-                      class="mb-3">
-                      <b-col
-                        lg="6"
-                        cols="12">
-                        <b-button
-                          block
-                          v-if="mode==='edit'"
-                          variant="outline-success"
-                          @click="editModalFindingName(data)">
-                          <font-awesome-icon
-                            v-if=(data.item.notes.length)
-                            icon="comments"></font-awesome-icon>
-                          Edit
-                        </b-button>
-                      </b-col>
-                      <b-col
-                        class="mt-1 mt-lg-0"
-                        lg="6"
-                        cols="12">
-                        <b-button
-                          block
-                          v-if="mode==='edit'"
-                          variant="outline-danger"
-                          @click="removeModalFinding(data)">
-                          Remove
-                        </b-button>
-                      </b-col>
-                    </b-row>
-                    <b-link class="table-edit-list" v-if="data.item.references.length" :to="{name: 'editList', params: {id: data.item.id}}">{{ data.item.name }}</b-link>
-                    <span v-if="data.item.references.length === 0">{{ data.item.name }}</span>
-                  </template>
-                  <template v-else>
-                    <b-link class="table-edit-list" v-if="data.item.references.length" :to="{name: 'editList', params: {id: data.item.id}}">{{ data.item.name }}</b-link>
-                    <span v-if="data.item.references.length === 0">{{ data.item.name }}</span>
-                  </template>
-                </template>
-                <template v-slot:cell(category_name)="data">
-                  <template v-if="checkPermissions()">
-                    <template v-if="data.item.category !== null">
-                      <b-button
-                        block
-                        variant="outline-info"
-                        @click="editModalFindingName(data)">Edit group</b-button>
-                      {{ data.item.category_name }}
-                      <span
-                        v-if="data.item.category_extra_info !== ''"
-                        v-b-tooltip.hover
-                        :title="data.item.category_extra_info">*</span>
-                    </template>
-                    <template v-else>
-                      <b-button
-                        v-if="mode==='edit' && data.item.references.length"
-                        variant="info"
-                        block
-                        @click="editModalFindingName(data)">Assign group</b-button>
-                    </template>
-                  </template>
-                </template>
-                <template v-slot:cell(cerqual_option)="data">
-                  <b-button
-                    v-if="mode==='edit' && data.item.references.length && checkPermissions()"
-                    class="d-print-none mb-3"
-                    :disabled="(data.item.references.length) ? false : true"
-                    block
-                    :variant="(data.item.cerqual_option === '') ? 'info' : 'outline-info'"
-                    :to="{name: 'editList', params: {id: data.item.id}}">
-                      <font-awesome-icon
-                        v-if="Object.prototype.hasOwnProperty.call(data.item, 'evidence_profile') && (data.item.evidence_profile.methodological_limitations.notes || data.item.evidence_profile.coherence.notes || data.item.evidence_profile.adequacy.notes || data.item.evidence_profile.relevance.notes || data.item.evidence_profile.cerqual.notes)"
-                        icon="comments"></font-awesome-icon>
-                      <span v-if="data.item.cerqual_option===''">Complete</span>
-                      <span v-if="data.item.cerqual_option!=''">Edit</span>
-                      GRADE-CERQual Assessment
-                    </b-button>
-                  <b>{{ data.item.cerqual_option }}</b>
-                </template>
-                <template v-slot:cell(cerqual_explanation)="data">
-                  <b-button
-                    v-if="mode==='edit' && data.item.references.length && checkPermissions()"
-                    class="d-print-none mb-3"
-                    :disabled="(data.item.references.length) ? false : true"
-                    block
-                    :variant="(data.item.cerqual_explanation==='') ? 'info' : 'outline-info'"
-                    :to="{name: 'editList', params: {id: data.item.id}}">
-                      <font-awesome-icon
-                        v-if="Object.prototype.hasOwnProperty.call(data.item, 'evidence_profile') && (data.item.evidence_profile.methodological_limitations.notes || data.item.evidence_profile.coherence.notes || data.item.evidence_profile.adequacy.notes || data.item.evidence_profile.relevance.notes || data.item.evidence_profile.cerqual.notes)"
-                        icon="comments"></font-awesome-icon>
-                      <span v-if="data.item.cerqual_explanation===''">Complete</span>
-                      <span v-if="data.item.cerqual_explanation!=''">Edit</span>
-                      GRADE-CERQual Assessment
-                  </b-button>
-                  <b class="cerqual-explanation" v-if="data.item.cerqual_option !== ''">{{ data.item.cerqual_explanation }}</b>
-                </template>
-                <template v-slot:cell(ref_list)="data">
-                  <template v-if="mode!=='edit'">
-                    {{ data.item.ref_list }}
-                  </template>
-                  <template v-else>
-                    <b-button
-                      v-if="checkPermissions()"
-                      block
-                      class="mb-3 d-print-none"
-                      :variant="(data.item.references.length) ? 'outline-info' : 'info'"
-                      @click="openModalReferences(data)">
-                      <span v-if="data.item.references.length">View or edit references</span>
-                      <span v-else>Select references</span>
-                    </b-button>
-                    There are <b>{{ data.item.raw_ref.length }}</b> references.
-                  </template>
-                </template>
-                <template v-slot:empty>
-                  <p class="text-center my-5">
-                    There are no findings to show, <a href="#" @click="modalAddList">add review finding</a>
-                  </p>
-                </template>
-                <template v-slot:table-busy>
-                  <div class="text-center text-danger my-2">
-                    <b-spinner class="align-middle"></b-spinner>
-                    <strong>Loading...</strong>
-                  </div>
-                </template>
-              </b-table>
-            </template>
-            -->
             <template
               v-if="mode==='edit' && checkPermissions()">
               <ViewTable
@@ -1200,26 +995,6 @@ export default {
     },
     clickTab: function (option) {
       this.tabOpened = option
-      // let theHash = ''
-      // switch (option) {
-      //   case 0:
-      //     theHash = 'Project-Property'
-      //     break
-      //   case 1:
-      //     theHash = 'My-Data'
-      //     break
-      //   case 2:
-      //     theHash = 'iSoQ'
-      //     break
-      //   case 3:
-      //     theHash = 'Guidance-on-applying-GRADE-CERQual'
-      //     break
-      // }
-      // if (Object.hasOwnProperty.call(this.$route.query, 'step')) {
-      //   this.$router.push({query: {tab: `${theHash}`, step: this.$route.query.step}})
-      // } else {
-      //   this.$router.push({query: {tab: `${theHash}`}})
-      // }
     },
     uiShowLoaders: function (status) {
       this.ui.publish.showLoader = status
@@ -1237,7 +1012,6 @@ export default {
         for (const r of this.references) {
           if (r.id === array[i]) {
             authorsList.push(this.getAuthorsFormat(r.authors, r.publication_year))
-            // authors = this.getAuthorsFormat(r.authors, r.publication_year) + '; ' + authors
           }
         }
       }
@@ -1989,22 +1763,10 @@ export default {
     #chars-of-studies-table thead th:first-child {
       width: 25%;
     }
-    /* #chars-of-studies-table tbody tr td button {
-      display: none;
-    }
-    #chars-of-studies-table tbody tr:hover td button {
-      display: inline;
-    } */
   div >>>
     #methodological-table thead th:first-child {
       width: 25%;
     }
-    /* #methodological-table tbody tr td button {
-      display: none;
-    }
-    #methodological-table tbody tr:hover td button {
-      display: inline;
-    } */
   div >>>
     #extracted-data-table thead th:first-child {
       width: 25%;

From b02a44ce8f5d736e7bf6059112b47bbe9590d967 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Tue, 25 Mar 2025 15:34:25 -0300
Subject: [PATCH 03/11] Refactor error handling in viewProject.vue to use
 Commons.printErrors method

---
 src/components/project/viewProject.vue | 17 +----------------
 1 file changed, 1 insertion(+), 16 deletions(-)

diff --git a/src/components/project/viewProject.vue b/src/components/project/viewProject.vue
index 956c7256..52fa29a7 100644
--- a/src/components/project/viewProject.vue
+++ b/src/components/project/viewProject.vue
@@ -1637,22 +1637,7 @@ export default {
       return _category
     },
     printErrors: function (error) {
-      if (error.response) {
-        // The request was made and the server responded with a status code
-        // that falls out of the range of 2xx
-        console.log(error.response.data)
-        console.log(error.response.status)
-        console.log(error.response.headers)
-      } else if (error.request) {
-        // The request was made but no response was received
-        // `error.request` is an instance of XMLHttpRequest in the browser and an instance of
-        // http.ClientRequest in node.js
-        console.log(error.request)
-      } else {
-        // Something happened in setting up the request that triggered an Error
-        console.log('Error', error.message)
-      }
-      console.log(error.config)
+      this.Commons.printErrors(error)
     },
     createExtractedData: async function (findingID) {
       const _references = JSON.parse(JSON.stringify(this.references))

From 2cccfabd3cc17e6e78cee778e180fe1a0de97af0 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Tue, 25 Mar 2025 16:26:32 -0300
Subject: [PATCH 04/11] Add parseReference method to Commons for formatted
 reference output

---
 src/utils/commons.js | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/src/utils/commons.js b/src/utils/commons.js
index 17775544..e738d3bc 100644
--- a/src/utils/commons.js
+++ b/src/utils/commons.js
@@ -16,6 +16,28 @@ export default class Commons {
     return authors
   }
 
+  static parseReference (reference, onlyAuthors = false, hasSemicolon = true) {
+    let result = ''
+    const semicolon = hasSemicolon ? '; ' : ''
+    if (Object.prototype.hasOwnProperty.call(reference, 'authors')) {
+      if (reference.authors.length) {
+        if (reference.authors.length === 1) {
+          result = reference.authors[0].split(',')[0] + ' ' + reference.publication_year + semicolon
+        } else if (reference.authors.length === 2) {
+          result = reference.authors[0].split(',')[0] + ' & ' + reference.authors[1].split(',')[0] + ' ' + reference.publication_year + semicolon
+        } else {
+          result = reference.authors[0].split(',')[0] + ' et al. ' + reference.publication_year + semicolon
+        }
+        if (!onlyAuthors) {
+          result = result + reference.title
+        }
+      } else {
+        return 'author(s) not found'
+      }
+    }
+    return result
+  }
+
   static getAuthorsFormat (authors = [], pubYear = '') {
     if (authors.length) {
       const nroAuthors = authors.length

From 9939263b2a73f12705f4a21693ef1ac9796ac98d Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Tue, 25 Mar 2025 16:37:42 -0300
Subject: [PATCH 05/11] Refactor parseReference method in crudTables.vue to use
 Commons utility

---
 src/components/project/crudTables.vue | 22 +++-------------------
 1 file changed, 3 insertions(+), 19 deletions(-)

diff --git a/src/components/project/crudTables.vue b/src/components/project/crudTables.vue
index 20a243d4..4231c881 100644
--- a/src/components/project/crudTables.vue
+++ b/src/components/project/crudTables.vue
@@ -353,6 +353,7 @@
 <script>
 import axios from 'axios'
 import Papa from 'papaparse'
+import Commmons from '@/utils/commons.js'
 const ExportCSV = require('export-to-csv').ExportToCsv
 
 export default {
@@ -975,6 +976,7 @@ export default {
       }
       for (const reference of this.references) {
         if (!references.includes(reference.id)) {
+          console.log('this.processItems')
           newItems.push({
             ref_id: reference.id,
             authors: this.parseReference(reference, true, false)
@@ -985,25 +987,7 @@ export default {
       return items
     },
     parseReference: (reference, onlyAuthors = false, hasSemicolon = true) => {
-      let result = ''
-      const semicolon = hasSemicolon ? '; ' : ''
-      if (Object.prototype.hasOwnProperty.call(reference, 'authors')) {
-        if (reference.authors.length) {
-          if (reference.authors.length === 1) {
-            result = reference.authors[0].split(',')[0] + ' ' + reference.publication_year + semicolon
-          } else if (reference.authors.length === 2) {
-            result = reference.authors[0].split(',')[0] + ' & ' + reference.authors[1].split(',')[0] + ' ' + reference.publication_year + semicolon
-          } else {
-            result = reference.authors[0].split(',')[0] + ' et al. ' + reference.publication_year + semicolon
-          }
-          if (!onlyAuthors) {
-            result = result + reference.title
-          }
-        } else {
-          return 'author(s) not found'
-        }
-      }
-      return result
+      return Commmons.parseReference(reference, onlyAuthors, hasSemicolon)
     },
     deleteFieldFromCharsSudiesEdit: function (index) {
       let params = {}

From d9164cb3414600ef847496177fc6ff1217af57c1 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Wed, 7 May 2025 10:58:56 -0400
Subject: [PATCH 06/11] Add placeholder text to textarea in crudTables.vue for
 improved user guidance

---
 src/components/project/crudTables.vue | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/components/project/crudTables.vue b/src/components/project/crudTables.vue
index 4231c881..0f3862fa 100644
--- a/src/components/project/crudTables.vue
+++ b/src/components/project/crudTables.vue
@@ -243,6 +243,7 @@
               <b-form-textarea
                 v-if="!['ref_id', 'authors'].includes(field.key)"
                 v-model="dataTableFieldsModal.items[dataTableFieldsModal.selected_item_index][field.key]"
+                :placeholder="(type === 'isoqf_assessments') ? 'Enter both your assessment and the explanation for your assessment here' : ''"
                 rows="2"
                 max-rows="100"></b-form-textarea>
             </template>

From ba1094158ce661bbe4d210fc2804d65bce6283b3 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Mon, 2 Jun 2025 10:36:47 -0400
Subject: [PATCH 07/11] Initialize sharedToken property and ensure it defaults
 to an empty string if undefined or null

---
 src/components/organization/viewOrganization.vue | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/components/organization/viewOrganization.vue b/src/components/organization/viewOrganization.vue
index 46a30c91..754c8a65 100644
--- a/src/components/organization/viewOrganization.vue
+++ b/src/components/organization/viewOrganization.vue
@@ -621,6 +621,12 @@ export default {
         project.can_read.includes(this.$store.state.user.id) ||
         project.can_write.includes(this.$store.state.user.id)
       ) {
+        if (!Object.prototype.hasOwnProperty.call(project, 'sharedToken')) {
+          project.sharedToken = ''
+        }
+        if (project.sharedToken === null || project.sharedToken === undefined) {
+          project.sharedToken = ''
+        }
         if (!Object.prototype.hasOwnProperty.call(project, 'sharedTokenOnOff')) {
           if (Object.prototype.hasOwnProperty.call(project, 'sharedToken') && project.sharedToken.length) {
             project.sharedTokenOnOff = true

From e05a9b1f7da18fc14590ade5eb3822de056891e8 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Tue, 12 Aug 2025 08:23:21 -0400
Subject: [PATCH 08/11] remove bullet point for methodological assessments

---
 src/components/project/crudTables.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/project/crudTables.vue b/src/components/project/crudTables.vue
index 0f3862fa..7bdc4787 100644
--- a/src/components/project/crudTables.vue
+++ b/src/components/project/crudTables.vue
@@ -134,7 +134,7 @@
         </p>
         <ul class="font-weight-light text-danger">
           <li>Do not add columns for author or year (these will be added automatically)</li>
-          <li>Do not add methodological assessments (critical/quality appraisal). These go in a separate table.</li>
+          <li v-if="type !== 'isoqf_assessments'">Do not add methodological assessments (critical/quality appraisal). These go in a separate table.</li>
         </ul>
         <b-form-group
           label="Number of columns">

From 4e5b167317143348234cf8337882b0e2f0b356a8 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Wed, 24 Sep 2025 09:23:40 -0300
Subject: [PATCH 09/11] Refactor mode handling in ViewTable and actionButtons
 components for improved permission-based logic

---
 src/components/project/ViewTable.vue     | 13 ++--
 src/components/project/actionButtons.vue |  6 +-
 src/components/project/viewProject.vue   | 75 +++++++++++++++++++-----
 3 files changed, 74 insertions(+), 20 deletions(-)

diff --git a/src/components/project/ViewTable.vue b/src/components/project/ViewTable.vue
index 158904be..6ec0d556 100644
--- a/src/components/project/ViewTable.vue
+++ b/src/components/project/ViewTable.vue
@@ -1,7 +1,6 @@
 <template>
   <div>
     <b-table
-      :selectable="(mode==='view')?true:false"
       select-mode="multi"
       selected-variant="warning"
       bordered
@@ -112,12 +111,18 @@
           <span v-if="data.item.references.length === 0">{{ data.item.name }}</span>
         </span>
         <span v-else>
-          {{ data.item.name }}
+          <template v-if="mode==='view' && data.item.references.length">
+            <b-link class="table-edit-list" :to="{name: 'editList', params: {id: data.item.id}}">{{ data.item.name }}</b-link>
+          </template>
+          <template v-else>
+            {{ data.item.name }}
+          </template>
         </span>
       </template>
       <template v-slot:cell(category_name)="data">
         <template v-if="data.item.category !== null">
           <b-button
+            v-if="mode==='edit'"
             block
             variant="outline-info"
             @click="editModalFindingName(data)">Edit group</b-button>
@@ -455,8 +460,8 @@ export default {
     },
     mode: {
       type: String,
-      required: true,
-      default: 'view'
+      required: false,
+      default: ''
     },
     isBusy: {
       type: Boolean,
diff --git a/src/components/project/actionButtons.vue b/src/components/project/actionButtons.vue
index 1ceceb57..330e9262 100644
--- a/src/components/project/actionButtons.vue
+++ b/src/components/project/actionButtons.vue
@@ -166,7 +166,11 @@ const videoHelp = () => import(/* webpackChunkName: "videohelp" */ '../videoHelp
 export default {
   name: 'actionButtons',
   props: {
-    mode: String,
+    mode: {
+      type: String,
+      required: false,
+      default: ''
+    },
     preview: {
       type: Boolean,
       default: false
diff --git a/src/components/project/viewProject.vue b/src/components/project/viewProject.vue
index 52fa29a7..e6df467e 100644
--- a/src/components/project/viewProject.vue
+++ b/src/components/project/viewProject.vue
@@ -164,7 +164,7 @@
       </div>
       <div :class="{'block mt-3': (tabOpened===2)?true:false, 'd-none': (tabOpened===2)?!true:!false}" :disabled="(references.length) ? false : true">
         <action-buttons
-          :mode="mode"
+          :mode="effectiveMode"
           :permissions="checkPermissions()"
           :project="project"
           :ui="ui"
@@ -253,7 +253,7 @@
             <b-row
               class="mb-2">
               <b-col
-                v-if="mode!=='view'"
+                v-if="effectiveMode === 'edit'"
                 md="3"
                 cols="12">
                 <b-button
@@ -266,7 +266,7 @@
                 </b-button>
               </b-col>
               <b-col
-                v-if="mode!=='view'"
+                v-if="effectiveMode === 'edit'"
                 md="4"
                 cols="12">
                 <b-button
@@ -279,7 +279,7 @@
                 </b-button>
               </b-col>
               <b-col
-                v-if="mode!=='view' && lists.length > 1"
+                v-if="effectiveMode === 'edit' && lists.length > 1"
                 md="3"
                 cols="12">
                 <b-button
@@ -321,7 +321,7 @@
                 </b-modal>
               </b-col>
               <b-col
-                v-if="mode!=='view' && lists.length > 1"
+                v-if="effectiveMode === 'edit' && lists.length > 1"
                 md="2"
                 cols="12">
                 <b-button
@@ -333,7 +333,7 @@
             </b-row>
           </b-col>
           <b-col
-            v-if="mode==='edit' && this.lists.length && ui.project.displaySearch"
+            v-if="effectiveMode === 'edit' && lists.length && ui.project.displaySearch"
             cols="12"
             class="my-2 d-print-none">
             <b-card
@@ -373,13 +373,13 @@
           </b-col>
           <b-col cols="12" class="toDoc">
             <template
-              v-if="mode==='edit' && checkPermissions()">
+              v-if="checkPermissions(['can_read', 'can_write'])">
               <ViewTable
                 :lists="lists"
                 :list_categories="list_categories"
                 :fields="fields"
                 :project="project"
-                :mode="mode"
+                :mode="effectiveMode"
                 :isBusy="table_settings.isBusy"
                 :references="references"
                 :refs="refs"
@@ -752,7 +752,7 @@ export default {
       fileReferences: [],
       selected_list_index: null,
       lastId: 1,
-      mode: 'edit',
+      mode: '',
       msgUploadReferences: '',
       charsOfStudies: {
         fields: [],
@@ -888,6 +888,14 @@ export default {
             }
           }
           this.project = _project
+          // set mode based on permissions: prefer write -> edit, otherwise read -> view
+          if (this.checkPermissions('can_write')) {
+            this.mode = 'edit'
+          } else if (this.checkPermissions('can_read')) {
+            this.mode = 'view'
+          } else {
+            this.mode = ''
+          }
           this.ui.project.show_criteria = true
           this.getLists()
         })
@@ -1688,16 +1696,41 @@ export default {
       this.clickTab(2)
     },
     checkPermissions: function (type = 'can_write') {
-      if (this.$store.state.user.personal_organization === this.$route.params.org_id) {
-        return true
+      // normalize input to an array of permission keys
+      let perms = []
+      if (Array.isArray(type)) {
+        perms = type
+      } else if (typeof type === 'string') {
+        perms = type.split(',').map(t => t.trim()).filter(Boolean)
       } else {
-        if (!Object.prototype.hasOwnProperty.call(this.project, type)) {
-          return false
+        perms = ['can_write']
+      }
+
+      // if the current user belongs to the same personal organization, allow
+      if (this.$store && this.$store.state && this.$store.state.user && this.$store.state.user.personal_organization === this.$route.params.org_id) {
+        return true
+      }
+
+      // check any of the requested permissions on the project
+      for (const perm of perms) {
+        if (!Object.prototype.hasOwnProperty.call(this.project, perm)) {
+          continue
         }
-        if (this.project[type].includes(this.$store.state.user.id)) {
-          return true
+
+        const val = this.project[perm]
+        // val is expected to be an array of user ids, but could be a comma-separated string
+        if (Array.isArray(val)) {
+          if (val.includes(this.$store.state.user.id)) {
+            return true
+          }
+        } else if (typeof val === 'string') {
+          const arr = val.split(',').map(x => x.trim()).filter(Boolean)
+          if (arr.includes(String(this.$store.state.user.id))) {
+            return true
+          }
         }
       }
+
       return false
     },
     updateModificationTime: function () {
@@ -1719,6 +1752,18 @@ export default {
       }
       txt = txt + 'Summary of Qualitative Findings Table'
       return txt
+    },
+    effectiveMode: function () {
+      // If explicit mode is set to edit or view, use it
+      if (this.mode === 'edit') return 'edit'
+      if (this.mode === 'view') return 'view'
+
+      // If mode is empty, derive from permissions: prefer write
+      if (this.checkPermissions('can_write')) return 'edit'
+      if (this.checkPermissions('can_read')) return 'view'
+
+      // safe default: empty string when user has no read/write permissions
+      return ''
     }
   }
 }

From 39c7f949ca041a11b83e6fcd352bbe0fef12eed4 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Wed, 24 Sep 2025 09:30:19 -0300
Subject: [PATCH 10/11] Remove multi-select mode from b-table in ViewTable
 component

---
 src/components/project/ViewTable.vue | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/components/project/ViewTable.vue b/src/components/project/ViewTable.vue
index 6ec0d556..41463779 100644
--- a/src/components/project/ViewTable.vue
+++ b/src/components/project/ViewTable.vue
@@ -1,7 +1,6 @@
 <template>
   <div>
     <b-table
-      select-mode="multi"
       selected-variant="warning"
       bordered
       head-variant="light"

From e398bb1cc068ec93f12392e3f454fbbe2a9a2f18 Mon Sep 17 00:00:00 2001
From: damian-garrido <damian@epistemonikos.org>
Date: Wed, 24 Sep 2025 12:48:14 -0300
Subject: [PATCH 11/11] Update terminology in evidenceProfileForm to
 consistently refer to "review finding" instead of "finding"

---
 src/components/list/evidenceProfileForm.vue | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/list/evidenceProfileForm.vue b/src/components/list/evidenceProfileForm.vue
index ceb07ef4..8c11306a 100644
--- a/src/components/list/evidenceProfileForm.vue
+++ b/src/components/list/evidenceProfileForm.vue
@@ -32,7 +32,7 @@
             </p>
             <p class="font-weight-light">
               <b><u>Remember</u></b> this is an assessment of the whole body of
-              evidence supporting this finding, not an assessment of an individual
+              evidence supporting this review finding, not an assessment of an individual
               contributing study. (guidance available
               <b-link :to="{
                 name: 'viewProject',
@@ -464,7 +464,7 @@
                   ui.methodological_assessments.extracted_data.display_warning
                 " class="text-danger">
                   <font-awesome-icon icon="exclamation-circle"></font-awesome-icon>
-                  Some or all of the extracted data for this finding are missing.
+                  Some or all of the extracted data for this review finding are missing.
                   Add them into the table below using the edit button for each
                   included study.
                 </p>
@@ -520,7 +520,7 @@
             <h4>Extracted Data</h4>
             <p v-if="ui.adequacy.extracted_data.display_warning" class="text-danger">
               <font-awesome-icon icon="exclamation-circle"></font-awesome-icon>
-              Some or all of the extracted data for this finding are missing. Add
+              Some or all of the extracted data for this review finding are missing. Add
               them into the table below using the edit button for each included
               study.
             </p>
@@ -575,7 +575,7 @@
                 <h4>Extracted Data</h4>
                 <p v-if="ui.adequacy.extracted_data.display_warning" class="text-danger">
                   <font-awesome-icon icon="exclamation-circle"></font-awesome-icon>
-                  Some or all of the extracted data for this finding are missing.
+                  Some or all of the extracted data for this review finding are missing.
                   Add them into the table below using the edit button for each
                   included study.
                 </p>