ResponseEntity out

src/main/java/es/uv/saic/feign/CoreClient.java

@@ -1,17 +1,23 @@
 package es.uv.saic.feign;
 
+import java.io.IOException;
 import java.math.BigInteger;
+import java.net.MalformedURLException;
 import java.util.List;
 
+import javax.xml.parsers.ParserConfigurationException;
+
 import org.springframework.cloud.openfeign.FeignClient;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestParam;
+import org.xml.sax.SAXException;
 
 import es.uv.saic.domain.Indicador;
 import es.uv.saic.dto.IndicadorEnquestaDTO;
+import es.uv.saic.dto.IndicadorEnquestaValorDTO;
 import es.uv.saic.dto.IndicadorEnquestaValorDTOImp;
 import es.uv.saic.dto.InstanciaTascaDTO;
 import es.uv.saic.dto.NomProcesOrganDTO;
@@ -24,18 +30,16 @@ import es.uv.saic.dto.TascaInformeTransferDTO;
 public interface CoreClient {
     //IndicadorController.java
     @GetMapping("/getFromTitulacio/{idTitulacio}/{curs}")
-    List<Indicador> getFromTitulacion(@PathVariable("idTitulacio") Integer idTitulacio, @PathVariable("curs") Integer curs);
+    public List<Indicador> getFromTitulacion(@PathVariable Integer idTitulacio, @PathVariable Integer curs) 
+        throws ParserConfigurationException, MalformedURLException, IOException, SAXException;
 
-    @PostMapping("/allInds2")
-    List<IndicadorEnquestaValorDTOImp> getAllInds2(@RequestBody IndicadorEnquestaDTO indicadorEnquestaDTO);
+   @PostMapping("/allInds")
+    public List<IndicadorEnquestaValorDTO> getAllInds2(@RequestBody IndicadorEnquestaDTO indicadorEnquestaDTO);
 
     //TascaController.java
     @GetMapping("/instanciatasca/{id}")
     InstanciaTascaDTO findInstanciaTascaById(@PathVariable("id") BigInteger id);
 
-    @GetMapping("/instanciatasca2/{id}")
-    InstanciaTascaDTO findInstanciaTascaById2(@PathVariable("id") BigInteger id);
-
     @PostMapping("/getReportFromNomProcesOrgan")
     InstanciaTascaDTO getReportFromNomProcesOrgan(@RequestBody NomProcesOrganDTO nomProcesOrgan);
     

src/main/java/es/uv/saic/service/PlantillaService.java

@@ -316,8 +316,9 @@ public class PlantillaService {
 		
 		/*  Indicadores de encuestas y otros almacenados en BD  */
 		IndicadorEnquestaDTO indicadorEnquestaDTO = new IndicadorEnquestaDTO(idTitulacio, idCentre, curs);
-		List<IndicadorEnquestaValorDTOImp> enquestesT = core.getAllInds2(indicadorEnquestaDTO);
-
+		List<IndicadorEnquestaValorDTOImp> enquestesT = core.getAllInds2(indicadorEnquestaDTO).stream()
+															.map(ind -> (IndicadorEnquestaValorDTOImp) ind)
+															.toList();
 				
 		for(IndicadorEnquestaValorDTOImp i : enquestesT) {
 			String indicador = i.getAmbit().toLowerCase().equals("t") ? (i.getEnquesta().toLowerCase()+"_"+i.getIndicador().toLowerCase()) : (i.getEnquesta().toLowerCase()+"_"+i.getIndicador().toLowerCase()+"_"+i.getAmbit().toLowerCase());
@@ -424,7 +425,7 @@ public class PlantillaService {
 			e.remove();
 		}
 		
-		InstanciaTascaDTO ita = core.findInstanciaTascaById2(idtascai);
+		InstanciaTascaDTO ita = core.findInstanciaTascaById(idtascai);
 		if(ita.getTasca().getNomRol().equals("u_uq")) {
 			d.body().append(this.parseComments(d.html()));
 		}
@@ -485,7 +486,7 @@ public class PlantillaService {
 		}
 		
 		if(idtascai.isPresent()) {
-			InstanciaTascaDTO ita = core.findInstanciaTascaById2(idtascai.get());
+			InstanciaTascaDTO ita = core.findInstanciaTascaById(idtascai.get());
 			if(ita.getTasca().getNomRol().equals("u_uq")) {
 				d.body().append(this.parseComments(d.html()));
 			}

src/main/java/es/uv/saic/web/DocumentController.java

@@ -3,7 +3,6 @@ package es.uv.saic.web;
 import java.io.IOException;
 
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
@@ -42,15 +41,8 @@ public class DocumentController {
      * endpoint para findByID
      */
     @GetMapping("/document/{idDocument}")
-    public ResponseEntity<?> findByID(@PathVariable Integer idDocument) {
-        try {
-            Document document = ds.findById(idDocument);
-            return (document != null) ? ResponseEntity.ok(document) : 
-                ResponseEntity.badRequest().body("El documento no se ha podido encontrar.") ;
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("No se encontro el documento: " + e.getMessage());
-        }
+    public Document findByID(@PathVariable Integer idDocument) {
+       return ds.findById(idDocument);
     }
     
 
@@ -58,17 +50,9 @@ public class DocumentController {
      * endpoint para upload
      */
     @PostMapping("/document/upload")
-    public ResponseEntity<String> upload(@RequestBody DocumentTmpDTO documentTmpDTO) {
-        try{
-            return ResponseEntity.ok(ds.upload(documentTmpDTO.getIdDocument(), documentTmpDTO.getIdCategoria(),
-                documentTmpDTO.getLugar(), documentTmpDTO.getTlugar(), documentTmpDTO.getFile()));
-        } catch(IllegalStateException e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("Error: " + e.getMessage());
-        } catch(IOException e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("Error: " + e.getMessage());
-        }
+    public String upload(@RequestBody DocumentTmpDTO documentTmpDTO) throws IOException {
+        return ds.upload(documentTmpDTO.getIdDocument(), documentTmpDTO.getIdCategoria(),
+            documentTmpDTO.getLugar(), documentTmpDTO.getTlugar(), documentTmpDTO.getFile());
     }
 
     @PostMapping("/document/archive")

src/main/java/es/uv/saic/web/DownloadController.java

@@ -284,7 +284,9 @@ public class DownloadController {
 		/*  Indicadores de encuestas  */
 		List<IndicadorEnquestaValorDTOImp> enquestesT;
 		IndicadorEnquestaDTO iEnquestaDTO = new IndicadorEnquestaDTO(idTitulacio, idCentre, curs);
-		enquestesT = core.getAllInds2(iEnquestaDTO);
+		enquestesT = core.getAllInds2(iEnquestaDTO).stream()
+							.map(ind -> (IndicadorEnquestaValorDTOImp) ind)
+							.toList();
 				
 		for(IndicadorEnquestaValorDTOImp i : enquestesT) {
 			String indicador = i.getAmbit().equals("t") ? (i.getEnquesta()+"_"+i.getIndicador()) : (i.getEnquesta()+"_"+i.getIndicador()+"_"+i.getAmbit());
@@ -343,7 +345,9 @@ public class DownloadController {
 		/*  Indicadores de encuestas  */
 		List<IndicadorEnquestaValorDTOImp> enquestesT;	
 		IndicadorEnquestaDTO iEnquestaDTO = new IndicadorEnquestaDTO(idTitulacio, idCentre, curs);
-		enquestesT = core.getAllInds2(iEnquestaDTO);
+		enquestesT = core.getAllInds2(iEnquestaDTO).stream()
+										.map(ind -> (IndicadorEnquestaValorDTOImp) ind)
+										.toList();
 				
 		for(IndicadorEnquestaValorDTOImp i : enquestesT) {
 			String indicador = i.getAmbit().equals("t") ? (i.getEnquesta()+"_"+i.getIndicador()) : (i.getEnquesta()+"_"+i.getIndicador()+"_"+i.getAmbit());

src/main/java/es/uv/saic/web/PlantillaController.java

@@ -1,9 +1,15 @@
 package es.uv.saic.web;
 
+import java.io.IOException;
 import java.util.List;
 
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.DeleteMapping;
+import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.PostMapping;
+import org.springframework.web.bind.annotation.RequestBody;
+import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
 import es.uv.saic.domain.Plantilla;
@@ -11,13 +17,6 @@ import es.uv.saic.dto.PdfDTO;
 import es.uv.saic.dto.TemplateDataDTO;
 import es.uv.saic.service.PlantillaService;
 
-import org.springframework.web.bind.annotation.PathVariable;
-import org.springframework.web.bind.annotation.PostMapping;
-import org.springframework.web.bind.annotation.RequestBody;
-import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.DeleteMapping;
-import org.springframework.web.bind.annotation.GetMapping;
-
 
 @RestController
 @RequestMapping("/docs")
@@ -26,29 +25,13 @@ public class PlantillaController {
     private PlantillaService ps;
 
     @GetMapping("/plantilla")
-    public ResponseEntity<?> findAll() {
-        try {
-            List<Plantilla> plantillas = ps.findAll();
-
-             return (plantillas != null) ? ResponseEntity.ok(plantillas) : 
-                ResponseEntity.badRequest().body("No existen plantillas") ;
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("No se han podido recoger las plantillas: " + e.getMessage());
-        }
+    public List<Plantilla> findAll() {
+        return ps.findAll();
     }
 
     @GetMapping("/plantilla/{id}")
-    public ResponseEntity<?> findByID(@PathVariable Integer id) {
-        try {
-            Plantilla plantilla = ps.findByID(id);
-
-             return (plantilla != null) ? ResponseEntity.ok(plantilla) : 
-                ResponseEntity.badRequest().body("No existe la plantilla buscada") ;
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("No se ha podido recoger la plantilla " + e.getMessage());
-        }
+    public Plantilla findByID(@PathVariable Integer id) {
+        return ps.findByID(id);
     }
     
     @GetMapping("/plantilla/{versio}/{codi}/{ambit}")
@@ -66,74 +49,41 @@ public class PlantillaController {
     }
 
     @PostMapping("/plantilla/save")
-    public ResponseEntity<?> save(@RequestBody Plantilla plantilla) {
+    public String save(@RequestBody Plantilla plantilla) {
         try {
             ps.save(plantilla);
-            return ResponseEntity.ok("Save");
+            return "Save";
         } catch (Exception e) {
             e.printStackTrace();
-            return ResponseEntity.badRequest().body("Error guardando la plantilla: " + e.getMessage());
+            return "Error guardando la plantilla: " + e.getMessage();
         }
     }
     
     @DeleteMapping("/plantilla/delete")
-    public ResponseEntity<?> delete(@RequestBody Plantilla plantilla) {
-        try {
-            ps.delete(plantilla);
-            return ResponseEntity.ok("Delete");
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("Error borrando la plantilla: " + e.getMessage());
-        }
+    public String delete(@RequestBody Plantilla plantilla) {
+        ps.delete(plantilla);
+        return "Delete";
     }
     
     @GetMapping("/plantilla/used/{id}")
-    public ResponseEntity<?> isUsed(@PathVariable Integer id) {
-        try {
-            Boolean isUsed = ps.isUsed(id);
-
-             return (isUsed != null) ? ResponseEntity.ok(isUsed) : 
-                ResponseEntity.badRequest().body("No existe la plantilla que se esta buscando") ;
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("No se ha podido comprobar si la plantilla es usada: " + e.getMessage());
-        }
+    public boolean isUsed(@PathVariable Integer id) {
+       return ps.isUsed(id);
     }
 
     @PostMapping("/addTemplateData")
-    public ResponseEntity<?> addTemplateData(@RequestBody TemplateDataDTO td) {
-        try {
-            String template = ps.addTemplateData(td.getInstanciaTasca(), td.getTemplate());        
-            return (template != null) ? ResponseEntity.ok(template) : 
-                ResponseEntity.badRequest().body("La plantilla no se ha podido rellenar.") ; 
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("La plantilla no se ha podido rellenar: " + e.getMessage());
-        }
+    public String addTemplateData(@RequestBody TemplateDataDTO td) {
+        return ps.addTemplateData(td.getInstanciaTasca(), td.getTemplate());
     }
     
     @PostMapping("/addTemplateData2")
-    public ResponseEntity<?> addTemplateData2(@RequestBody TemplateDataDTO td) {
-        try {
-            String template = ps.addTemplateData(td.getIdTitulacio(), td.getIdCentre(), td.getCurs(), td.getTemplate());        
-            return (template != null) ? ResponseEntity.ok(template) : 
-                ResponseEntity.badRequest().body("La plantilla no se ha podido rellenar.") ; 
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("La plantilla no se ha podido rellenar: " + e.getMessage());
-        }
-    }
+    public String addTemplateData2(@RequestBody TemplateDataDTO td) {
+        return ps.addTemplateData(td.getIdTitulacio(), td.getIdCentre(), td.getCurs(), td.getTemplate());        
+}
 
     @PostMapping("/savePDF")
-    public ResponseEntity<?> savePDF(@RequestBody PdfDTO pdf) {
-        try {
-            String dst = ps.savePDF(pdf.getContent(), pdf.getIdtasca());        
-            return (dst != null) ? ResponseEntity.ok(dst) : 
-                ResponseEntity.badRequest().body("La plantilla no se ha podido rellenar.") ; 
-        } catch (Exception e) {
-            e.printStackTrace();
-            return ResponseEntity.badRequest().body("La plantilla no se ha podido rellenar: " + e.getMessage());
-        }
+    public String savePDF(@RequestBody PdfDTO pdf) throws IOException, InterruptedException {
+        return ps.savePDF(pdf.getContent(), pdf.getIdtasca());        
+
     }
     
 }