Port X-02f to dtq-dev-9-base: item-page URI DOI resolver and the sidebar expand-all truncatable hooks

Source: no single commit - the fork delta is measured against dspace-7.6.5, and the v9
upgrade took all of these files wholesale from vanilla 9.3, so the fork hunks were never
applied (guard X1 sweep 2026-09-10, card X-02f).

(A) Sidebar expand-all
- truncatable-part: @input() externalToggle, @output() truncated, toggle(event?, expand?)
  with stopPropagation and a toggleWithoutId() fallback for parts with no id, isExpanded
  reduced to `expand` (the `expandable` flip in toggle() goes with it).
- The v9 body of truncateElement() is kept (scrollHeight > clientHeight); only its guard is
  widened, and `truncated` is emitted through a change-only gate because truncateElement()
  runs on every ngAfterViewChecked.
- sidebar-search-list-element: the expandable/expanded/truncatedStates controller and the
  .toggleIcons chevrons, on top of FE-24's descriptionLang hunks.
- listable-object-component-loader.component.scss positions the chevrons.

(B) item-page-uri-field resolves a bare DOI through ItemIdentifierService
  (loadDoiResolverConfiguration + DEFAULT_DOI_RESOLVER), not a second ConfigurationDataService.
  The prefix is applied ONLY to dc.identifier.doi: on this base the component also renders
  dc.identifier.uri, coar.notify.endorsedBy and three datacite.relation.* fields, and the
  fork's blanket "no http means DOI" rule would corrupt those.

(C) auth-nav-menu.component.scss gains `.dropdown-toggle` WITH the leading dot - the fork's
  selector has no dot, so it targets a <dropdown-toggle> element that does not exist.

Not ported, deliberately: getParentHierarchyTitle/getAllParentsRecursive (a separate unowned
feature), and the fork's three vacuous truncatable tests (.expandIcon / .collapseIcon, whose
selectors exist nowhere in the tree, and a tautological getter assertion).

Closes card X-02f.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: milanmajchrak

src/app/item-page/simple/field-components/specific-field/uri/item-page-uri-field.component.html

@@ -1,3 +1,3 @@
 <div class="item-page-field">
-  <ds-metadata-uri-values [mdValues]="item?.allMetadata(fields)" [separator]="separator" [label]="label"></ds-metadata-uri-values>
+  <ds-metadata-uri-values [mdValues]="getUriMetadataValues()" [separator]="separator" [label]="label"></ds-metadata-uri-values>
 </div>

src/app/item-page/simple/field-components/specific-field/uri/item-page-uri-field.component.spec.ts

@@ -7,15 +7,18 @@ import {
   TestBed,
   waitForAsync,
 } from '@angular/core/testing';
+import { By } from '@angular/platform-browser';
 import {
   TranslateLoader,
   TranslateModule,
 } from '@ngx-translate/core';
+import { of } from 'rxjs';
 
 import { APP_CONFIG } from '../../../../../../config/app-config.interface';
 import { environment } from '../../../../../../environments/environment';
 import { BrowseService } from '../../../../../core/browse/browse.service';
 import { BrowseDefinitionDataService } from '../../../../../core/browse/browse-definition-data.service';
+import { ItemIdentifierService } from '../../../../../shared/item-identifier.service';
 import { BrowseDefinitionDataServiceStub } from '../../../../../shared/testing/browse-definition-data-service.stub';
 import { BrowseServiceStub } from '../../../../../shared/testing/browse-service.stub';
 import { TranslateLoaderMock } from '../../../../../shared/testing/translate-loader.mock';
@@ -30,8 +33,19 @@ const mockField = 'dc.identifier.uri';
 const mockValue = 'test value';
 const mockLabel = 'test label';
 
