Skip to content

Commit 46bbf35

Browse files
Merge branch 'develop' into 11771-fix-npe-in-update-licenses
2 parents c2a90ef + ad031a9 commit 46bbf35

File tree

6 files changed

+17
-12
lines changed

6 files changed

+17
-12
lines changed

doc/sphinx-guides/source/container/dev-usage.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -145,13 +145,13 @@ Accessing Harvesting Log Files
145145

146146
\1. Open a terminal and access the Dataverse container.
147147

148-
Run the following command to access the Dataverse container (assuming your container is named dataverse-1):
148+
Run the following command to access the Dataverse container:
149149

150150
.. code-block::
151151
152-
docker exec -it dataverse-1 bash
152+
docker exec -it dev_dataverse bash
153153
154-
This command opens an interactive shell within the dataverse-1 container.
154+
This command opens an interactive shell within the dev_dataverse container.
155155

156156
\2. Navigate to the log files directory.
157157

doc/sphinx-guides/source/developers/tips.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ Here's an example of using these credentials from within the PostgreSQL containe
124124

125125
.. code-block:: bash
126126
127-
pdurbin@beamish dataverse % docker exec -it postgres-1 bash
127+
pdurbin@beamish dataverse % docker exec -it dev_postgres bash
128128
root@postgres:/# export PGPASSWORD=secret
129129
root@postgres:/# psql -h localhost -U dataverse dataverse
130130
psql (16.3 (Debian 16.3-1.pgdg120+1))

