fix: Incorrect owner shown for bookmarks, measures & dimensions

Fixes #121
ptarmiganlabs · Jan 27, 2023 · b571619 · b571619
1 parent af88014
commit b571619
Show file tree

Hide file tree

Showing 5 changed files with 50 additions and 51 deletions.
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -53,37 +53,37 @@
       // ------------------------------------
       // Import from Excel file
       // ------------------------------------
-      "args": [
-        "task-import",
-        "--auth-type",
-        "cert",
-        "--host",
-        "192.168.100.109",
-        "--auth-cert-file",
-        "./cert/client.pem",
-        "--auth-cert-key-file",
-        "./cert/client_key.pem",
-        "--auth-user-dir",
-        "LAB",
-        "--auth-user-id",
-        "goran",
+      // "args": [
+      //   "task-import",
+      //   "--auth-type",
+      //   "cert",
+      //   "--host",
+      //   "192.168.100.109",
+      //   "--auth-cert-file",
+      //   "./cert/client.pem",
+      //   "--auth-cert-key-file",
+      //   "./cert/client_key.pem",
+      //   "--auth-user-dir",
+      //   "LAB",
+      //   "--auth-user-id",
+      //   "goran",
 
-        "--file-type",
-        "excel",
+      //   "--file-type",
+      //   "excel",
 
-        "--file-name",
-        "tasks3.xlsx",
-        "--sheet-name",
-        "Ctrl-Q task export",
+      //   "--file-name",
+      //   "tasks3.xlsx",
+      //   "--sheet-name",
+      //   "Ctrl-Q task export",
 
-        // "--overwrite",
-        // "no",
+      //   // "--overwrite",
+      //   // "no",
 
-        // "--limit-import-count",
-        // "2",
+      //   // "--limit-import-count",
+      //   // "2",
 
-        "--dry-run"
-      ]
+      //   "--dry-run"
+      // ]
 
 
       // ------------------------------------
@@ -341,27 +341,27 @@
       //   "Dim3"
       // ]
 
-      //   Get all bookmark by name
-    //   "args": [
-    //     "bookmark-get",
-    //     "--auth-type",
-    //     "cert",
-    //     "--host",
-    //     "192.168.100.109",
-    //     "--app-id",
-    //     "2933711d-6638-41d4-a2d2-6dd2d965208b",
-    //     "--auth-user-dir",
-    //     "LAB",
-    //     "--auth-user-id",
-    //     "goran",
-    //     "--id-type",
-    //     "id",
-    //     // "--bookmark",
-    //     // "Bookmark 1",
-    //     // "Bookmark 3",
-    //     "--output-format",
-    //     "table"
-    // ]
+      //   Get all bookmarks
+      "args": [
+        "bookmark-get",
+        "--auth-type",
+        "cert",
+        "--host",
+        "192.168.100.109",
+        "--app-id",
+        "2933711d-6638-41d4-a2d2-6dd2d965208b",
+        "--auth-user-dir",
+        "LAB",
+        "--auth-user-id",
+        "goran",
+        "--id-type",
+        "id",
+        // "--bookmark",
+        // "Bookmark 1",
+        // "Bookmark 3",
+        "--output-format",
+        "table"
+    ]
 
       //   Get measure by id
       // "args": [

diff --git a/src/lib/cmd/getbookmark.js b/src/lib/cmd/getbookmark.js
@@ -160,7 +160,7 @@ const getBookmark = async (options) => {
                     bookmark.qMeta.publishTime,
                     bookmark.qMeta.createdDate,
                     bookmark.qMeta.modifiedDate,
-                    `${bookmark.qMeta.owner.authUserDirectory}\\${bookmark.qMeta.owner.authUserId}`,
+                    `${bookmark.qMeta.owner.userDirectory}\\${bookmark.qMeta.owner.userId}`,
                 ]);
             }
 

diff --git a/src/lib/cmd/getdim.js b/src/lib/cmd/getdim.js
@@ -187,7 +187,7 @@ const getMasterDimension = async (options) => {
                     dimension.qMeta.publishTime,
                     dimension.qMeta.createdDate,
                     dimension.qMeta.modifiedDate,
-                    `${dimension.qMeta.owner.authUserDirectory}\\${dimension.qMeta.owner.authUserId}`,
+                    `${dimension.qMeta.owner.userDirectory}\\${dimension.qMeta.owner.userId}`,
                     dimension.qMeta.tags !== undefined ? dimension.qMeta.tags : '',
                 ]);
             }

diff --git a/src/lib/cmd/getmeasure.js b/src/lib/cmd/getmeasure.js
@@ -175,7 +175,7 @@ const getMasterMeasure = async (options) => {
                     measure.qMeta.publishTime,
                     measure.qMeta.createdDate,
                     measure.qMeta.modifiedDate,
-                    `${measure.qMeta.owner.authUserDirectory}\\${measure.qMeta.owner.authUserId}`,
+                    `${measure.qMeta.owner.userDirectory}\\${measure.qMeta.owner.userId}`,
                     measure.qMeta.tags,
                 ]);
             }

diff --git a/src/lib/task/class_alltasks.js b/src/lib/task/class_alltasks.js
@@ -184,7 +184,6 @@ class QlikSenseTasks {
                     );
 
                     // Create a fake ID for this task. Used to associate task with schema/composite events
-                    // const fakeTaskId = `task-${nanoid.nanoid()}`;
                     const fakeTaskId = `reload-task-${uuidv4()}`;
 
                     let currentTask = null;