Se modifican los servicios de ModelUzyTavEvalProy y se hacen algunos cambios en…

Se modifican los servicios de ModelUzyTavEvalProy y se hacen algunos cambios en los servicios de ServiceUzyTaValorPar, ServiceUzyTavConparaEva, ServiceUzyTavOpcionInstitucion.java

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Controller/UzyTavEvalProy.java → src/main/java/ec/edu/espe/movilidad/MovilidadWS/Controller/UzyTavEvalProyController.java

 package ec.edu.espe.movilidad.MovilidadWS.Controller;
 
-import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavEvalProy;
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavEvalProy;
 import ec.edu.espe.movilidad.MovilidadWS.Service.UzyTavEvalProy.IServiceUzyTavEvalProy;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
@@ -14,31 +13,35 @@ import static ec.edu.espe.movilidad.MovilidadWS.Constant.GlobalConstants.V1_API_
 @RestController
 @CrossOrigin(origins = "*")
 @RequestMapping(V1_API_VERSION+"/evalproy")
-public class UzyTavEvalProy {
+public class UzyTavEvalProyController {
 
-    @Autowired
-    IServiceUzyTavEvalProy serviceUzyTavEvalProy;
+
+    private final IServiceUzyTavEvalProy serviceUzyTavEvalProy;
+
+    public UzyTavEvalProyController(IServiceUzyTavEvalProy serviceUzyTavEvalProy) {
+        this.serviceUzyTavEvalProy = serviceUzyTavEvalProy;
+    }
 
     @GetMapping("/exampleFindId/{id}")
-    public ResponseEntity<ModelUzyTavEvalProy> ListarPorID(@PathVariable Long id) {
+    public ResponseEntity<DtoUzyTavEvalProy> ListarPorID(@PathVariable Long id) {
         return new ResponseEntity<>(serviceUzyTavEvalProy.ListarPorID(id), HttpStatus.OK);
     }
 
     @GetMapping("/getAll")
-    public ResponseEntity<List<ModelUzyTavEvalProy>> ListarRegistros() {
+    public ResponseEntity<List<DtoUzyTavEvalProy>> ListarRegistros() {
         return new ResponseEntity<>(serviceUzyTavEvalProy.ListarRegistros(), HttpStatus.OK);
     }
 
 
     @PostMapping("/guardar")
-    public ResponseEntity<ModelUzyTavEvalProy> guardar(@RequestBody ModelUzyTavEvalProy modelUzyTavEvalProy) {
-        return new  ResponseEntity<>(serviceUzyTavEvalProy.guardar(modelUzyTavEvalProy), HttpStatus.OK);
+    public ResponseEntity<DtoUzyTavEvalProy> guardar(@RequestBody DtoUzyTavEvalProy dtoUzyTavEvalProy) {
+        return new  ResponseEntity<>(serviceUzyTavEvalProy.guardar(dtoUzyTavEvalProy), HttpStatus.OK);
     }
 
 
     @PutMapping("/editar/{id}")
-    public ResponseEntity<ModelUzyTavEvalProy> editar(@PathVariable Long id, @RequestBody ModelUzyTavEvalProy modelUzyTavEvalProy) {
-        return new  ResponseEntity<>(serviceUzyTavEvalProy.editar(id, modelUzyTavEvalProy), HttpStatus.OK);
+    public ResponseEntity<DtoUzyTavEvalProy> editar(@PathVariable Long id, @RequestBody DtoUzyTavEvalProy dtoUzyTavEvalProy) {
+        return new  ResponseEntity<>(serviceUzyTavEvalProy.editar(id, dtoUzyTavEvalProy), HttpStatus.OK);
     }
 
 

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Dao/DaoUzyTavEvalProy.java

@@ -2,7 +2,9 @@ package ec.edu.espe.movilidad.MovilidadWS.Dao;
 
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavEvalProy;
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.stereotype.Repository;
 
+@Repository
 public interface DaoUzyTavEvalProy extends JpaRepository<ModelUzyTavEvalProy, Long> {
 
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Dto/DtoUzyTavEvalProy.java

 package ec.edu.espe.movilidad.MovilidadWS.Dto;
 
 import lombok.Data;
-import org.springframework.web.multipart.MultipartFile;
 
-import java.util.Date;
 
 @Data
 public class DtoUzyTavEvalProy {
@@ -11,7 +9,7 @@ public class DtoUzyTavEvalProy {
     private Long uzytavevalproy_id;
     private Long uzytavcabevalp_id;
     private Long uzytavaloruzytparametros_id;
-    private Long uzytavevalproy_puntaje;
+    private Integer uzytavevalproy_puntaje;
     private Long uzytavparaeva_id;
     private String uzytavevalproy_observ;
 

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/UzyTMenuMapper.java

@@ -5,6 +5,10 @@ import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTMenu;
 import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
 import org.modelmapper.ModelMapper;
+
+import java.util.List;
+import java.util.stream.Collectors;
+
 @Component
 public class UzyTMenuMapper {
 
@@ -22,6 +26,12 @@ public class UzyTMenuMapper {
     public ModelUzyTMenu dtoToEntity(DtoUzyTMenu dto) {
         return modelMapper.map(dto, ModelUzyTMenu.class);
     }
+
+    public List<DtoUzyTMenu> entitiesToDtos(List<ModelUzyTMenu> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
     private void configureMappings() {
         modelMapper.addMappings(new PropertyMap<ModelUzyTMenu, DtoUzyTMenu>() {
             @Override

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/UzyTaValorParMapper.java

@@ -8,6 +8,8 @@ import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
 
 import javax.persistence.EntityManager;
+import java.util.List;
+import java.util.stream.Collectors;
 
 @Component
 public class UzyTaValorParMapper {
@@ -24,6 +26,12 @@ public class UzyTaValorParMapper {
         return modelMapper.map(entity, DtoUzyTaValorPar.class);
     }
 
+    public List<DtoUzyTaValorPar> entitiesToDtos(List<ModelUzyTaValorPar> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
+
     public ModelUzyTaValorPar dtoToEntity(DtoUzyTaValorPar dto) {
         ModelUzyTaValorPar entity = modelMapper.map(dto, ModelUzyTaValorPar.class);
         Long paraevaId = dto.getUzytavparaeva_id();

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/UzyTavConparaEvaMapper.java

@@ -3,14 +3,19 @@ package ec.edu.espe.movilidad.MovilidadWS.Mapper.Components_Class;
 import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavConparaEva;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavConparaEva;
 import org.modelmapper.ModelMapper;
+import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
 
+import java.util.List;
+import java.util.stream.Collectors;
+
 @Component
 public class UzyTavConparaEvaMapper {
     private final ModelMapper modelMapper;
 
     public UzyTavConparaEvaMapper(ModelMapper modelMapper) {
         this.modelMapper = modelMapper;
+        configureMappings();
     }
 
     public DtoUzyTavConparaEva entityToDto(ModelUzyTavConparaEva entity) {
@@ -20,4 +25,20 @@ public class UzyTavConparaEvaMapper {
     public ModelUzyTavConparaEva dtoToEntity(DtoUzyTavConparaEva dto) {
         return modelMapper.map(dto, ModelUzyTavConparaEva.class);
     }
+
+
+    public List<DtoUzyTavConparaEva> entitiesToDtos(List<ModelUzyTavConparaEva> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
+
+    private void configureMappings() {
+        modelMapper.addMappings(new PropertyMap<ModelUzyTavConparaEva, DtoUzyTavConparaEva>() {
+            @Override
+            protected void configure() {
+                map().setUzytavconparaeva_id(source.getUzytavconparaeva_id());
+            }
+        });
+    }
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/UzyTavEvalProyMapper.java

 package ec.edu.espe.movilidad.MovilidadWS.Mapper.Components_Class;
 
 import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavEvalProy;
+import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTaValorPar;
+import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavCabeValp;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavEvalProy;
 import org.modelmapper.ModelMapper;
+import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
 
+import javax.persistence.EntityManager;
+import java.util.List;
+import java.util.stream.Collectors;
+
 @Component
 public class UzyTavEvalProyMapper {
     private final ModelMapper modelMapper;
+    private final EntityManager entityManager;
 
-    public UzyTavEvalProyMapper(ModelMapper modelMapper) {
+    public UzyTavEvalProyMapper(ModelMapper modelMapper, EntityManager entityManager) {
         this.modelMapper = modelMapper;
+        this.entityManager = entityManager;
+        configureMappings();
     }
 
     public DtoUzyTavEvalProy entityToDto(ModelUzyTavEvalProy entity) {
@@ -18,6 +28,31 @@ public class UzyTavEvalProyMapper {
     }
 
     public ModelUzyTavEvalProy dtoToEntity(DtoUzyTavEvalProy dto) {
-        return modelMapper.map(dto, ModelUzyTavEvalProy.class);
+
+        ModelUzyTavEvalProy entity = modelMapper.map(dto, ModelUzyTavEvalProy.class);
+        Long cabevalpID = dto.getUzytavcabevalp_id();
+        Long valorparID = dto.getUzytavaloruzytparametros_id();
+        ModelUzyTavCabeValp cabeval = entityManager.find(ModelUzyTavCabeValp.class, cabevalpID);
+        ModelUzyTaValorPar valorPar = entityManager.find(ModelUzyTaValorPar.class, valorparID);
+        entity.setUzytavcabevalp(cabeval);
+        entity.setUzytavalorpar(valorPar);
+        return entity;
+    }
+
+    public List<DtoUzyTavEvalProy> entitiesToDtos(List<ModelUzyTavEvalProy> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
+
+    private void configureMappings() {
+        modelMapper.addMappings(new PropertyMap<ModelUzyTavEvalProy, DtoUzyTavEvalProy>() {
+            @Override
+            protected void configure() {
+                map().setUzytavevalproy_id(source.getUzytavevalproy_id());
+                map().setUzytavcabevalp_id(source.getUzytavcabevalp().getUzytavcabevalp_id());
+                map().setUzytavaloruzytparametros_id(source.getUzytavalorpar().getUzytavaloruzytparametros_id());
+            }
+        });
     }
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/UzyTavOpcionInstitucionMapper.java

 package ec.edu.espe.movilidad.MovilidadWS.Mapper.Components_Class;
-
 import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavOpcionInstitucion;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTProvincia;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavOpcionInstitucion;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavPrograma_V;
 import org.modelmapper.ModelMapper;
+import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
-
 import javax.persistence.EntityManager;
+import java.util.List;
+import java.util.stream.Collectors;
 
 @Component
 public class UzyTavOpcionInstitucionMapper {
@@ -17,11 +18,9 @@ public class UzyTavOpcionInstitucionMapper {
     public UzyTavOpcionInstitucionMapper(ModelMapper modelMapper, EntityManager entityManager) {
         this.modelMapper = modelMapper;
         this.entityManager = entityManager;
+        configureMappings();
     }
 
-
-
-
     public DtoUzyTavOpcionInstitucion entityToDto(ModelUzyTavOpcionInstitucion entity) {
         return modelMapper.map(entity, DtoUzyTavOpcionInstitucion.class);
     }
@@ -41,4 +40,21 @@ public class UzyTavOpcionInstitucionMapper {
         return entity;
     }
 
+    public List<DtoUzyTavOpcionInstitucion> entitiesToDtos(List<ModelUzyTavOpcionInstitucion> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
+
+    private void configureMappings() {
+        modelMapper.addMappings(new PropertyMap<ModelUzyTavOpcionInstitucion, DtoUzyTavOpcionInstitucion>() {
+            @Override
+            protected void configure() {
+                map().setUzytavopcion_institucion_id(source.getUzytavopcion_institucion_id());
+                map().setUzytavprograma_v_id(source.getUzytavprograma_v().getUzytavprograma_v_id());
+                map().setUzytprovincia_id(source.getUzytprovincia().getUzytprovincia_id());
+            }
+        });
+    }
+
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/UzyTavParaEvaMapper.java

@@ -8,6 +8,8 @@ import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
 
 import javax.persistence.EntityManager;
+import java.util.List;
+import java.util.stream.Collectors;
 
 @Component
 public class UzyTavParaEvaMapper {
@@ -30,7 +32,11 @@ public class UzyTavParaEvaMapper {
         entity.setUzytavconparaeva(lineaOperativa);
         return entity;
     }
-
+    public List<DtoUzyTavParaEva> entitiesToDtos(List<ModelUzyTavParaEva> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
 
     private void configureMappings() {
         modelMapper.addMappings(new PropertyMap<ModelUzyTavParaEva, DtoUzyTavParaEva>() {

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Mapper/Components_Class/Uzytavprograma_VMapper.java

@@ -6,6 +6,8 @@ import org.modelmapper.ModelMapper;
 import org.modelmapper.PropertyMap;
 import org.springframework.stereotype.Component;
 import javax.persistence.EntityManager;
+import java.util.List;
+import java.util.stream.Collectors;
 
 
 @Component
@@ -30,7 +32,6 @@ public class Uzytavprograma_VMapper {
     }
     //Convierte entidad a dto.
     public DtoUzyTavPrograma_V entityToDto(ModelUzyTavPrograma_V entity) {
-
         return modelMapper.map(entity, DtoUzyTavPrograma_V.class);
     }
     // Convierte de DTO a Entidad
@@ -44,4 +45,10 @@ public class Uzytavprograma_VMapper {
         return entity;
     }
 
+    public List<DtoUzyTavPrograma_V> entitiesToDtos(List<ModelUzyTavPrograma_V> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
+
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Model/ModelUzyTavEvalProy.java

 package ec.edu.espe.movilidad.MovilidadWS.Model;
 
-
-import com.fasterxml.jackson.annotation.JsonIgnore;
-import lombok.Data;
+import lombok.Getter;
+import lombok.Setter;
 
 import javax.persistence.*;
 
-@Data
+@Getter
+@Setter
 @Entity
 @Table(name = "uzytavevalproy", schema = "UTIC1")
 public class ModelUzyTavEvalProy {
@@ -22,22 +22,19 @@ public class ModelUzyTavEvalProy {
 
 
     @Column(name = "uzytavparaeva_id")
-    private Integer uzytavparaeva_id;
+    private Long uzytavparaeva_id;
 
     @Column(name = "uzytavevalproy_observ")
     private String uzytavevalproy_observ;
 
     //RELACION CON CABEVALP-TABLA PADRE
-    @JsonIgnore
-    @ManyToOne( cascade = CascadeType.ALL)
+    @ManyToOne
     @JoinColumn(name = "uzytavcabevalp_id", referencedColumnName = "uzytavcabevalp_id")
     private ModelUzyTavCabeValp uzytavcabevalp;
 
 
     //RELACIÓN CON PARAMETROS_ID- TABLA PADRE
-
-    @JsonIgnore
-    @ManyToOne( cascade = CascadeType.ALL)
+    @ManyToOne
     @JoinColumn(name = "uzytavaloruzytparametros_id", referencedColumnName = "uzytavaloruzytparametros_id")
     private ModelUzyTaValorPar uzytavalorpar;
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTMenu/ServiceUzyTMenu.java

@@ -9,7 +9,6 @@ import org.springframework.stereotype.Service;
 
 import java.util.List;
 import java.util.Optional;
-import java.util.stream.Collectors;
 
 @Service
 public class ServiceUzyTMenu implements IServiceUzyTMenu {
@@ -35,9 +34,7 @@ public class ServiceUzyTMenu implements IServiceUzyTMenu {
     @Override
     public List<DtoUzyTMenu> ListarRegistros() {
         List<ModelUzyTMenu> entities = daoUzyTMenu.findAll();
-        return entities.stream()
-                .map(uzyTMenuMapper::entityToDto)
-                .collect(Collectors.toList());
+        return uzyTMenuMapper.entitiesToDtos(entities);
     }
 
     @Override

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTaValorPar/ServiceUzyTaValorPar.java

@@ -8,7 +8,6 @@ import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTaValorPar;
 import org.springframework.stereotype.Service;
 import org.springframework.web.bind.annotation.PathVariable;
 import java.util.List;
-import java.util.stream.Collectors;
 
 @Service
 public class ServiceUzyTaValorPar implements IServiceUzyTaValorPar {
@@ -46,9 +45,7 @@ public class ServiceUzyTaValorPar implements IServiceUzyTaValorPar {
     public List<DtoUzyTaValorPar> ListarRegistros() {
 
         List<ModelUzyTaValorPar> entities = daoUzyTaValorPar.findAll();
-        return entities.stream()
-                .map(mapper::entityToDto)
-                .collect(Collectors.toList());
+        return mapper.entitiesToDtos(entities);
     }
 
 

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavConparaEva/ServiceUzyTavConparaEva.java

@@ -66,9 +66,7 @@ public class ServiceUzyTavConparaEva implements IServiceUzyTavConparaEva {
     public List<DtoUzyTavConparaEva> ListarRegistrosParametrosFinales() {
 
         List<ModelUzyTavConparaEva> entities = daoUzyTavConparaEva.noActivo().orElse(new ArrayList<>());
-        return entities.stream()
-                .map(mapper::entityToDto)
-                .collect(Collectors.toList());
+        return mapper.entitiesToDtos(entities);
     }
 
 

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavEvalProy/IServiceUzyTavEvalProy.java

 package ec.edu.espe.movilidad.MovilidadWS.Service.UzyTavEvalProy;
 
-import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavEvalProy;
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavEvalProy;
 
 import java.util.List;
 
 public interface IServiceUzyTavEvalProy {
 
-    public ModelUzyTavEvalProy ListarPorID(Long id);
+    DtoUzyTavEvalProy ListarPorID(Long id);
 
-    public List<ModelUzyTavEvalProy> ListarRegistros();
+    List<DtoUzyTavEvalProy> ListarRegistros();
 
-    public ModelUzyTavEvalProy guardar(ModelUzyTavEvalProy modelUzyTavEvalProy);
+    DtoUzyTavEvalProy guardar(DtoUzyTavEvalProy dtoUzyTavEvalProy);
 
+    DtoUzyTavEvalProy editar(Long id, DtoUzyTavEvalProy dtoUzyTavEvalProy);
 
-
-    public ModelUzyTavEvalProy editar(Long id, ModelUzyTavEvalProy modelUzyTavEvalProy);
-
-    void eliminar(Long id);
+    boolean eliminar(Long id);
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavEvalProy/ServiceUzyTavEvalProy.java

 package ec.edu.espe.movilidad.MovilidadWS.Service.UzyTavEvalProy;
 
 import ec.edu.espe.movilidad.MovilidadWS.Dao.DaoUzyTavEvalProy;
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavEvalProy;
+import ec.edu.espe.movilidad.MovilidadWS.Exceptions.ResourceNotFoundException;
+import ec.edu.espe.movilidad.MovilidadWS.Mapper.Components_Class.UzyTavEvalProyMapper;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavEvalProy;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.web.bind.annotation.PathVariable;
 
@@ -10,46 +12,66 @@ import java.util.List;
 
 @Service
 public class ServiceUzyTavEvalProy implements IServiceUzyTavEvalProy{
-    @Autowired
-    DaoUzyTavEvalProy daoUzyTavEvalProy;
 
-    @Override
-    public ModelUzyTavEvalProy ListarPorID(@PathVariable  Long id) {
-        return daoUzyTavEvalProy.findById(id).get();
+    private final DaoUzyTavEvalProy daoUzyTavEvalProy;
+
+    private final UzyTavEvalProyMapper mapper;
+
+    public ServiceUzyTavEvalProy(DaoUzyTavEvalProy daoUzyTavEvalProy, UzyTavEvalProyMapper mapper) {
+        this.daoUzyTavEvalProy = daoUzyTavEvalProy;
+        this.mapper = mapper;
     }
 
     @Override
-    public List<ModelUzyTavEvalProy> ListarRegistros() {
-        return  daoUzyTavEvalProy.findAll();
+    public DtoUzyTavEvalProy ListarPorID(@PathVariable  Long id) {
+        ModelUzyTavEvalProy entity = daoUzyTavEvalProy.findById(id)
+                .orElseThrow(() -> new ResourceNotFoundException("No se encontró el registro con ID: " + id));
+        return mapper.entityToDto(entity);
+    }
 
+    @Override
+    public List<DtoUzyTavEvalProy> ListarRegistros() {
+        List<ModelUzyTavEvalProy> entities = daoUzyTavEvalProy.findAll();
+        return mapper.entitiesToDtos(entities);
     }
 
     @Override
-    public ModelUzyTavEvalProy guardar(ModelUzyTavEvalProy modelUzyTavEvalProy) {
-        return daoUzyTavEvalProy.save(modelUzyTavEvalProy);
+    public DtoUzyTavEvalProy guardar(DtoUzyTavEvalProy dtoUzyTavEvalProy) {
+        ModelUzyTavEvalProy entity = mapper.dtoToEntity(dtoUzyTavEvalProy);
+        ModelUzyTavEvalProy nuevoEntity = daoUzyTavEvalProy.save(entity);
+        return mapper.entityToDto(nuevoEntity);
 
     }
 
 
     @Override
-    public ModelUzyTavEvalProy editar(@PathVariable Long id, ModelUzyTavEvalProy modelUzyTavEvalProy) {
-        ModelUzyTavEvalProy dato = daoUzyTavEvalProy.findById(id).get();
-        //.orElseThrow(()->new ControlExcepciones("No existe el registro con el ID : " + id));
+    public DtoUzyTavEvalProy editar(@PathVariable Long id, DtoUzyTavEvalProy dtoUzyTavEvalProy) {
+        try {
+        ModelUzyTavEvalProy entity = daoUzyTavEvalProy.findById(id).get();
         //Seteamos los nuevos datos del registro
-        dato.setUzytavevalproy_id(modelUzyTavEvalProy.getUzytavevalproy_id());
-        dato.setUzytavevalproy_puntaje(modelUzyTavEvalProy.getUzytavevalproy_puntaje());
-        dato.setUzytavparaeva_id(modelUzyTavEvalProy.getUzytavparaeva_id());
-        dato.setUzytavevalproy_observ(modelUzyTavEvalProy.getUzytavevalproy_observ());
+            if (entity != null) {
+                entity.setUzytavevalproy_id(dtoUzyTavEvalProy.getUzytavevalproy_id());
+                entity.setUzytavevalproy_puntaje(dtoUzyTavEvalProy.getUzytavevalproy_puntaje());
+                entity.setUzytavparaeva_id(dtoUzyTavEvalProy.getUzytavparaeva_id());
+                entity.setUzytavevalproy_observ(dtoUzyTavEvalProy.getUzytavevalproy_observ());
 
 
-        ModelUzyTavEvalProy datoActualizado = daoUzyTavEvalProy.save(dato);
-        return datoActualizado;
+                ModelUzyTavEvalProy updatedEntity = daoUzyTavEvalProy.save(entity);
+                return mapper.entityToDto(updatedEntity);
+            }else {
+                throw new ResourceNotFoundException("No se encontró el registro con ID: " + id);
+            }
+        } catch (Exception e) {
+            throw new ResourceNotFoundException("Error al editar el registro: " + e.getMessage());
+        }
     }
-
     @Override
-    public void eliminar(@PathVariable Long id) {
-        ModelUzyTavEvalProy dato = daoUzyTavEvalProy.findById(id).get();
-        //.orElseThrow(() -> new ControlExcepciones("No existe el registro con el ID : " + id));
-        daoUzyTavEvalProy.delete(dato);
+    public boolean eliminar(Long id) {
+        ModelUzyTavEvalProy entity = daoUzyTavEvalProy.findById(id).orElse(null);
+        if (entity != null) {
+            daoUzyTavEvalProy.delete(entity);
+            return true;
+        }
+        return false;
     }
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavOpcionInstitucion/ServiceUzyTavOpcionInstitucion.java

@@ -8,7 +8,7 @@ import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavOpcionInstitucion;
 import org.springframework.stereotype.Service;
 
 import java.util.List;
-import java.util.stream.Collectors;
+
 
 @Service
 public class ServiceUzyTavOpcionInstitucion implements IServiceUzyTavOpcionInstitucion{
@@ -33,9 +33,7 @@ public class ServiceUzyTavOpcionInstitucion implements IServiceUzyTavOpcionInsti
     @Override
     public List<DtoUzyTavOpcionInstitucion> ListarRegistros() {
         List<ModelUzyTavOpcionInstitucion> entities = daoUzyTavOpcionInstitucion.findAll();
-        return entities.stream()
-                .map(mapper::entityToDto)
-                .collect(Collectors.toList());
+        return mapper.entitiesToDtos(entities);
     }
 
     @Override

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavPrograma_V/ServiceUzyTavPrograma_V.java

@@ -9,7 +9,6 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
 import java.util.List;
-import java.util.stream.Collectors;
 
 @Service
 
@@ -34,11 +33,8 @@ public class ServiceUzyTavPrograma_V implements IServiceUzyTavPrograma_V{
 
     @Override
     public List<DtoUzyTavPrograma_V> ListarRegistros() {
-
         List<ModelUzyTavPrograma_V> entities = daoUzyTavProgramaV.findAll();
-        return entities.stream()
-                .map(mapper::entityToDto)
-                .collect(Collectors.toList());
+        return mapper.entitiesToDtos(entities);
     }
 
     @Override