[MNT-23143] Add withCredentials to isLoggedIn

Alfresco · Nov 24, 2022 · 90ab51c · 90ab51c
1 parent 80106a4
commit 90ab51c
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/src/alfrescoApi.ts b/src/alfrescoApi.ts
@@ -452,9 +452,9 @@ export class AlfrescoApi implements Emitter, AlfrescoApiType {
             if (this.isBpmConfiguration()) {
                 return this.processAuth.isLoggedIn();
             } else if (this.isEcmConfiguration()) {
-                return this.contentAuth.isLoggedIn();
+                return this.config.withCredentials ? true : this.contentAuth.isLoggedIn();
             } else if (this.isEcmBpmConfiguration()) {
-                return this.contentAuth.isLoggedIn() && this.processAuth.isLoggedIn();
+                return this.config.withCredentials ? true : (this.contentAuth.isLoggedIn() && this.processAuth.isLoggedIn());
             } else {
                 return false;
             }

diff --git a/test/alfrescoApi.spec.ts b/test/alfrescoApi.spec.ts
@@ -107,6 +107,28 @@ describe('Basic configuration test', () => {
             expect(alfrescoJsApi.isEcmLoggedIn()).equal(true);
         });
 
+        it('should withCredentials true parameter with hostEcm should be reflected in isLoggedIn', () => {
+            const hostEcm = 'http://127.0.0.1:8080';
+            const alfrescoJsApi = new AlfrescoApi({
+                hostEcm,
+                provider: 'ECM',
+                withCredentials: true
+            } as AlfrescoApiConfig);
+
+            expect(alfrescoJsApi.isLoggedIn()).equal(true);
+        });
+
+        it('should withCredentials true parameter with ALL provider should be reflected in isLoggedIn', () => {
+            const hostEcm = 'http://127.0.0.1:8080';
+            const alfrescoJsApi = new AlfrescoApi({
+                hostEcm,
+                provider: 'ALL',
+                withCredentials: true
+            } as AlfrescoApiConfig);
+
+            expect(alfrescoJsApi.isLoggedIn()).equal(true);
+        });
+
         it('should withCredentials false parameter should be reflected in the clients', () => {
             const config = {
                 hostEcm: 'http://testServer.com:1616',