hazza/DSpace
1
0
Fork 0
mirror of https://github.com/DSpace/DSpace.git synced 2025-10-17 23:13:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

75404: [Issue 3062] Self links contain embed params

Browse Source
This commit is contained in:
Yana De Pauw
2021-01-04 11:33:46 +01:00
parent 250c87dc16
commit 01bf6caf1e
2 changed files with 14 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
dspace-server-webapp/src/main/java/org/dspace/app/rest/RestResourceController.java
View File

@@ -1021,15 +1021,18 @@ public class RestResourceController implements InitializingBean {
/**
* Internal method to convert the parameters provided as a MultivalueMap as a string to use in the self-link.
* This function will exclude all "embed" parameters
* @param parameters
* @return encoded uriString containing request parameters
* @return encoded uriString containing request parameters without embed parameter
*/
private String getEncodedParameterStringFromRequestParams(
@RequestParam MultiValueMap<String, Object> parameters) {
UriComponentsBuilder uriComponentsBuilder = UriComponentsBuilder.newInstance();
for (String key : parameters.keySet()) {
uriComponentsBuilder.queryParam(key, parameters.get(key));
if (!StringUtils.equals(key, "embed")) {
uriComponentsBuilder.queryParam(key, parameters.get(key));
}
}
return uriComponentsBuilder.encode().build().toString();
}

9
dspace-server-webapp/src/test/java/org/dspace/app/rest/RestResourceControllerIT.java
View File

@@ -91,6 +91,15 @@ public class RestResourceControllerIT extends AbstractControllerIntegrationTest
endsWith("/api/core/metadatafields/search/byFieldName")));
}
@Test
public void selfLinkContainsRequestParametersAndEmbedsWhenProvided() throws Exception {
// When we call a search endpoint with additional parameters and an embed parameter
getClient().perform(get("/api/core/metadatafields/search/byFieldName?schema=dc&offset=0&embed=schema"))
// The self link should contain those same parameters
.andExpect(jsonPath("$._links.self.href", endsWith(
"/api/core/metadatafields/search/byFieldName?schema=dc&offset=0")));
}
}