feat(pid): update DataverseServiceBean.wantsDatasetVersionPids() calls with info about minor or major version

Author: poikilotherm

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/FinalizeDatasetPublicationCommand.java

@@ -191,8 +191,9 @@ public Dataset execute(CommandContext ctxt) throws CommandException {
                  * Note: although it may seem wrong to trigger registering any version, it is a (configurable?)
                  * decision in the provider's realm if minor versions are just updates of a major or their own.
                  */
-                if (ctxt.dataverses().wantsDatasetVersionPids(theDataset.getOwner())) {
-                    DatasetVersion version = theDataset.getLatestVersion();
+                DatasetVersion version = theDataset.getLatestVersion();
+                
+                if (ctxt.dataverses().wantsDatasetVersionPids(theDataset.getOwner(), version.getMinorVersionNumber() > 0)) {
                     GlobalIdServiceBean idServiceBean = GlobalIdServiceBean.getBean(theDataset.getProtocol(), ctxt);
                     Objects.requireNonNull(idServiceBean, "Could not retrieve PID provider");
 
@@ -434,7 +435,8 @@ private void publicizeExternalIdentifier(Dataset dataset, CommandContext ctxt) t
                 }
 
                 // Publish a PID for this dataset version (if activated). Leave details to the provider.
-                if (ctxt.dataverses().wantsDatasetVersionPids(dataset.getOwner())) {
+                
+                if (ctxt.dataverses().wantsDatasetVersionPids(dataset.getOwner(), dataset.getLatestVersion().getMinorVersionNumber() > 0)) {
                     DatasetVersion version = dataset.getLatestVersion();
                     idServiceBean.publicizeIdentifier(version);
                 }

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/RegisterDvObjectCommand.java

@@ -86,7 +86,9 @@ protected void executeImpl(CommandContext ctxt) throws CommandException {
                 if (target.isReleased()) {
                     // If this is a dataset, release a new dataset version first before, to be included in the dataset
                     if (target.isInstanceofDataset() &&
-                        ctxt.dataverses().wantsDatasetVersionPids((Dataverse) target.getOwner())) {
+                        ctxt.dataverses().wantsDatasetVersionPids(
+                            (Dataverse) target.getOwner(),
+                            ((Dataset) target).getLatestVersion().getMinorVersionNumber() > 0)) {
                         // TODO: publicize dataset version as well
                     }
 

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/UpdateDvObjectPIDMetadataCommand.java

@@ -50,7 +50,7 @@ protected void executeImpl(CommandContext ctxt) throws CommandException {
         GlobalIdServiceBean idServiceBean = GlobalIdServiceBean.getBean(target.getProtocol(), ctxt);
         try {
             // First publicize new dataset version if enabled, so it can be included in the update of the dataset
-            if (ctxt.dataverses().wantsDatasetVersionPids(target.getOwner())) {
+            if (ctxt.dataverses().wantsDatasetVersionPids(target.getOwner(), target.getLatestVersion().getMinorVersionNumber() > 0)) {
                 // TODO: publicize dataset version as well
             }