From 8e5d650be4940b3f27baae76e8e1b9d81fbd70ab Mon Sep 17 00:00:00 2001
From: Felipe Trost <felipe.trost@gmail.com>
Date: Fri, 13 Dec 2024 12:49:18 +0100
Subject: [PATCH 1/6] fix(ms2/process-page): throw UnauthorizedError -> correct
 error message

---
 .../(dashboard)/[environmentId]/processes/[processId]/page.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/page.tsx b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/page.tsx
index bfe3d8e02..1ae4b8f4e 100644
--- a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/page.tsx
+++ b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/page.tsx
@@ -6,6 +6,7 @@ import { toCaslResource } from '@/lib/ability/caslAbility';
 import AddUserControls from '@/components/add-user-controls';
 import { getProcess, getProcesses } from '@/lib/data/DTOs';
 import { getProcessBPMN } from '@/lib/data/processes';
+import { UnauthorizedError } from '@/lib/ability/abilityHelper';
 
 type ProcessProps = {
   params: { processId: string; environmentId: string };
@@ -25,7 +26,7 @@ const Process = async ({ params: { processId, environmentId }, searchParams }: P
   const processes = await getProcesses(userId, ability, false);
 
   if (!ability.can('view', toCaslResource('Process', process))) {
-    throw new Error('Forbidden.');
+    throw new UnauthorizedError();
   }
 
   const selectedVersionBpmn = selectedVersionId

From 17993ad4486d5b8f4a5566f113224c4245451317 Mon Sep 17 00:00:00 2001
From: Felipe Trost <felipe.trost@gmail.com>
Date: Sun, 15 Dec 2024 03:41:13 +0100
Subject: [PATCH 2/6] feat(ms2/share-modal): select version of process in share
 modal

---
 ...modeler-share-modal-option-public-link.tsx | 182 ++++++++++--------
 .../[processId]/modeler-share-modal.tsx       |   8 +-
 .../processes/[processId]/modeler-toolbar.tsx |   1 +
 3 files changed, 105 insertions(+), 86 deletions(-)

diff --git a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal-option-public-link.tsx b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal-option-public-link.tsx
index 9d37f3740..8f8e90303 100644
--- a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal-option-public-link.tsx
+++ b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal-option-public-link.tsx
@@ -1,5 +1,5 @@
 'use client';
-import { App, Button, Checkbox, Col, Flex, Input, QRCode, Row } from 'antd';
+import { App, Button, Checkbox, Col, Flex, Input, QRCode, Row, Select, Space } from 'antd';
 import { DownloadOutlined, CopyOutlined } from '@ant-design/icons';
 import { useEffect, useRef, useState } from 'react';
 import { useParams, useSearchParams } from 'next/navigation';
@@ -10,21 +10,24 @@ import {
 import { useEnvironment } from '@/components/auth-can';
 
 import styles from './modeler-share-modal-option-public-link.module.scss';
+import { Process } from '@/lib/data/process-schema';
 
 type ModelerShareModalOptionPublicLinkProps = {
   sharedAs: 'public' | 'protected';
   shareTimestamp: number;
   refresh: () => void;
+  processVersions: Process['versions'];
 };
 
 const ModelerShareModalOptionPublicLink = ({
   sharedAs,
   shareTimestamp,
   refresh,
+  processVersions,
 }: ModelerShareModalOptionPublicLinkProps) => {
   const { processId } = useParams();
   const query = useSearchParams();
-  const selectedVersionId = query.get('version');
+  const [selectedVersionId, setSelectedVersionId] = useState<string | null>(query.get('version'));
   const environment = useEnvironment();
 
   const [shareLink, setShareLink] = useState('');
@@ -185,95 +188,104 @@ const ModelerShareModalOptionPublicLink = ({
   };
 
   return (
-    <>
-      <div style={{ marginBottom: '5px' }}>
+    <Space direction="vertical">
+      <Select
+        defaultValue={selectedVersionId || '-1'}
+        options={[
+          { value: '-1', label: 'latest version' },
+          ...processVersions.map((version) => ({ value: version.id, label: version.name })),
+        ]}
+        onChange={(value) => {
+          setSelectedVersionId(value === '-1' ? null : value);
+        }}
+      />
+
+      <div>
         <Checkbox checked={isShareLinkChecked} onChange={handleShareLinkChecked}>
           Share Process with Public Link
         </Checkbox>
       </div>
-      <div>
-        <Row>
-          <Col span={18} style={{ paddingBottom: '10px', paddingLeft: '25px' }}>
-            <Flex vertical gap="small" justify="left" align="left">
-              <Checkbox
-                checked={registeredUsersonlyChecked}
-                onChange={handlePermissionChanged}
-                disabled={isShareLinkEmpty}
-              >
-                Visible only for registered user
-              </Checkbox>
-            </Flex>
-          </Col>
-          <Col span={18}>
-            <Input
-              type={'text'}
-              value={shareLink}
+      <Row>
+        <Col span={18} style={{ paddingBottom: '10px', paddingLeft: '25px' }}>
+          <Flex vertical gap="small" justify="left" align="left">
+            <Checkbox
+              checked={registeredUsersonlyChecked}
+              onChange={handlePermissionChanged}
               disabled={isShareLinkEmpty}
-              name="generated share link"
-              style={{ border: '1px solid #000' }}
-            />
-          </Col>
-          <Col span={12}>
-            <Flex
-              vertical={false}
-              style={{ paddingTop: '10px', flexWrap: 'wrap-reverse' }}
-              justify="center"
-              align="center"
             >
-              {isShareLinkChecked && (
-                <div id="qrcode" ref={canvasRef}>
-                  <QRCode
-                    style={{
-                      border: '1px solid #000',
-                    }}
-                    value={shareLink}
-                    size={130}
-                  />
-                </div>
-              )}
-            </Flex>
-          </Col>
-          <Col span={6} style={{ paddingTop: '10px' }}>
-            <Flex vertical gap={10}>
-              <Button
-                className={styles.OptionButton}
-                onClick={handleCopyLink}
-                disabled={isShareLinkEmpty}
-              >
-                Copy link
-              </Button>
-              <Button
-                className={styles.OptionButton}
-                onClick={handleOpenSharedPage}
-                disabled={isShareLinkEmpty}
-              >
-                Open
-              </Button>
-              <Button
-                icon={<DownloadOutlined />}
-                title="Save as PNG"
-                className={styles.OptionButton}
-                hidden={isShareLinkEmpty}
-                onClick={() => handleQRCodeAction('download')}
-                disabled={isShareLinkEmpty}
-              >
-                Save QR Code
-              </Button>
-              <Button
-                icon={<CopyOutlined />}
-                title="Copy as PNG"
-                className={styles.OptionButton}
-                hidden={isShareLinkEmpty}
-                onClick={() => handleQRCodeAction('copy')}
-                disabled={isShareLinkEmpty}
-              >
-                Copy QR Code
-              </Button>
-            </Flex>
-          </Col>
-        </Row>
-      </div>
-    </>
+              Visible only for registered user
+            </Checkbox>
+          </Flex>
+        </Col>
+        <Col span={18}>
+          <Input
+            type={'text'}
+            value={shareLink}
+            disabled={isShareLinkEmpty}
+            name="generated share link"
+            style={{ border: '1px solid #000' }}
+          />
+        </Col>
+        <Col span={12}>
+          <Flex
+            vertical={false}
+            style={{ paddingTop: '10px', flexWrap: 'wrap-reverse' }}
+            justify="center"
+            align="center"
+          >
+            {isShareLinkChecked && (
+              <div id="qrcode" ref={canvasRef}>
+                <QRCode
+                  style={{
+                    border: '1px solid #000',
+                  }}
+                  value={shareLink}
+                  size={130}
+                />
+              </div>
+            )}
+          </Flex>
+        </Col>
+        <Col span={6} style={{ paddingTop: '10px' }}>
+          <Flex vertical gap={10}>
+            <Button
+              className={styles.OptionButton}
+              onClick={handleCopyLink}
+              disabled={isShareLinkEmpty}
+            >
+              Copy link
+            </Button>
+            <Button
+              className={styles.OptionButton}
+              onClick={handleOpenSharedPage}
+              disabled={isShareLinkEmpty}
+            >
+              Open
+            </Button>
+            <Button
+              icon={<DownloadOutlined />}
+              title="Save as PNG"
+              className={styles.OptionButton}
+              hidden={isShareLinkEmpty}
+              onClick={() => handleQRCodeAction('download')}
+              disabled={isShareLinkEmpty}
+            >
+              Save QR Code
+            </Button>
+            <Button
+              icon={<CopyOutlined />}
+              title="Copy as PNG"
+              className={styles.OptionButton}
+              hidden={isShareLinkEmpty}
+              onClick={() => handleQRCodeAction('copy')}
+              disabled={isShareLinkEmpty}
+            >
+              Copy QR Code
+            </Button>
+          </Flex>
+        </Col>
+      </Row>
+    </Space>
   );
 };
 
diff --git a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal.tsx b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal.tsx
index daafe5c69..26d405cc9 100644
--- a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal.tsx
+++ b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-share-modal.tsx
@@ -32,10 +32,15 @@ import { set } from 'zod';
 type ShareModalProps = {
   onExport: () => void;
   onExportMobile: (type: ProcessExportOptions['type']) => void;
+  versions: Process['versions'];
 };
 type SharedAsType = 'public' | 'protected';
 
-const ModelerShareModalButton: FC<ShareModalProps> = ({ onExport, onExportMobile }) => {
+const ModelerShareModalButton: FC<ShareModalProps> = ({
+  onExport,
+  onExportMobile,
+  versions: processVersions,
+}) => {
   const { processId } = useParams();
   const environment = useEnvironment();
   const [isOpen, setIsOpen] = useState(false);
@@ -276,6 +281,7 @@ const ModelerShareModalButton: FC<ShareModalProps> = ({ onExport, onExportMobile
           sharedAs={sharedAs as SharedAsType}
           shareTimestamp={shareTimestamp}
           refresh={checkIfProcessShared}
+          processVersions={processVersions}
         />
       ),
     },
diff --git a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-toolbar.tsx b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-toolbar.tsx
index 890de3423..74f66e565 100644
--- a/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-toolbar.tsx
+++ b/src/management-system-v2/app/(dashboard)/[environmentId]/processes/[processId]/modeler-toolbar.tsx
@@ -317,6 +317,7 @@ const ModelerToolbar = ({
               <ModelerShareModalButton
                 onExport={handleProcessExportModalToggle}
                 onExportMobile={handleProcessExportModalToggleMobile}
+                versions={versions}
               />
               <Tooltip title="Open Documentation">
                 <Button icon={<FilePdfOutlined />} onClick={handleOpenDocumentation} />

From 08251163909799848e296db899bf4ad448a32e11 Mon Sep 17 00:00:00 2001
From: Felipe Trost <felipe.trost@gmail.com>
Date: Sun, 15 Dec 2024 12:08:57 +0100
Subject: [PATCH 3/6] fix(ms2/legacy-store): parse dates of processes

---
 .../lib/data/legacy/_process.ts                  | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/management-system-v2/lib/data/legacy/_process.ts b/src/management-system-v2/lib/data/legacy/_process.ts
index ee3fc6d12..e319b1b4b 100644
--- a/src/management-system-v2/lib/data/legacy/_process.ts
+++ b/src/management-system-v2/lib/data/legacy/_process.ts
@@ -57,6 +57,14 @@ export function getProcessMetaObjects() {
   return processMetaObjects;
 }
 
+function parseDates<T extends ProcessMetadata>(process: T) {
+  // The type says these fields are dates, but they're turned into a string when stored
+  process.lastEditedOn = new Date(process.lastEditedOn);
+  process.createdOn = new Date(process.createdOn);
+
+  return process;
+}
+
 /** Returns all processes for a user */
 export async function getProcesses(userId: string, ability: Ability, includeBPMN = false) {
   const processes = Object.values(processMetaObjects);
@@ -69,7 +77,7 @@ export async function getProcesses(userId: string, ability: Ability, includeBPMN
       ),
   );
 
-  return userProcesses;
+  return userProcesses.map(parseDates);
 }
 
 export async function getProcess(processDefinitionsId: string, includeBPMN = false) {
@@ -79,7 +87,7 @@ export async function getProcess(processDefinitionsId: string, includeBPMN = fal
   }
 
   const bpmn = includeBPMN ? await getProcessBpmn(processDefinitionsId) : null;
-  return { ...process, bpmn };
+  return parseDates({ ...process, bpmn });
 }
 
 /**
@@ -213,7 +221,7 @@ export function moveProcess({
 
   if (!dontUpdateOldFolder) {
     const oldFolder = foldersMetaObject.folders[process.folderId];
-    if (!oldFolder) throw new Error("Consistensy Error: Process' folder not found");
+    if (!oldFolder) throw new Error("Consistency Error: Process' folder not found");
     const processOldFolderIdx = oldFolder.children.findIndex(
       (item) => 'type' in item && item.type === 'process' && item.id === processDefinitionsId,
     );
@@ -239,7 +247,7 @@ export async function updateProcessMetaData(
 
   const newMetaData = {
     ...processMetaObjects[processDefinitionsId],
-    lastEdited: new Date().toUTCString(),
+    lastEditedOn: new Date(),
   };
 
   mergeIntoObject(newMetaData, metaChanges, true, true, true);

From 4768497a57f3324a69ecc5c1fca77d9157b951c6 Mon Sep 17 00:00:00 2001
From: Felipe Trost <felipe.trost@gmail.com>
Date: Mon, 16 Dec 2024 03:16:56 +0100
Subject: [PATCH 4/6] fix(ms2/process-documentation): don't mutate state
 variable directly

---
 src/management-system-v2/app/shared-viewer/process-document.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/management-system-v2/app/shared-viewer/process-document.tsx b/src/management-system-v2/app/shared-viewer/process-document.tsx
index babd709f7..25beb58ab 100644
--- a/src/management-system-v2/app/shared-viewer/process-document.tsx
+++ b/src/management-system-v2/app/shared-viewer/process-document.tsx
@@ -230,7 +230,7 @@ const ProcessDocument: React.FC<ProcessDocumentProps> = ({
 
   // transform the document data into the respective pages of the document
   useEffect(() => {
-    processHierarchy && getContent(processHierarchy, processPages);
+    processHierarchy && getContent(processHierarchy, [...processPages]);
   }, [processHierarchy]);
 
   return (

From bdb98b704fbb3c4f7fdba9018edefba3651b7963 Mon Sep 17 00:00:00 2001
From: Felipe Trost <felipe.trost@gmail.com>
Date: Mon, 16 Dec 2024 03:18:14 +0100
Subject: [PATCH 5/6] quickfix(ms2/documentation-page): creatorId not defined
 on legacy store

---
 src/management-system-v2/app/shared-viewer/process-document.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/management-system-v2/app/shared-viewer/process-document.tsx b/src/management-system-v2/app/shared-viewer/process-document.tsx
index 25beb58ab..ef0f00e95 100644
--- a/src/management-system-v2/app/shared-viewer/process-document.tsx
+++ b/src/management-system-v2/app/shared-viewer/process-document.tsx
@@ -256,7 +256,7 @@ const ProcessDocument: React.FC<ProcessDocumentProps> = ({
               <div className={cn(styles.Title, { [styles.TitlePage]: settings.titlepage })}>
                 <Title>{processData.name}</Title>
                 <div className={styles.TitleInfos}>
-                  <div>Owner: {processData.creatorId.split('|').pop()}</div>
+                  <div>Owner: {processData.creatorId?.split('|').pop()}</div>
                   {version.id ? (
                     <>
                       <div>Version: {version.name || version.id}</div>

From 32dbf81df173f10496a2e580e655200485187467 Mon Sep 17 00:00:00 2001
From: Felipe Trost <felipe.trost@gmail.com>
Date: Mon, 16 Dec 2024 03:58:17 +0100
Subject: [PATCH 6/6] fix(ms2/process-document): if processHierarchy changes
 pages should start as an empty array

---
 src/management-system-v2/app/shared-viewer/process-document.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/management-system-v2/app/shared-viewer/process-document.tsx b/src/management-system-v2/app/shared-viewer/process-document.tsx
index ef0f00e95..ee795a03d 100644
--- a/src/management-system-v2/app/shared-viewer/process-document.tsx
+++ b/src/management-system-v2/app/shared-viewer/process-document.tsx
@@ -230,7 +230,7 @@ const ProcessDocument: React.FC<ProcessDocumentProps> = ({
 
   // transform the document data into the respective pages of the document
   useEffect(() => {
-    processHierarchy && getContent(processHierarchy, [...processPages]);
+    processHierarchy && getContent(processHierarchy, []);
   }, [processHierarchy]);
 
   return (