Remove overly elaborate licenses prop

ImperialCollegeLondon · Dec 9, 2024 · 6231cb9 · 6231cb9
1 parent 140adc3
commit 6231cb9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/site/ic_data_repo/assets/semantic-ui/js/ic_data_repo/LimitedLicenseField.js b/site/ic_data_repo/assets/semantic-ui/js/ic_data_repo/LimitedLicenseField.js
@@ -99,7 +99,7 @@ class LicenseFieldForm extends Component {
               );
             })}
           </List>
-          <LimitedLicenseModal
+          {values.metadata.rights.length === 0 && (<LimitedLicenseModal
             searchConfig={searchConfig}
             trigger={
               <Button type="button" key="standard" icon labelPosition="left">
@@ -111,9 +111,9 @@ class LicenseFieldForm extends Component {
               formikArrayPush(selectedLicense);
             }}
             action="add"
-            licenses={values.metadata.rights.length}
             serializeLicenses={serializeLicenses}
-          />
+          />)
+          }
         </Form.Field>
       </DndProvider>
     );

diff --git a/site/ic_data_repo/assets/semantic-ui/js/ic_data_repo/LimitedLicenseModal.js b/site/ic_data_repo/assets/semantic-ui/js/ic_data_repo/LimitedLicenseModal.js
@@ -67,7 +67,6 @@ export class LimitedLicenseModal extends Component {
 
   render() {
     const {
-      licenses,
       trigger,
       action,
       searchConfig,
@@ -83,10 +82,6 @@ export class LimitedLicenseModal extends Component {
       link: "",
     };
 
-    // Do not display the add button if a license has been selected
-    if(licenses)
-        return null;
-
     const searchApi = new InvenioSearchApi(searchConfig.searchApi);
     return (
       <Formik