diff --git a/CHANGELOG.md b/CHANGELOG.md index 9af9ce81bb..b10bba8e94 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -21,6 +21,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Removed - Enunciate rest documentation +- Obsolete fields in data transfer objects ## 2.0.0-rc4 - 2020-02-14 ### Added diff --git a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/GroupDto.java b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/GroupDto.java index 3566589b3c..d17ab83fef 100644 --- a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/GroupDto.java +++ b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/GroupDto.java @@ -12,9 +12,10 @@ import sonia.scm.util.ValidationUtil; import javax.validation.constraints.Pattern; import java.time.Instant; import java.util.List; -import java.util.Map; -@Getter @Setter @NoArgsConstructor +@Getter +@Setter +@NoArgsConstructor public class GroupDto extends HalRepresentation { private Instant creationDate; @@ -24,7 +25,6 @@ public class GroupDto extends HalRepresentation { @Pattern(regexp = ValidationUtil.REGEX_NAME) private String name; private String type; - private Map properties; private List members; private boolean external; diff --git a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/RepositoryDto.java b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/RepositoryDto.java index df0fb0bafc..852d62fd5d 100644 --- a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/RepositoryDto.java +++ b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/RepositoryDto.java @@ -7,16 +7,17 @@ import de.otto.edison.hal.Links; import lombok.Getter; import lombok.NoArgsConstructor; import lombok.Setter; -import javax.validation.constraints.Email; -import javax.validation.constraints.NotEmpty; import sonia.scm.util.ValidationUtil; +import javax.validation.constraints.Email; +import javax.validation.constraints.NotEmpty; import javax.validation.constraints.Pattern; import java.time.Instant; import java.util.List; -import java.util.Map; -@Getter @Setter @NoArgsConstructor +@Getter +@Setter +@NoArgsConstructor public class RepositoryDto extends HalRepresentation { @Email @@ -30,10 +31,8 @@ public class RepositoryDto extends HalRepresentation { private String namespace; @Pattern(regexp = ValidationUtil.REGEX_REPOSITORYNAME) private String name; - private boolean archived = false; @NotEmpty private String type; - protected Map properties; RepositoryDto(Links links, Embedded embedded) { super(links, embedded); diff --git a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/UserDto.java b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/UserDto.java index d8a9885398..798c4b2513 100644 --- a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/UserDto.java +++ b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/UserDto.java @@ -7,13 +7,12 @@ import de.otto.edison.hal.Links; import lombok.Getter; import lombok.NoArgsConstructor; import lombok.Setter; -import javax.validation.constraints.Email; -import javax.validation.constraints.NotEmpty; import sonia.scm.util.ValidationUtil; +import javax.validation.constraints.Email; +import javax.validation.constraints.NotEmpty; import javax.validation.constraints.Pattern; import java.time.Instant; -import java.util.Map; @NoArgsConstructor @Getter @Setter public class UserDto extends HalRepresentation { @@ -30,7 +29,6 @@ public class UserDto extends HalRepresentation { @JsonInclude(JsonInclude.Include.NON_NULL) private String password; private String type; - private Map properties; UserDto(Links links, Embedded embedded) { super(links, embedded); diff --git a/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryRootResourceTest.java b/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryRootResourceTest.java index fc0de8fe5c..60d9e3496d 100644 --- a/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryRootResourceTest.java +++ b/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryRootResourceTest.java @@ -138,19 +138,6 @@ public class RepositoryRootResourceTest extends RepositoryTestBase { assertTrue(response.getContentAsString().contains("\"name\":\"repo\"")); } - @Test - public void shouldMapProperties() throws URISyntaxException, UnsupportedEncodingException { - Repository repository = mockRepository("space", "repo"); - repository.setProperty("testKey", "testValue"); - - MockHttpRequest request = MockHttpRequest.get("/" + RepositoryRootResource.REPOSITORIES_PATH_V2 + "space/repo"); - MockHttpResponse response = new MockHttpResponse(); - - dispatcher.invoke(request, response); - - assertTrue(response.getContentAsString().contains("\"testKey\":\"testValue\"")); - } - @Test public void shouldGetAll() throws URISyntaxException, UnsupportedEncodingException { PageResult singletonPageResult = createSingletonPageResult(mockRepository("space", "repo")); diff --git a/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryToRepositoryDtoMapperTest.java b/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryToRepositoryDtoMapperTest.java index 4b02508ae8..9c6f87a765 100644 --- a/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryToRepositoryDtoMapperTest.java +++ b/scm-webapp/src/test/java/sonia/scm/api/v2/resources/RepositoryToRepositoryDtoMapperTest.java @@ -77,16 +77,6 @@ public class RepositoryToRepositoryDtoMapperTest { assertEquals("none@example.com", dto.getContact()); } - @Test - public void shouldMapPropertiesProperty() { - Repository repository = createTestRepository(); - repository.setProperty("testKey", "testValue"); - - RepositoryDto dto = mapper.map(repository); - - assertEquals("testValue", dto.getProperties().get("testKey")); - } - @Test @SubjectAware(username = "unpriv") public void shouldCreateLinksForUnprivilegedUser() {