From 7048ef345fe8c8550fb97fcbfb1b215dd17a0e19 Mon Sep 17 00:00:00 2001
From: HangLe <hannyle91@gmail.com>
Date: Thu, 24 Nov 2022 15:08:38 +0200
Subject: [PATCH 1/5] Fix for data table's disabled options

---
 .../src/common/globalFunctions.js             | 13 ++++
 .../src/components/ContainerTable.vue         | 63 ++++++++++---------
 .../src/components/Containers.vue             |  5 +-
 .../src/components/ObjectTable.vue            |  7 ++-
 4 files changed, 53 insertions(+), 35 deletions(-)

diff --git a/swift_browser_ui_frontend/src/common/globalFunctions.js b/swift_browser_ui_frontend/src/common/globalFunctions.js
index e25dd90b8..ef3e49fa4 100644
--- a/swift_browser_ui_frontend/src/common/globalFunctions.js
+++ b/swift_browser_ui_frontend/src/common/globalFunctions.js
@@ -53,5 +53,18 @@ export async function getSharedContainers (projectId) {
     : [];
 }
 
+export async function getAccessDetails (
+  projectId,
+  folderName,
+  sourceProjectId)
+{
+  return store.state.client
+    ? await store.state.client.getAccessDetails(
+      projectId,
+      folderName,
+      sourceProjectId)
+    : [];
+}
+
 
 
diff --git a/swift_browser_ui_frontend/src/components/ContainerTable.vue b/swift_browser_ui_frontend/src/components/ContainerTable.vue
index 8ebfe6a2d..20137f22a 100644
--- a/swift_browser_ui_frontend/src/components/ContainerTable.vue
+++ b/swift_browser_ui_frontend/src/components/ContainerTable.vue
@@ -27,6 +27,7 @@ import {
   toggleCreateFolderModal,
   getSharingContainers,
   getSharedContainers,
+  getAccessDetails,
   toggleCopyFolderModal,
 } from "@/common/globalFunctions";
 import {swiftDeleteContainer} from "@/common/api";
@@ -139,7 +140,18 @@ export default {
         return "";
       };
 