pom.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1266,6 +1266,7 @@
12661266
</image>
12671267
</images>
12681268
<autoCreateCustomNetworks>true</autoCreateCustomNetworks>
1269+
<containerNamePattern>%a</containerNamePattern>
12691270
</configuration>
12701271
</plugin>
12711272
<plugin>

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2138,6 +2138,10 @@ private String init(boolean initFull) {
21382138

21392139
if (retrieveDatasetVersionResponse != null && !retrieveDatasetVersionResponse.wasRequestedVersionRetrieved()) {
21402140
//msg("checkit " + retrieveDatasetVersionResponse.getDifferentVersionMessage());
2141+
if ("DRAFT".equals(version)) {
2142+
// redirect to the latest published instead:
2143+
return "/dataset.xhtml?persistentId=" + dataset.getGlobalId().asString() + "&faces-redirect=true";
2144+
}
21412145
JsfHelper.addWarningMessage(retrieveDatasetVersionResponse.getDifferentVersionMessage());//BundleUtil.getStringFromBundle("dataset.message.metadataSuccess"));
21422146
}
21432147

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
import edu.harvard.iq.dataverse.settings.SettingsServiceBean;
1515
import edu.harvard.iq.dataverse.settings.SettingsServiceBean.Key;
1616
import edu.harvard.iq.dataverse.util.BundleUtil;
17-
import edu.harvard.iq.dataverse.util.MailUtil;
1817
import edu.harvard.iq.dataverse.util.StringUtil;
1918
import edu.harvard.iq.dataverse.util.SystemConfig;
2019
import edu.harvard.iq.dataverse.UserNotification.Type;
@@ -51,7 +50,8 @@
5150
public class SettingsWrapper implements java.io.Serializable {
5251

5352
static final Logger logger = Logger.getLogger(SettingsWrapper.class.getCanonicalName());
54-
53+
public static final String COMMA_BETWEEN_OPTIONAL_WHITE_SPACE = "\\s*,\\s*";
54+
5555
@EJB
5656
SettingsServiceBean settingsService;
5757

@@ -396,7 +396,7 @@ public boolean isRsyncOnly() {
396396
if (uploadMethods==null){
397397
rsyncOnly = false;
398398
} else {
399-
rsyncOnly = Arrays.asList(uploadMethods.toLowerCase().split("\\s*,\\s*")).size() == 1 && uploadMethods.toLowerCase().equals(SystemConfig.FileUploadMethods.RSYNC.toString());
399+
rsyncOnly = Arrays.asList(uploadMethods.toLowerCase().split(COMMA_BETWEEN_OPTIONAL_WHITE_SPACE)).size() == 1 && uploadMethods.toLowerCase().equals(SystemConfig.FileUploadMethods.RSYNC.toString());
400400
}
401401
}
402402
}
@@ -428,7 +428,7 @@ public Integer getUploadMethodsCount() {
428428
if (uploadMethods==null){
429429
uploadMethodsCount = 0;
430430
} else {
431-
uploadMethodsCount = Arrays.asList(uploadMethods.toLowerCase().split("\\s*,\\s*")).size();
431+
uploadMethodsCount = Arrays.asList(uploadMethods.toLowerCase().split(COMMA_BETWEEN_OPTIONAL_WHITE_SPACE)).size();
432432
}
433433
}
434434
return uploadMethodsCount;
@@ -502,7 +502,7 @@ public boolean shouldBeAnonymized(DatasetField df) {
502502
if (anonymizedFieldTypes == null) {
503503
anonymizedFieldTypes = new ArrayList<String>();
504504
String names = get(SettingsServiceBean.Key.AnonymizedFieldTypeNames.toString(), "");
505-
anonymizedFieldTypes.addAll(Arrays.asList(names.split(",\\s")));
505+
anonymizedFieldTypes.addAll(Arrays.asList(names.split(COMMA_BETWEEN_OPTIONAL_WHITE_SPACE)));
506506
}
507507
return anonymizedFieldTypes.contains(df.getDatasetFieldType().getName());
508508
}
@@ -830,7 +830,7 @@ private Boolean getUploadMethodAvailable(String method){
830830
if (uploadMethods==null){
831831
return false;
832832
} else {
833-
return Arrays.asList(uploadMethods.toLowerCase().split("\\s*,\\s*")).contains(method);
833+
return Arrays.asList(uploadMethods.toLowerCase().split(COMMA_BETWEEN_OPTIONAL_WHITE_SPACE)).contains(method);
834834
}
835835
}
836836

src/main/java/edu/harvard/iq/dataverse/api/Datasets.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5317,7 +5317,7 @@ public Response getPrivateUrlDatasetVersion(@PathParam("privateUrlToken") String
53175317
}
53185318
JsonObjectBuilder responseJson;
53195319
if (isAnonymizedAccess) {
5320-
List<String> anonymizedFieldTypeNamesList = new ArrayList<>(Arrays.asList(anonymizedFieldTypeNames.split(",\\s")));
5320+
List<String> anonymizedFieldTypeNamesList = new ArrayList<>(Arrays.asList(anonymizedFieldTypeNames.split(SettingsWrapper.COMMA_BETWEEN_OPTIONAL_WHITE_SPACE)));
53215321
responseJson = json(dsv, anonymizedFieldTypeNamesList, true, returnOwners);
53225322
} else {
53235323
responseJson = json(dsv, null, true, returnOwners);
@@ -5343,7 +5343,7 @@ public Response getPreviewUrlDatasetVersion(@PathParam("previewUrlToken") String
53435343
}
53445344
JsonObjectBuilder responseJson;
53455345
if (isAnonymizedAccess) {
5346-
List<String> anonymizedFieldTypeNamesList = new ArrayList<>(Arrays.asList(anonymizedFieldTypeNames.split(",\\s")));
5346+
List<String> anonymizedFieldTypeNamesList = new ArrayList<>(Arrays.asList(anonymizedFieldTypeNames.split(SettingsWrapper.COMMA_BETWEEN_OPTIONAL_WHITE_SPACE)));
53475347
responseJson = json(dsv, anonymizedFieldTypeNamesList, true, returnOwners);
53485348
} else {
53495349
responseJson = json(dsv, null, true, returnOwners);

0 commit comments

Comments
 (0)