Skip to content

Commit

Permalink
VUU25: Downgrade Java 17->11 for layout-server
Browse files Browse the repository at this point in the history
  • Loading branch information
cfisher-scottlogic committed Oct 9, 2023
1 parent 10e53ad commit 58ab147
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion layout-server/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
<name>layout-server</name>
<description>layout-server</description>
<properties>
<java.version>17</java.version>
<java.version>11</java.version>
</properties>

<dependencies>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
package org.finos.vuu.layoutserver.controller;

import java.util.List;
import java.util.UUID;
import lombok.RequiredArgsConstructor;
import org.finos.vuu.layoutserver.dto.request.LayoutRequestDTO;
import org.finos.vuu.layoutserver.dto.response.CreateLayoutResponseDTO;
Expand All @@ -19,9 +21,6 @@
import org.springframework.web.bind.annotation.ResponseStatus;
import org.springframework.web.bind.annotation.RestController;

import java.util.List;
import java.util.UUID;

@RequiredArgsConstructor
@RestController
@RequestMapping("/layouts")
Expand Down Expand Up @@ -50,9 +49,9 @@ public LayoutResponseDTO getLayout(@PathVariable UUID id) {
public List<MetadataResponseDTO> getMetadata() {

return layoutService.getMetadata()
.stream()
.map(metadata -> mapper.map(metadata, MetadataResponseDTO.class))
.toList();
.stream()
.map(metadata -> mapper.map(metadata, MetadataResponseDTO.class))
.collect(java.util.stream.Collectors.toList());
}

/**
Expand Down Expand Up @@ -80,7 +79,8 @@ public CreateLayoutResponseDTO createLayout(@RequestBody LayoutRequestDTO layout
@ResponseStatus(HttpStatus.NO_CONTENT)
@PutMapping("/{id}")
public void updateLayout(@PathVariable UUID id, @RequestBody LayoutRequestDTO newLayout) {
// TODO I'm not sure about this, I think layoutService should be doing more logic, not the mapper here
// TODO I'm not sure about this, I think layoutService should be doing more logic, not
// the mapper here
Layout layout = layoutService.getLayout(id);

mapper.map(newLayout, layout);
Expand Down

0 comments on commit 58ab147

Please sign in to comment.