+const DOI_FIELD = 'dc.identifier.doi';
+const NON_DOI_FIELD = 'coar.notify.endorsedBy';
+const RESOLVER = 'https://doi.test';
+
+let itemIdentifierService: jasmine.SpyObj<ItemIdentifierService>;
+
+const hrefOfFirstAnchor = (f: ComponentFixture<ItemPageUriFieldComponent>): string =>
+  f.debugElement.query(By.css('a')).nativeElement.getAttribute('href');
+
 describe('ItemPageUriFieldComponent', () => {
   beforeEach(waitForAsync(() => {
+    itemIdentifierService = jasmine.createSpyObj('itemIdentifierService', ['loadDoiResolverConfiguration']);
+    itemIdentifierService.loadDoiResolverConfiguration.and.returnValue(of(RESOLVER));
     TestBed.configureTestingModule({
       imports: [TranslateModule.forRoot({
         loader: {
@@ -43,6 +57,7 @@ describe('ItemPageUriFieldComponent', () => {
         { provide: APP_CONFIG, useValue: environment },
         { provide: BrowseDefinitionDataService, useValue: BrowseDefinitionDataServiceStub },
         { provide: BrowseService, useValue: BrowseServiceStub },
+        { provide: ItemIdentifierService, useValue: itemIdentifierService },
       ],
       schemas: [NO_ERRORS_SCHEMA],
     }).overrideComponent(ItemPageUriFieldComponent, {
@@ -62,4 +77,41 @@ describe('ItemPageUriFieldComponent', () => {
   it('should display display the correct metadata value', () => {
     expect(fixture.nativeElement.innerHTML).toContain(mockValue);
   });
+
+  describe('DOI resolver', () => {
+    const render = (field: string, value: string) => {
+      fixture = TestBed.createComponent(ItemPageUriFieldComponent);
+      comp = fixture.componentInstance;
+      comp.item = mockItemWithMetadataFieldsAndValue([field], value);
+      comp.fields = [field];
+      comp.label = mockLabel;
+      fixture.detectChanges();
+      return fixture;
+    };
+
+    it('should prefix a bare dc.identifier.doi with the configured resolver', () => {
+      const f = render(DOI_FIELD, '10.1234/abc');
+
+      expect(hrefOfFirstAnchor(f)).toEqual(RESOLVER + '/' + '10.1234/abc');
+    });
+
+    it('should leave a dc.identifier.doi that already has a scheme alone', () => {
+      const f = render(DOI_FIELD, 'https://doi.test/10.1234/abc');
+
+      expect(hrefOfFirstAnchor(f)).toEqual('https://doi.test/10.1234/abc');
+    });
+
+    it('should leave a bare coar.notify.endorsedBy value untouched', () => {
+      const f = render(NON_DOI_FIELD, '10.1234/abc');
+
+      expect(hrefOfFirstAnchor(f)).toEqual('10.1234/abc');
+    });
+
+    it('should not produce a double slash when the configured resolver ends with one', () => {
+      itemIdentifierService.loadDoiResolverConfiguration.and.returnValue(of('https://doi.test/'));
+      const f = render(DOI_FIELD, '10.1234/abc');
+
+      expect(hrefOfFirstAnchor(f)).toEqual('https://doi.test/10.1234/abc');
+    });
+  });
 });

src/app/item-page/simple/field-components/specific-field/uri/item-page-uri-field.component.ts

@@ -1,10 +1,20 @@
 import {
   Component,
   Input,
+  OnInit,
 } from '@angular/core';
 
+import { BrowseService } from '../../../../../core/browse/browse.service';
+import { BrowseDefinitionDataService } from '../../../../../core/browse/browse-definition-data.service';
 import { Item } from '../../../../../core/shared/item.model';
+import { MetadataValue } from '../../../../../core/shared/metadata.models';
+import { isEmpty } from '../../../../../shared/empty.util';
+import {
+  DEFAULT_DOI_RESOLVER,
+  ItemIdentifierService,
+} from '../../../../../shared/item-identifier.service';
 import { MetadataUriValuesComponent } from '../../../../field-components/metadata-uri-values/metadata-uri-values.component';
+import { DOI_METADATA_FIELD } from '../../clarin-generic-item-field/clarin-generic-item-field.constants';
 import { ItemPageFieldComponent } from '../item-page-field.component';
 
 @Component({
@@ -18,7 +28,18 @@ import { ItemPageFieldComponent } from '../item-page-field.component';
  * This component can be used to represent any uri on a simple item page.
  * It expects 4 parameters: The item, a separator, the metadata keys and an i18n key
  */
-export class ItemPageUriFieldComponent extends ItemPageFieldComponent {
+export class ItemPageUriFieldComponent extends ItemPageFieldComponent implements OnInit {
+
+  /**
+   * The configured DOI resolver, used to turn a bare DOI into a resolvable link.
+   */
+  doiResolver: string;
+
+  constructor(protected browseDefinitionDataService: BrowseDefinitionDataService,
+              protected browseService: BrowseService,
+              protected itemIdentifierService: ItemIdentifierService) {
+    super(browseDefinitionDataService, browseService);
+  }
 
   /**
    * The item to display metadata for
@@ -41,4 +62,40 @@ export class ItemPageUriFieldComponent extends ItemPageFieldComponent {
    */
   @Input() label: string;
 
+  ngOnInit(): void {
+    this.itemIdentifierService.loadDoiResolverConfiguration().subscribe((resolver: string) => {
+      this.doiResolver = isEmpty(resolver) ? DEFAULT_DOI_RESOLVER : resolver;
+    });
+  }
+
+  /**
+   * The metadata values to render.
+   *
+   * A bare DOI (no scheme) is not resolvable on its own, so it is prefixed with the configured
+   * `identifier.doi.resolver`. Only `dc.identifier.doi` is treated that way: this component also
+   * renders `dc.identifier.uri`, `coar.notify.endorsedBy` and three `datacite.relation.*` fields,
+   * and prefixing a bare value of any of those with a DOI resolver would corrupt the link.
+   */
+  getUriMetadataValues(): MetadataValue[] {
+    const mdValues: MetadataValue[] = this.item?.allMetadata(this.fields);
+
+    if (isEmpty(mdValues) || !this.fields?.includes(DOI_METADATA_FIELD)) {
+      return mdValues;
+    }
+
+    // allMetadata() returns readonly values, so build new ones rather than mutating them.
+    return mdValues.map((mdValue: MetadataValue) =>
+      Object.assign(new MetadataValue(), mdValue, { value: this.resolveDoi(mdValue.value) }));
+  }
+
+  /**
+   * Prefix a bare DOI with the resolver. A value that already carries a scheme is left alone.
+   */
+  private resolveDoi(value: string): string {
+    if (isEmpty(value) || /^https?:\/\//.test(value) || isEmpty(this.doiResolver)) {
+      return value;
+    }
+    return `${this.doiResolver.replace(/\/+$/, '')}/${value}`;
+  }
+
 }

src/app/shared/auth-nav-menu/auth-nav-menu.component.scss

@@ -12,7 +12,7 @@
   background-color: transparent !important;
 }
 
-.loginLink, .dropdownLogin, .logoutLink, .dropdownLogout {
+.loginLink, .dropdownLogin, .logoutLink, .dropdownLogout, .dropdown-toggle {
   color: var(--ds-header-icon-color);
 
   &:hover, &:focus {

src/app/shared/object-collection/shared/listable-object/listable-object-component-loader.component.scss

@@ -1,3 +1,10 @@
 :host {
   width: 100%;
 }
+
+::ng-deep .toggleIcons {
+  position: absolute;
+  top: 5px;
+  right: 5px;
+  cursor: pointer;
+}

src/app/shared/object-list/sidebar-search-list-element/sidebar-search-list-element.component.html

@@ -1,17 +1,33 @@
-<ds-truncatable-part [maxLines]="1" [background]="isCurrent() ? 'primary' : 'default'" [showToggle]="false">
+<ds-truncatable-part [externalToggle]="true" (truncated)="onTruncatedStateChange(1, $event)" [maxLines]="1" [background]="isCurrent() ? 'primary' : 'default'" [showToggle]="false">
   <div [ngClass]="isCurrent() ? 'text-light' : 'text-body'"
   [innerHTML]="(parentTitle$ && parentTitle$ | async) ? (parentTitle$ | async) : ('home.breadcrumbs' | translate)"></div>
 </ds-truncatable-part>
-<ds-truncatable-part [maxLines]="1" [background]="isCurrent() ? 'primary' : 'default'" [showToggle]="false">
+<ds-truncatable-part [externalToggle]="true" (truncated)="onTruncatedStateChange(2, $event)" [maxLines]="1" [background]="isCurrent() ? 'primary' : 'default'" [showToggle]="false">
   <div class="fw-bold"
     [ngClass]="isCurrent() ? 'text-light' : 'text-primary'"
   [innerHTML]="dsoTitle"></div>
 </ds-truncatable-part>
 @if (description) {
-  <ds-truncatable-part [maxLines]="1" [background]="isCurrent() ? 'primary' : 'default'" [showToggle]="false">
+  <ds-truncatable-part [externalToggle]="true" (truncated)="onTruncatedStateChange(3, $event)" [maxLines]="1" [background]="isCurrent() ? 'primary' : 'default'" [showToggle]="false">
     <div class="text-secondary"
       [ngClass]="isCurrent() ? 'text-light' : 'text-secondary'"
       [attr.lang]="descriptionLang || null"
     [innerHTML]="description"></div>
   </ds-truncatable-part>
 }
+@if (expandable) {
+  <div class="toggleIcons">
+    @if (!expanded) {
+      <i (click)="toggleView($event, true)"
+        class="fa fa-angle-down p-1"
+        aria-hidden="true"
+        [title]="'sidebar.expand.all' | translate"></i>
+    }
+    @if (expanded) {
+      <i (click)="toggleView($event, false)"
+        class="fa fa-angle-up p-1"
+        aria-hidden="true"
+        [title]="'sidebar.collapse.all' | translate"></i>
+    }
+  </div>
+}

src/app/shared/object-list/sidebar-search-list-element/sidebar-search-list-element.component.spec.ts

@@ -1,7 +1,12 @@
-import { NO_ERRORS_SCHEMA } from '@angular/core';
+import {
+  NO_ERRORS_SCHEMA,
+  QueryList,
+} from '@angular/core';
 import {
   ComponentFixture,
+  fakeAsync,
   TestBed,
+  tick,
   waitForAsync,
 } from '@angular/core/testing';
 import { RouterTestingModule } from '@angular/router/testing';
@@ -16,6 +21,7 @@ import { mockTruncatableService } from '../../mocks/mock-trucatable.service';
 import { createSuccessfulRemoteDataObject$ } from '../../remote-data.utils';
 import { SearchResult } from '../../search/models/search-result.model';
 import { TruncatableService } from '../../truncatable/truncatable.service';
+import { TruncatablePartComponent } from '../../truncatable/truncatable-part/truncatable-part.component';
 import { VarDirective } from '../../utils/var.directive';
 
 export function createSidebarSearchListElementTests(
@@ -73,5 +79,42 @@ export function createSidebarSearchListElementTests(
     it('should contain the correct description', () => {
       expect(component.description).toEqual(expectedDescription);
     });
+
+    it('should expand every truncatable part at once and not open the item itself', () => {
+      const parts = [1, 2, 3].map(() => new TruncatablePartComponent(mockTruncatableService as any));
+      const queryList = new QueryList<TruncatablePartComponent>();
+      queryList.reset(parts);
+      component.truncatableComponents = queryList;
+      const event = jasmine.createSpyObj('event', ['stopPropagation']);
+
+      component.toggleView(event, true);
+
+      expect(event.stopPropagation).toHaveBeenCalled();
+      expect(component.expanded).toBeTrue();
+      parts.forEach((part) => expect(part.lines).toEqual('none'));
+    });
+
+    it('should collapse every truncatable part again', () => {
+      const parts = [1, 2, 3].map(() => new TruncatablePartComponent(mockTruncatableService as any));
+      parts.forEach((part) => part.toggleWithoutId(true));
+      const queryList = new QueryList<TruncatablePartComponent>();
+      queryList.reset(parts);
+      component.truncatableComponents = queryList;
+
+      component.toggleView(jasmine.createSpyObj('event', ['stopPropagation']), false);
+
+      expect(component.expanded).toBeFalse();
+      parts.forEach((part) => expect(part.lines).toEqual('1'));
+    });
+
+    it('should only offer the expand-all control once a part reports itself truncated', fakeAsync(() => {
+      component.onTruncatedStateChange(1, false);
+      tick();
+      expect(component.expandable).toBeFalse();
+
+      component.onTruncatedStateChange(2, true);
+      tick();
+      expect(component.expandable).toBeTrue();
+    }));
   };
 }