Prov: Save context cases added #4343

Fixes a bug where:
- User uploads DataFile and saves it
- User goes to editDatafilePage after save and uploads prov/sets entity.
- Entity is not saved because editDatafilePage did not think it needed to save the context.

Logic was added to save context if the page wasn't saved otherwise.

src/main/java/edu/harvard/iq/dataverse/EditDatafilesPage.java

@@ -1263,11 +1263,11 @@ public String save() {
 
         workingVersion = dataset.getEditVersion();
         logger.fine("working version id: "+workingVersion.getId());
-        
+
         if(systemConfig.isProvCollectionEnabled()) {        
             try {
-                provPopupFragmentBean.saveStagedProvJson(false);
-
+                //If datasetUpdateRequired did not trigger save the prov code will need to save its staged changes.
+                provPopupFragmentBean.saveStagedProvJson(!datasetUpdateRequired); 
             } catch (AbstractApiBean.WrappedResponse ex) {
                 JsfHelper.addErrorMessage(getBundleString("file.metadataTab.provenance.error"));
                 Logger.getLogger(EditDatafilesPage.class.getName()).log(Level.SEVERE, null, ex);

src/main/java/edu/harvard/iq/dataverse/ProvPopupFragmentBean.java

@@ -265,7 +265,7 @@ public void saveStagedProvJson(boolean saveContext) throws AbstractApiBean.Wrapp
                 df = execCommand(new PersistProvJsonCommand(dvRequestService.getDataverseRequest(), df, provString, df.getProvEntityName(), saveContext));
             } 
             mapEntry.dataFile = df;
-            provenanceUpdates.put(mapEntry.dataFile.getChecksumValue(), mapEntry); //MAD: Modifying the set as we go through it is probably causing errors
+            provenanceUpdates.put(mapEntry.dataFile.getChecksumValue(), mapEntry); //Updates the datafile to the latest.
         }
     }