90873: issue 1380 - Fix to make default bundle config work without yml changes

config/config.example.yml

@@ -229,10 +229,8 @@ themes:
         href: assets/dspace/images/favicons/manifest.webmanifest
 
 # The default bundles that should always be displayed as suggestions when you upload a new bundle
-standardBundles:
+bundle:
-  - bundle: ORIGINAL
+  - standardBundles: [ ORIGINAL, THUMBNAIL, LICENSE ]
-  - bundle: THUMBNAIL
-  - bundle: LICENSE
 
 # Whether to enable media viewer for image and/or video Bitstreams (i.e. Bitstreams whose MIME type starts with 'image' or 'video').
 # For images, this enables a gallery viewer where you can zoom or page through images.

src/app/item-page/bitstreams/upload/upload-bitstream.component.ts

@@ -19,7 +19,6 @@ import { RequestService } from '../../../core/data/request.service';
 import { getBitstreamModuleRoute } from '../../../app-routing-paths';
 import { getEntityEditRoute } from '../../item-page-routing-paths';
 import { environment } from '../../../../environments/environment';
-import { StandardBundleConfig } from '../../../../config/standard-bundle-config.interface';
 
 @Component({
   selector: 'ds-upload-bitstream',
@@ -115,29 +114,27 @@ export class UploadBitstreamComponent implements OnInit, OnDestroy {
         if (remoteData.hasSucceeded) {
           if (remoteData.payload.page) {
             this.bundles = remoteData.payload.page;
-          } else if (environment.standardBundles) {
+            for (const defaultBundle of environment.bundle.standardBundles) {
-            this.bundles = environment.standardBundles.map((defaultBundle: StandardBundleConfig) => Object.assign(new Bundle(), {
-                _name: defaultBundle.bundle,
-                type: 'bundle'
-              })
-            );
-          }
-          if (remoteData.payload.page && environment.standardBundles) {
-            for (const defaultBundle of environment.standardBundles) {
               let check = true;
               remoteData.payload.page.forEach((bundle: Bundle) => {
                 // noinspection JSDeprecatedSymbols
-                if (defaultBundle.bundle === bundle.name) {
+                if (defaultBundle === bundle.name) {
                   check = false;
                 }
               });
               if (check) {
                 this.bundles.push(Object.assign(new Bundle(), {
-                  _name: defaultBundle.bundle,
+                  _name: defaultBundle,
                   type: 'bundle'
                 }));
               }
             }
+          } else {
+            this.bundles = environment.bundle.standardBundles.map((defaultBundle: string) => Object.assign(new Bundle(), {
+                _name: defaultBundle,
+                type: 'bundle'
+              })
+            );
           }
           return observableOf(remoteData.payload.page);
         }
@@ -232,7 +229,9 @@ export class UploadBitstreamComponent implements OnInit, OnDestroy {
   onClick(bundle: Bundle) {
     this.selectedBundleId = bundle.id;
     this.selectedBundleName = bundle.name;
-    this.setUploadUrl();
+    if (bundle.id) {
+      this.setUploadUrl();
+    }
   }
 
   /**

src/config/app-config.interface.ts

@@ -14,7 +14,7 @@ import { AuthConfig } from './auth-config.interfaces';
 import { UIServerConfig } from './ui-server-config.interface';
 import { MediaViewerConfig } from './media-viewer-config.interface';
 import { BrowseByConfig } from './browse-by-config.interface';
-import { StandardBundleConfig } from './standard-bundle-config.interface';
+import { BundleConfig } from './bundle-config.interface';
 
 interface AppConfig extends Config {
   ui: UIServerConfig;
@@ -33,7 +33,7 @@ interface AppConfig extends Config {
   collection: CollectionPageConfig;
   themes: ThemeConfig[];
   mediaViewer: MediaViewerConfig;
-  standardBundles: StandardBundleConfig[];
+  bundle: BundleConfig;
 }
 
 const APP_CONFIG = new InjectionToken<AppConfig>('APP_CONFIG');

src/config/bundle-config.interface.ts

@@ -0,0 +1,11 @@
+import { Config } from './config.interface';
+
+export interface BundleConfig extends Config {
+
+  /**
+   * List of standard bundles to select in adding bitstreams to items
+   * Used by {@link UploadBitstreamComponent}.
+   */
+  standardBundles: string[];
+
+}

src/config/default-app-config.ts

@@ -14,6 +14,7 @@ import { ServerConfig } from './server-config.interface';
 import { SubmissionConfig } from './submission-config.interface';
 import { ThemeConfig } from './theme.model';
 import { UIServerConfig } from './ui-server-config.interface';
+import { BundleConfig } from './bundle-config.interface';
 
 export class DefaultAppConfig implements AppConfig {
   production = false;
@@ -301,11 +302,9 @@ export class DefaultAppConfig implements AppConfig {
   ];
   // The default bundles that should always be displayed when you edit or add a bundle even when no bundle has been
   // added to the item yet.
-  standardBundles: [
+  bundle: BundleConfig = {
-    { bundle: 'ORIGINAL' },
+    standardBundles: ['ORIGINAL', 'THUMBNAIL', 'LICENSE']
-    { bundle: 'THUMBNAIL' },
+  };
-    { bundle: 'LICENSE' }
-  ];
   // Whether to enable media viewer for image and/or video Bitstreams (i.e. Bitstreams whose MIME type starts with "image" or "video").
   // For images, this enables a gallery viewer where you can zoom or page through images.
   // For videos, this enables embedded video streaming

src/config/standard-bundle-config.interface.ts

@@ -1,10 +0,0 @@
-import { Config } from './config.interface';
-
-export interface StandardBundleConfig extends Config {
-
-  /**
-   * Used by {@link UploadBitstreamComponent}.
-   */
-  bundle: string;
-
-}

src/environments/environment.test.ts

@@ -224,11 +224,9 @@ export const environment: BuildConfig = {
       name: 'base',
     },
   ],
-  standardBundles: [
+  bundle: {
-    { bundle: 'ORIGINAL' },
+    standardBundles: ['ORIGINAL', 'THUMBNAIL', 'LICENSE'],
-    { bundle: 'THUMBNAIL' },
+  },
-    { bundle: 'LICENSE' }
-  ],
   mediaViewer: {
     image: true,
     video: true