fix(edit-content) fix minors bugs Edit Content

core-web/libs/edit-content/src/lib/components/dot-edit-content-sidebar/components/dot-edit-content-sidebar-information/dot-edit-content-sidebar-information.component.html

@@ -31,12 +31,8 @@
                 <span class="sidebar-card__title">{{ 'Created' | dm }}</span>
                 <span
                     class="sidebar-card__subtitle"
-                    [tooltipPosition]="'left'"
-                    [pTooltip]="
-                        !contentlet?.creationDate
-                            ? ('edit.content.sidebar.information.no.created.yet' | dm)
-                            : null
-                    ">
+                    [pTooltip]="contentlet?.ownerName"
+                    tooltipPosition="left">
                     {{
                         contentlet?.creationDate
                             ? (contentlet?.ownerName | dotNameFormat) ||
@@ -59,11 +55,7 @@
                 <span
                     class="sidebar-card__subtitle"
                     [tooltipPosition]="'left'"
-                    [pTooltip]="
-                        !contentlet?.modDate
-                            ? ('edit.content.sidebar.information.moddate.tooltip' | dm)
-                            : null
-                    ">
+                    [pTooltip]="contentlet?.modUserName">
                     {{ contentlet?.modDate ? (contentlet?.modUserName | dotNameFormat) : '-' }}
                 </span>
                 @if (contentlet?.modDate) {
@@ -81,11 +73,7 @@
                 <span
                     class="sidebar-card__subtitle"
                     [tooltipPosition]="'left'"
-                    [pTooltip]="
-                        !contentlet?.publishDate
-                            ? ('edit.content.sidebar.information.publishdate.tooltip' | dm)
-                            : null
-                    ">
+                    [pTooltip]="contentlet?.publishUserName">
                     {{
                         contentlet?.publishDate
                             ? (contentlet?.publishUserName | dotNameFormat)

core-web/libs/edit-content/src/lib/components/dot-edit-content-sidebar/dot-edit-content-sidebar.component.html

@@ -55,6 +55,17 @@
                     data-testId="workflow" />
             </dot-edit-content-sidebar-section>
         </p-tabPanel>
+
+        <p-tabPanel [header]="'History' | dm">
+            <ng-template pTemplate="content">
+                <span class="temporal-content">{{ 'Coming soon' }}</span>
+            </ng-template>
+        </p-tabPanel>
+        <p-tabPanel [header]="'Settings' | dm">
+            <ng-template pTemplate="content">
+                <span class="temporal-content">{{ 'Coming soon' }}</span>
+            </ng-template>
+        </p-tabPanel>
     </p-tabView>
 </aside>
 

core-web/libs/edit-content/src/lib/components/dot-edit-content-sidebar/dot-edit-content-sidebar.component.scss

@@ -62,6 +62,13 @@
     }
 }
 
+.temporal-content {
+    padding-top: $spacing-3;
+    padding-left: $spacing-3;
+    display: flex;
+    width: 100%;
+}
+
 .content-sidebar {
     overflow: auto;
     width: 100%;

core-web/libs/edit-content/src/lib/pipes/contentlet-status.pipe.spec.ts

@@ -45,13 +45,13 @@ describe('ContentletStatusPipe', () => {
         expect(result.classes).toBe('p-chip-gray');
     });
 
-    it('should transform contentlet status to "Changed"', () => {
+    it('should transform contentlet status to "Published"', () => {
         const contentlet = { live: true, working: true, archived: false } as DotCMSContentlet;
 
         const result = pipe.transform(contentlet);
 
-        expect(result.label).toBe('Changed');
-        expect(result.classes).toBe('p-chip-pink');
+        expect(result.label).toBe('Published');
+        expect(result.classes).toBe('p-chip-success');
     });
 
     it('should transform contentlet status to empty label and default classes', () => {
@@ -62,4 +62,11 @@ describe('ContentletStatusPipe', () => {
         expect(result.label).toBe('');
         expect(result.classes).toBe('');
     });
+
+    it('should transform undefined contentlet to "New" status', () => {
+        const result = pipe.transform(undefined);
+
+        expect(result.label).toBe('New');
+        expect(result.classes).toBe('p-chip-blue');
+    });
 });

core-web/libs/edit-content/src/lib/pipes/contentlet-status.pipe.ts

@@ -57,16 +57,24 @@ export class ContentletStatusPipe implements PipeTransform {
 
     private getContentletStatus(contentlet: DotCMSContentlet): DotEditContentStatus {
         // Archived: Content has been archived
-        if (contentlet.archived) return DotEditContentStatus.ARCHIVED;
-
-        // Changed: Has both live and working versions (unpublished changes exist)
-        if (contentlet.live && contentlet.working) return DotEditContentStatus.CHANGED;
+        if (contentlet.archived) {
+            return DotEditContentStatus.ARCHIVED;
+        }
 
-        // Published: Has live version but no working version (no unpublished changes)
-        if (contentlet.live && !contentlet.working) return DotEditContentStatus.PUBLISHED;
+        // Live content handling
+        if (contentlet.live) {
+            // Compare working and live inodes to determine if content has changed
+            if (contentlet.workingInode === contentlet.liveInode) {
+                return DotEditContentStatus.PUBLISHED;
+            } else {
+                return DotEditContentStatus.CHANGED;
+            }
+        }
 
-        // Draft: Has working version but no live version (never published)
-        if (contentlet.working && !contentlet.live) return DotEditContentStatus.DRAFT;
+        // Draft: Not live and has working version
+        if (contentlet.working) {
+            return DotEditContentStatus.DRAFT;
+        }
 
         // Unknown: None of the above conditions match
         return DotEditContentStatus.UNKNOWN;

core-web/libs/edit-content/src/lib/store/features/content.feature.ts

@@ -285,17 +285,14 @@ export function withContent() {
                                         const initialContentletState =
                                             !scheme || !step ? 'reset' : 'existing';
 
-                                        // The current step is the first step of the selected scheme
-                                        const currentScheme = parsedSchemes[currentSchemeId];
-
                                         patchState(store, {
                                             contentType,
                                             currentSchemeId,
                                             schemes: parsedSchemes,
                                             currentContentActions: parsedCurrentActions,
                                             contentlet,
                                             state: ComponentStatus.LOADED,
-                                            currentStep: currentScheme?.firstStep,
+                                            currentStep: step,
                                             lastTask: task,
                                             initialContentletState
                                         });

core-web/libs/edit-content/src/lib/store/features/workflow.feature.spec.ts

@@ -13,7 +13,8 @@ import {
     DotHttpErrorManagerService,
     DotMessageService,
     DotWorkflowActionsFireService,
-    DotWorkflowsActionsService
+    DotWorkflowsActionsService,
+    DotWorkflowService
 } from '@dotcms/data-access';
 import { ComponentStatus, DotCMSContentlet } from '@dotcms/dotcms-models';
 
@@ -23,7 +24,8 @@ import { withWorkflow } from './workflow.feature';
 import {
     MOCK_CONTENTLET_1_TAB,
     MOCK_WORKFLOW_ACTIONS_NEW_ITEMNTTYPE_1_TAB,
-    MOCK_WORKFLOW_DATA
+    MOCK_WORKFLOW_DATA,
+    MOCK_WORKFLOW_STATUS
 } from '../../utils/edit-content.mock';
 import { CONTENT_TYPE_MOCK } from '../../utils/mocks';
 import { parseCurrentActions, parseWorkflows } from '../../utils/workflows.utils';
@@ -44,7 +46,7 @@ describe('WorkflowFeature', () => {
     let router: SpyObject<Router>;
     let messageService: SpyObject<MessageService>;
     let dotMessageService: SpyObject<DotMessageService>;
-
+    let dotWorkflowService: SpyObject<DotWorkflowService>;
     const createStore = createServiceFactory({
         service: signalStore(
             withState({ ...initialRootState, ...mockInitialStateWithContent }),
@@ -56,7 +58,8 @@ describe('WorkflowFeature', () => {
             DotHttpErrorManagerService,
             DotMessageService,
             MessageService,
-            Router
+            Router,
+            DotWorkflowService
         ]
     });
 
@@ -68,7 +71,7 @@ describe('WorkflowFeature', () => {
         router = spectator.inject(Router);
         messageService = spectator.inject(MessageService);
         dotMessageService = spectator.inject(DotMessageService);
-
+        dotWorkflowService = spectator.inject(DotWorkflowService);
         dotMessageService.get.mockReturnValue('Success Message');
     });
 
@@ -86,6 +89,7 @@ describe('WorkflowFeature', () => {
                 workflowActionService.getByInode.mockReturnValue(
                     of(MOCK_WORKFLOW_ACTIONS_NEW_ITEMNTTYPE_1_TAB)
                 );
+                dotWorkflowService.getWorkflowStatus.mockReturnValue(of(MOCK_WORKFLOW_STATUS));
 
                 store.fireWorkflowAction(mockOptions);
                 tick();
@@ -156,7 +160,10 @@ describe('WorkflowFeature', () => {
                 expect(store.getCurrentStep()).toBeNull();
                 expect(messageService.add).toHaveBeenCalledWith(
                     expect.objectContaining({
-                        severity: 'success'
+                        detail: 'Success Message',
+                        icon: 'pi pi-spin pi-spinner',
+                        severity: 'info',
+                        summary: 'Success Message'
                     })
                 );
             }));
@@ -165,6 +172,7 @@ describe('WorkflowFeature', () => {
                 workflowActionsFireService.fireTo.mockReturnValue(of(MOCK_CONTENTLET_1_TAB));
 
                 store.fireWorkflowAction(mockOptions);
+                tick();
 
                 expect(messageService.add).toHaveBeenCalledWith(
                     expect.objectContaining({

core-web/libs/edit-content/src/lib/store/features/workflow.feature.ts

@@ -24,7 +24,8 @@ import {
     DotMessageService,
     DotRenderMode,
     DotWorkflowActionsFireService,
-    DotWorkflowsActionsService
+    DotWorkflowsActionsService,
+    DotWorkflowService
 } from '@dotcms/data-access';
 import {
     ComponentStatus,
@@ -200,6 +201,7 @@ export function withWorkflow() {
                 dotHttpErrorManagerService = inject(DotHttpErrorManagerService),
                 messageService = inject(MessageService),
                 dotMessageService = inject(DotMessageService),
+                dotWorkflowService = inject(DotWorkflowService),
                 router = inject(Router)
             ) => ({
                 /**
@@ -282,11 +284,18 @@ export function withWorkflow() {
                                             DotRenderMode.EDITING
                                         ),
                                         contentlet: of(contentlet),
-                                        isReset: of(isReset)
+                                        isReset: of(isReset),
+                                        // Workflow status for this inode
+                                        workflowStatus: dotWorkflowService.getWorkflowStatus(inode)
                                     });
                                 }),
                                 tapResponse({
-                                    next: ({ contentlet, currentContentActions, isReset }) => {
+                                    next: ({
+                                        contentlet,
+                                        currentContentActions,
+                                        isReset,
+                                        workflowStatus
+                                    }) => {
                                         // Always navigate if the inode has changed
                                         if (contentlet.inode !== currentContentlet?.inode) {
                                             router.navigate(['/content', contentlet.inode], {
@@ -298,6 +307,8 @@ export function withWorkflow() {
                                         const parsedCurrentActions =
                                             parseCurrentActions(currentContentActions);
 
+                                        const { step } = workflowStatus;
+
                                         if (isReset) {
                                             patchState(store, {
                                                 contentlet,
@@ -317,6 +328,7 @@ export function withWorkflow() {
                                                 currentContentActions: parsedCurrentActions,
                                                 currentSchemeId: store.currentSchemeId(),
                                                 state: ComponentStatus.LOADED,
+                                                currentStep: step,
                                                 error: null
                                             });
                                         }