Merge pull request openequella#1708 from SammyIsConfused/bugfix/fix_l…

…ost_metadata_due_to_mandatory_fields Contribution wizard lost metadata fix
SammyIsConfused · May 14, 2020 · b9d8fd1 · b9d8fd1
1 parent 6caa6f4
commit b9d8fd1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/Source/Plugins/Core/com.equella.core/src/com/tle/web/wizard/section/RootWizardSection.java b/Source/Plugins/Core/com.equella.core/src/com/tle/web/wizard/section/RootWizardSection.java
@@ -54,6 +54,7 @@
 import com.tle.web.template.Decorations;
 import com.tle.web.template.Decorations.MenuMode;
 import com.tle.web.viewurl.ItemSectionInfo;
+import com.tle.web.wizard.WebWizardPage;
 import com.tle.web.wizard.WizardExceptionHandler;
 import com.tle.web.wizard.WizardService;
 import com.tle.web.wizard.WizardState;
@@ -303,8 +304,8 @@ protected WizardSectionInfo getWizardInfo(SectionInfo info) {
 
  void validateMandatoryFields(SectionInfo info, WizardState state) {
  PagesSection ps = info.lookupSection(PagesSection.class);
- wizardService
- .getWizardPages(state)
+ wizardService.getWizardPages(state).stream()
+ .filter(WebWizardPage::isViewable)
  .forEach(p -> wizardService.ensureInitialisedPage(info, p, ps.getReloadFunction(), true));
  }
 }