-      this.containers = this.conts.slice(offset, offset + limit).reduce((
+      // Map the 'accessRights' to the container if it's a shared container
+      const mappedContainers = await Promise.all(this.conts.map(async(cont) => {
+        const sharedDetails = cont.owner ? await getAccessDetails(
+          this.$route.params.project,
+          cont.container,
+          cont.owner) : null;
+        const accessRights = sharedDetails ? sharedDetails.access : null;
+        return sharedDetails && accessRights ?
+          {...cont, accessRights} : {...cont};
+      }));
+
+      this.containers = mappedContainers.slice(offset, offset + limit).reduce((
         items,
         item,
       ) => {
@@ -230,35 +242,24 @@ export default {
                 },
               },
               // Share button is disabled for Shared (with you) Folders
-              item.owner ?
-                {
-                  value: null,
-                  component: {
-                    tag: "c-button",
-                    params: {
-                      text: true,
-                      size: "small",
-                      disabled: true,
-                    },
-                  },
-                } :
-                {
-                  value: this.$t("message.share.share"),
-                  component: {
-                    tag: "c-button",
-                    params: {
-                      text: true,
-                      size: "small",
-                      title: this.$t("message.share.share"),
-                      path: mdiShareVariantOutline,
-                      onClick: (item) => {
-                        this.$store.commit("toggleShareModal", true);
-                        this.$store.commit(
-                          "setFolderName", item.data.name.value);
-                      },
+              {
+                value: this.$t("message.share.share"),
+                component: {
+                  tag: "c-button",
+                  params: {
+                    text: true,
+                    size: "small",
+                    title: this.$t("message.share.share"),
+                    path: mdiShareVariantOutline,
+                    onClick: (item) => {
+                      this.$store.commit("toggleShareModal", true);
+                      this.$store.commit(
+                        "setFolderName", item.data.name.value);
                     },
+                    disabled: item.owner,
                   },
                 },
+              },
               {
                 value: null,
                 component: {
@@ -267,7 +268,7 @@ export default {
                     items: [
                       {
                         name: this.$t("message.copy"),
-                        action: item.owner 
+                        action: item.owner
                           ? () => toggleCopyFolderModal(item.name, item.owner)
                           : () => toggleCopyFolderModal(item.name),
                         disabled: !item.bytes ? true : false,
@@ -281,10 +282,12 @@ export default {
                         action: () => this.confirmDelete(
                           item.name, item.count,
                         ),
+                        disabled: item.owner && item.accessRights.length > 1,
                       },
                     ],
                     customTrigger: {
                       value: this.$t("message.options"),
+                      disabled: true,
                       component: {
                         tag: "c-button",
                         params: {
@@ -292,11 +295,11 @@ export default {
                           path: mdiDotsHorizontal,
                           title: "Menu with custom trigger",
                           size: "small",
+                          disabled: item.owner
+                            && item.accessRights.length === 1,
                         },
                       },
                     },
-                    path: mdiDotsHorizontal,
-
                   },
                 },
               },
diff --git a/swift_browser_ui_frontend/src/components/Containers.vue b/swift_browser_ui_frontend/src/components/Containers.vue
index 46879890a..629c85d62 100644
--- a/swift_browser_ui_frontend/src/components/Containers.vue
+++ b/swift_browser_ui_frontend/src/components/Containers.vue
@@ -94,9 +94,8 @@ export default {
           });
       }
       else if (this.$route.name === "SharedTo") {
-        this.renderingContainers = this.containers.filter(cont =>
-          cont.owner,
-        );
+        this.renderingContainers = this.containers ?
+          this.containers.filter(cont => cont.owner) : [];
       } else {
         this.renderingContainers = this.containers;
       }
diff --git a/swift_browser_ui_frontend/src/components/ObjectTable.vue b/swift_browser_ui_frontend/src/components/ObjectTable.vue
index 924c5d5d8..a0687e607 100644
--- a/swift_browser_ui_frontend/src/components/ObjectTable.vue
+++ b/swift_browser_ui_frontend/src/components/ObjectTable.vue
@@ -114,7 +114,10 @@
 <script>
 import { swiftDeleteObjects } from "@/common/api";
 import { getHumanReadableSize, truncate } from "@/common/conv";
-import { getSharedContainers } from "@/common/globalFunctions";
+import {
+  getSharedContainers,
+  getAccessDetails,
+} from "@/common/globalFunctions";
 import { liveQuery } from "dexie";
 import { useObservable } from "@vueuse/rxjs";
 import CObjectTable from "@/components/CObjectTable";
@@ -296,7 +299,7 @@ export default {
                 cont => cont.container === this.container) > -1) {
                 this.isSharedFolder = true;
                 const sharedDetails
-                  = await this.client.getAccessDetails(
+                  = await getAccessDetails(
                     this.project,
                     this.container,
                     this.$route.params.owner,

From 2202b694697d7aab1192ff9174adcf5e02da89b1 Mon Sep 17 00:00:00 2001
From: HangLe <hannyle91@gmail.com>
Date: Tue, 29 Nov 2022 10:34:02 +0200
Subject: [PATCH 2/5] Fix for sorting data table

---
 swift_browser_ui_frontend/src/common/conv.js | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/swift_browser_ui_frontend/src/common/conv.js b/swift_browser_ui_frontend/src/common/conv.js
index 50c59709a..e4655bb01 100644
--- a/swift_browser_ui_frontend/src/common/conv.js
+++ b/swift_browser_ui_frontend/src/common/conv.js
@@ -279,24 +279,30 @@ export function tokenize(text, ignoreSmallerThan = 2) {
 export const DEV = process.env.NODE_ENV === "development";
 
 export function sortObjects(objects, sortBy, sortDirection) {
-  sortBy = sortBy === "size" ? "bytes" : sortBy;
+  sortBy = sortBy === "size" ? "bytes"
+    : sortBy === "items" ? "count" : sortBy;
 
   objects.sort((a, b) => {
     let valueA = a[sortBy];
     let valueB = b[sortBy];
 
+
+
     // Handle tags as single string
     if (Array.isArray(valueA)) {
       valueA = valueA.join(" ");
       valueB = valueB.join(" ");
     }
 
+
     if (typeof valueA === "string") {
+      valueA = valueA.toLowerCase();
+      valueB = valueB.toLowerCase();
       if (sortDirection === "asc") {
-        return valueA.toLowerCase().localeCompare(valueB.toLowerCase());
+        return valueA < valueB ? -1 : valueA > valueB ? 1 : 0;
       }
 
-      return valueB.toLowerCase().localeCompare(valueA.toLowerCase());
+      return valueB < valueA ? -1 : valueB > valueA ? 1 : 0;
     }
 
     if (typeof valueA === "number") {

From 70701a8d4524a861d9c422992b262c40ec173354 Mon Sep 17 00:00:00 2001
From: HangLe <hannyle91@gmail.com>
Date: Tue, 29 Nov 2022 10:36:43 +0200
Subject: [PATCH 3/5] Fix for modals' width to have similarity

---
 swift_browser_ui_frontend/src/pages/BrowserPage.vue | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/swift_browser_ui_frontend/src/pages/BrowserPage.vue b/swift_browser_ui_frontend/src/pages/BrowserPage.vue
index a2593e0b6..5473d9378 100644
--- a/swift_browser_ui_frontend/src/pages/BrowserPage.vue
+++ b/swift_browser_ui_frontend/src/pages/BrowserPage.vue
@@ -10,6 +10,7 @@
       <c-modal
         v-control
         v-csc-model="openCreateFolderModal"
+        width="64vw"
       >
         <CreateFolderModal />
       </c-modal>
@@ -34,7 +35,7 @@
       <c-modal
         v-control
         v-csc-model="openShareModal"
-        width="50vw"
+        width="64vw"
       >
         <ShareModal />
       </c-modal>

From 1967d083a83a783f27d19de752a47df38ba44e93 Mon Sep 17 00:00:00 2001
From: HangLe <hannyle91@gmail.com>
Date: Tue, 29 Nov 2022 10:57:04 +0200
Subject: [PATCH 4/5] Updated Changelog

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index d7cf1fcc8..589b49adc 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -55,6 +55,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Made modal's scroll position to be always on top when opening a modal
   - Add tooltip for Copy Share ID button
   - Fine-tuned modals' font sizes and gaps between elements
+- (GH #871) Fixed for data table's folders' Options, sorting functionality, and modals' widths
 - (GH #858) Fixed for multiple bugs related to modals and background page's scrolling effect
 - (GH #853) Fix Node 18 needing python for npm install
 - (GH #827) Fixed for updating folder's items count and size when deleting objects inside it

From ccccd730c2bdeba6d3cdf28bc5d0bf4a790c184e Mon Sep 17 00:00:00 2001
From: HangLe <hannyle91@gmail.com>
Date: Tue, 27 Dec 2022 14:19:16 +0200
Subject: [PATCH 5/5] Remove empty lines

---
 swift_browser_ui_frontend/src/common/conv.js | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/swift_browser_ui_frontend/src/common/conv.js b/swift_browser_ui_frontend/src/common/conv.js
index e4655bb01..796660f22 100644
--- a/swift_browser_ui_frontend/src/common/conv.js
+++ b/swift_browser_ui_frontend/src/common/conv.js
@@ -286,22 +286,18 @@ export function sortObjects(objects, sortBy, sortDirection) {
     let valueA = a[sortBy];
     let valueB = b[sortBy];
 
-
-
     // Handle tags as single string
     if (Array.isArray(valueA)) {
       valueA = valueA.join(" ");
       valueB = valueB.join(" ");
     }
 
-
     if (typeof valueA === "string") {
       valueA = valueA.toLowerCase();
       valueB = valueB.toLowerCase();
       if (sortDirection === "asc") {
         return valueA < valueB ? -1 : valueA > valueB ? 1 : 0;
       }
-
       return valueB < valueA ? -1 : valueB > valueA ? 1 : 0;
     }
 
@@ -309,7 +305,6 @@ export function sortObjects(objects, sortBy, sortDirection) {
       if (sortDirection === "asc") {
         return valueA - valueB;
       }
-
       return valueB - valueA;
     }
   });