Skip to content

Commit 53cf432

Browse files
committed
reuse strings
1 parent d3b5e31 commit 53cf432

2 files changed

Lines changed: 10 additions & 20 deletions

File tree

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

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -672,32 +672,31 @@ protected Response getRoleAssignmentHistoryResponse(DvObject dvObject, Authentic
672672

673673
// Get the role assignment history
674674
List<DataverseRoleServiceBean.RoleAssignmentHistoryEntry> history = null;
675-
String definitionPoint;
676675
if (forFiles == false) {
677676
history = rolesSvc.getRoleAssignmentHistory(dvObject.getId());
678-
definitionPoint = BundleUtil.getStringFromBundle("datasets.api.permissions.history.definitionPoint");
679677
} else {
680678
history = rolesSvc.getFilesRoleAssignmentHistory(dvObject.getId());
681-
definitionPoint = BundleUtil.getStringFromBundle("datasets.api.permissions.history.definitionPoints");
682679
}
683680

684681
List<MediaType> acceptedTypes = headers.getAcceptableMediaTypes();
685682
boolean wantCSV = acceptedTypes.stream()
686683
.anyMatch(mt -> mt.toString().equals("text/csv"));
687684

688685
if (wantCSV) {
689-
String assigneeHeader = BundleUtil.getStringFromBundle("datasets.api.permissions.history.assignee");
690-
String roleHeader = BundleUtil.getStringFromBundle("datasets.api.permissions.history.role");
691-
String assignedByHeader = BundleUtil.getStringFromBundle("datasets.api.permissions.history.assignedBy");
692-
String assignedAtHeader = BundleUtil.getStringFromBundle("datasets.api.permissions.history.assignedAt");
693-
String revokedByHeader = BundleUtil.getStringFromBundle("datasets.api.permissions.history.revokedBy");
694-
String revokedAtHeader = BundleUtil.getStringFromBundle("datasets.api.permissions.history.revokedAt");
686+
//Reusing strings from history panel
687+
String definedOn = BundleUtil.getStringFromBundle("dataverse.permissions.history.definedOn");
688+
String assigneeHeader = BundleUtil.getStringFromBundle("dataverse.permissions.history.assignee");
689+
String roleHeader = BundleUtil.getStringFromBundle("dataverse.permissions.history.role");
690+
String assignedByHeader = BundleUtil.getStringFromBundle("dataverse.permissions.history.assignedBy");
691+
String assignedAtHeader = BundleUtil.getStringFromBundle("dataverse.permissions.history.assignedAt");
692+
String revokedByHeader = BundleUtil.getStringFromBundle("dataverse.permissions.history.revokedBy");
693+
String revokedAtHeader = BundleUtil.getStringFromBundle("dataverse.permissions.history.revokedAt");
695694

696695
// Generate CSV response
697696
StringBuilder csvBuilder = new StringBuilder();
698697
// Add CSV header with internationalized column names
699698
csvBuilder
700-
.append(definitionPoint).append(",")
699+
.append(definedOn).append(",")
701700
.append(assigneeHeader).append(",")
702701
.append(roleHeader).append(",")
703702
.append(assignedByHeader).append(",")
@@ -736,7 +735,7 @@ protected Response getRoleAssignmentHistoryResponse(DvObject dvObject, Authentic
736735
JsonArrayBuilder jsonArray = Json.createArrayBuilder();
737736
for (DataverseRoleServiceBean.RoleAssignmentHistoryEntry entry : history) {
738737
JsonObjectBuilder job = Json.createObjectBuilder()
739-
.add(definitionPoint, entry.getDefinitionPointIdsAsString())
738+
.add("definedOn", entry.getDefinitionPointIdsAsString())
740739
.add("assigneeIdentifier", entry.getAssigneeIdentifier())
741740
.add("roleName", entry.getRoleName())
742741
.add("assignedBy", entry.getAssignedBy())

src/main/java/propertyFiles/Bundle.properties

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3292,15 +3292,6 @@ updateDatasetFieldsCommand.api.processDatasetUpdate.parseError=Error parsing dat
32923292

32933293
#AbstractApiBean.java
32943294
abstractApiBean.error.datasetInternalVersionNumberIsOutdated=Dataset internal version number {0} is outdated
3295-
# Role Assignment History (used in the AbstractApiBean)
3296-
datasets.api.permissions.history.definitionPoint=Definition Point
3297-
datasets.api.permissions.history.definitionPoints=Definition Points
3298-
datasets.api.permissions.history.assignee=Assignee
3299-
datasets.api.permissions.history.role=Role
3300-
datasets.api.permissions.history.assignedBy=Assigned By
3301-
datasets.api.permissions.history.assignedAt=Assigned At
3302-
datasets.api.permissions.history.revokedBy=Revoked By
3303-
datasets.api.permissions.history.revokedAt=Revoked At
33043295

33053296
#RoleAssigneeServiceBean.java
33063297
roleAssigneeServiceBean.error.dataverseRequestCannotBeNull=DataverseRequest cannot be null.

0 commit comments

Comments
 (0)