Se corrigen los servicios de ModelUzyTavDetComca

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Controller/UzyTavDetComcaController.java

 package ec.edu.espe.movilidad.MovilidadWS.Controller;
 
-import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavDetComca;
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavDetComca;
 import ec.edu.espe.movilidad.MovilidadWS.Service.UzyTavDetComca.IServiceUzyTavDetComca;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
 
+import javax.validation.Valid;
 import java.util.List;
 
 import static ec.edu.espe.movilidad.MovilidadWS.Constant.GlobalConstants.V1_API_VERSION;
@@ -15,46 +15,51 @@ import static ec.edu.espe.movilidad.MovilidadWS.Constant.GlobalConstants.V1_API_
 @CrossOrigin(origins = "*")
 @RequestMapping(V1_API_VERSION+"/detcomca")
 public class UzyTavDetComcaController {
-    @Autowired
-    private IServiceUzyTavDetComca serviceUzyTavDetComca;
+
+    private final  IServiceUzyTavDetComca serviceUzyTavDetComca;
+
+    public UzyTavDetComcaController(IServiceUzyTavDetComca serviceUzyTavDetComca) {
+        this.serviceUzyTavDetComca = serviceUzyTavDetComca;
+    }
 
     @GetMapping("/exampleFindId/{id}")
-    public ResponseEntity<ModelUzyTavDetComca> ListarPorID(@PathVariable Long id) {
+    public ResponseEntity<DtoUzyTavDetComca> ListarPorID(@PathVariable Long id) {
         return new ResponseEntity<>(serviceUzyTavDetComca.ListarPorID(id), HttpStatus.OK);
     }
 
     @GetMapping("/getAll")
-    public ResponseEntity<List<ModelUzyTavDetComca>> ListarRegistros() {
+    public ResponseEntity<List<DtoUzyTavDetComca>> ListarRegistros() {
         return new ResponseEntity<>(serviceUzyTavDetComca.ListarRegistros(), HttpStatus.OK);
     }
 
     @GetMapping("/datosRealacionadosconCabComca/{id}")
-    public ResponseEntity<List<ModelUzyTavDetComca>> findByIdconDatosCabComca(@PathVariable("id") Long id) {
-        List<ModelUzyTavDetComca> result = serviceUzyTavDetComca.findByIdconDatosCabComca(id);
-
-
+    public ResponseEntity<List<Object[]>> findByIdconDatosCabComca(@PathVariable("id") Long id) {
+        List<Object[]> result = serviceUzyTavDetComca.findByIdconDatosCabComca(id);
         return ResponseEntity.ok(result);
     }
 
-    @PostMapping("/guardar")
-    public ResponseEntity<ModelUzyTavDetComca> guardar(@RequestBody ModelUzyTavDetComca modelUzyTavDetComca, @RequestParam("uzytavcabcomca_id")Long uzytavcabcomca_id) {
-        ModelUzyTavDetComca creado= serviceUzyTavDetComca.guardar(uzytavcabcomca_id, modelUzyTavDetComca);
-
-        return ResponseEntity.status(HttpStatus.CREATED).body(creado);
 
+    @PostMapping("/guardar")
+    public ResponseEntity<DtoUzyTavDetComca> guardar(@Valid @RequestBody DtoUzyTavDetComca dtoUzyTavDetComca) {
+        DtoUzyTavDetComca savedDto = serviceUzyTavDetComca.guardar(dtoUzyTavDetComca);
+        return ResponseEntity.ok(savedDto);
     }
 
 
     @PutMapping("/editar/{id}")
-    public ResponseEntity<ModelUzyTavDetComca> editar(@PathVariable Long id, @RequestBody ModelUzyTavDetComca modelUzyTavDetComca) {
-        return new  ResponseEntity<>(serviceUzyTavDetComca.editar(id, modelUzyTavDetComca), HttpStatus.OK);
+    public ResponseEntity<DtoUzyTavDetComca> editar(@PathVariable Long id, @RequestBody DtoUzyTavDetComca dtoUzyTavDetComca) {
+        return new  ResponseEntity<>(serviceUzyTavDetComca.editar(id, dtoUzyTavDetComca), HttpStatus.OK);
     }
 
 
     @DeleteMapping("/eliminar/{id}")
-    public ResponseEntity<Void> eliminar(@PathVariable Long id) {
-        serviceUzyTavDetComca.eliminar(id);
-        return ResponseEntity.ok().build();
+    public ResponseEntity<String> eliminar(@PathVariable Long id){
+        boolean eliminado = serviceUzyTavDetComca.eliminar(id);
+        if (eliminado) {
+            return ResponseEntity.ok("El registro se eliminó exitosamente.");
+        } else {
+            return ResponseEntity.notFound().build();
+        }
     }
 
 }

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

@@ -12,9 +12,9 @@ import java.util.List;
 public interface DaoUzyTavDetComca  extends JpaRepository<ModelUzyTavDetComca, Long> {
 
 
-    //Obtener los datos relacionados con la tabla CabComca mediante un id
+    //Obtener los datos relacionados con la tabla CabComca mediante un ID
     @Query("SELECT p FROM ModelUzyTavDetComca p INNER JOIN p.uzytavcabcomca c WHERE c.uzytavcabcomca_id = :id")
-    List<ModelUzyTavDetComca> findByIdconDatosCabComca(@Param("id") Long id);
+    List<Object[]> findByIdconDatosCabComca(@Param("id") Long id);
 
 
 }

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

+package ec.edu.espe.movilidad.MovilidadWS.Dto;
+import lombok.Data;
+import java.util.Date;
+
+@Data
+public class DtoUzyTavDetComca {
+
+    private Long uzytavdetcomca_id;
+    private Long uzytusuario_id_crea;
+    private Long uzytusuario_id_asignado;
+    private Date uzytavdetcomca_fech_crea;
+    private String uzytavdetcomca_estado;
+    private Long uzytavcabcomca_id;
+
+
+}

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

+package ec.edu.espe.movilidad.MovilidadWS.Mapper.Components_Class;
+
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavDetComca;
+import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavCabComca;
+import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavDetComca;
+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 UzyTavDetComcaMapper {
+
+    private final ModelMapper modelMapper;
+    private final EntityManager entityManager;
+
+    public UzyTavDetComcaMapper(ModelMapper modelMapper, EntityManager entityManager) {
+        this.modelMapper = modelMapper;
+        this.entityManager = entityManager;
+        configureMappings();
+    }
+
+    public DtoUzyTavDetComca entityToDto(ModelUzyTavDetComca entity) {
+        return modelMapper.map(entity, DtoUzyTavDetComca.class);
+    }
+
+    public ModelUzyTavDetComca dtoToEntity(DtoUzyTavDetComca dto) {
+
+        ModelUzyTavDetComca entity = modelMapper.map(dto, ModelUzyTavDetComca.class);
+        Long cabcomcaID = dto.getUzytavcabcomca_id();
+        ModelUzyTavCabComca cabComca = entityManager.find(ModelUzyTavCabComca.class, cabcomcaID);
+        entity.setUzytavcabcomca(cabComca);
+        return entity;
+    }
+
+    public List<DtoUzyTavDetComca> entitiesToDtos(List<ModelUzyTavDetComca> entities) {
+        return entities.stream()
+                .map(this::entityToDto)
+                .collect(Collectors.toList());
+    }
+
+    private void configureMappings() {
+        modelMapper.addMappings(new PropertyMap<ModelUzyTavDetComca, DtoUzyTavDetComca>() {
+            @Override
+            protected void configure() {
+                map().setUzytavdetcomca_id(source.getUzytavdetcomca_id());
+                map().setUzytavcabcomca_id(source.getUzytavcabcomca().getUzytavcabcomca_id());
+            }
+        });
+    }
+
+
+}

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

@@ -2,13 +2,15 @@ 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.*;
 import java.util.Date;
 import java.util.Set;
 
-@Data
+@Getter
+@Setter
 @Entity
 @Table(name = "uzytavdetcomca", schema = "UTIC1")
 public class ModelUzyTavDetComca {
@@ -18,8 +20,6 @@ public class ModelUzyTavDetComca {
     @Column(name = "uzytavdetcomca_id")
     private Long uzytavdetcomca_id;
 
-
-
     @Column(name = "uzytusuario_id_crea")
     private Long uzytusuario_id_crea;
 
@@ -39,7 +39,6 @@ public class ModelUzyTavDetComca {
     private ModelUzyTavCabComca uzytavcabcomca;
 
     //RELACIÓN CON LA TABLA uzytavcabevalp- TABLA HIJA
-
     @JsonIgnore
     @OneToMany(mappedBy = "uzytavdetcomca", cascade = CascadeType.ALL)
     private Set<ModelUzyTavCabeValp> modelUzyTavCabeValpSet;

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavDetComca/IServiceUzyTavDetComca.java

 package ec.edu.espe.movilidad.MovilidadWS.Service.UzyTavDetComca;
 
-import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavDetComca;
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavDetComca;
 import org.springframework.data.repository.query.Param;
 
 import java.util.List;
 
 public interface IServiceUzyTavDetComca {
 
-    public ModelUzyTavDetComca ListarPorID(Long id);
+    DtoUzyTavDetComca ListarPorID(Long id);
 
-    public List<ModelUzyTavDetComca> ListarRegistros();
+    List<DtoUzyTavDetComca> ListarRegistros();
 
-    public ModelUzyTavDetComca guardar(Long uzytavcabcomca_id, ModelUzyTavDetComca modelUzyTavDetComca);
+    DtoUzyTavDetComca guardar(DtoUzyTavDetComca dtoUzyTavDetComca);
 
-    List<ModelUzyTavDetComca> findByIdconDatosCabComca(@Param("id") Long id);
+    List<Object[]> findByIdconDatosCabComca(@Param("id") Long id);
 
-    public ModelUzyTavDetComca editar(Long id, ModelUzyTavDetComca modelUzyTavDetComca);
+    DtoUzyTavDetComca editar(Long id, DtoUzyTavDetComca dtoUzyTavDetComca);
 
-    void eliminar(Long id);
+    boolean eliminar(Long id);
 
 }

src/main/java/ec/edu/espe/movilidad/MovilidadWS/Service/UzyTavDetComca/ServiceUzyTavDetComca.java

 package ec.edu.espe.movilidad.MovilidadWS.Service.UzyTavDetComca;
 
-import ec.edu.espe.movilidad.MovilidadWS.Dao.DaoUzyTavCabComca;
 import ec.edu.espe.movilidad.MovilidadWS.Dao.DaoUzyTavDetComca;
-import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavCabComca;
+import ec.edu.espe.movilidad.MovilidadWS.Dto.DtoUzyTavDetComca;
+import ec.edu.espe.movilidad.MovilidadWS.Exceptions.ResourceNotFoundException;
+import ec.edu.espe.movilidad.MovilidadWS.Mapper.Components_Class.UzyTavDetComcaMapper;
 import ec.edu.espe.movilidad.MovilidadWS.Model.ModelUzyTavDetComca;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.web.bind.annotation.PathVariable;
 
-import javax.persistence.EntityNotFoundException;
 import java.util.List;
 
 @Service
 public class ServiceUzyTavDetComca implements IServiceUzyTavDetComca{
-    @Autowired
-    DaoUzyTavDetComca daoUzyTavDetComca;
-    @Autowired
-    DaoUzyTavCabComca daoUzyTavCabComca;
+
+    private final DaoUzyTavDetComca daoUzyTavDetComca;
+    private final UzyTavDetComcaMapper mapper;
+
+    public ServiceUzyTavDetComca(DaoUzyTavDetComca daoUzyTavDetComca, UzyTavDetComcaMapper mapper) {
+        this.daoUzyTavDetComca = daoUzyTavDetComca;
+        this.mapper = mapper;
+    }
+
 
     @Override
-    public ModelUzyTavDetComca ListarPorID(@PathVariable Long id) {
-        return daoUzyTavDetComca.findById(id).get();
+    public DtoUzyTavDetComca ListarPorID(@PathVariable Long id) {
+        ModelUzyTavDetComca entity = daoUzyTavDetComca.findById(id)
+                .orElseThrow(() -> new ResourceNotFoundException("No se encontró el registro con ID: " + id));
+        return mapper.entityToDto(entity);
     }
 
     @Override
-    public List<ModelUzyTavDetComca> ListarRegistros() {
-        return  daoUzyTavDetComca.findAll();
+    public List<DtoUzyTavDetComca> ListarRegistros() {
+        List<ModelUzyTavDetComca> entities = daoUzyTavDetComca.findAll();
+        return mapper.entitiesToDtos(entities);
     }
 
     @Override
-    public ModelUzyTavDetComca guardar(Long uzytavcabcomca_id, ModelUzyTavDetComca modelUzyTavDetComca) {
-        ModelUzyTavCabComca modelUzyTavCabComca = daoUzyTavCabComca.findById(uzytavcabcomca_id)
-                .orElseThrow(() -> new EntityNotFoundException("No se encontró la entidad ModelUzyTavCabComca con el ID proporcionado"));
-        modelUzyTavDetComca.setUzytavcabcomca(modelUzyTavCabComca);
-        return daoUzyTavDetComca.save(modelUzyTavDetComca);
+    public DtoUzyTavDetComca guardar(DtoUzyTavDetComca dtoUzyTavDetComca) {
+        ModelUzyTavDetComca entity = mapper.dtoToEntity(dtoUzyTavDetComca);
+        ModelUzyTavDetComca nuevoEntity = daoUzyTavDetComca.save(entity);
+        return mapper.entityToDto(nuevoEntity);
     }
 
     @Override
-    public List<ModelUzyTavDetComca> findByIdconDatosCabComca(Long id) {
+    public List<Object[]> findByIdconDatosCabComca(Long id) {
         return daoUzyTavDetComca.findByIdconDatosCabComca(id);
     }
 
     @Override
-    public ModelUzyTavDetComca editar(@PathVariable Long id, ModelUzyTavDetComca modelUzyTavDetComca) {
-        ModelUzyTavDetComca dato = daoUzyTavDetComca.findById(id).get();
-        //.orElseThrow(()->new ControlExcepciones("No existe el registro con el ID : " + id));
-        //Seteamos los nuevos datos del registro
+    public DtoUzyTavDetComca editar(@PathVariable Long id, DtoUzyTavDetComca dtoUzyTavDetComca) {
+        try {
+        ModelUzyTavDetComca entity = daoUzyTavDetComca.findById(id).get();
+            if (entity != null) {
 
-        dato.setUzytavdetcomca_id(modelUzyTavDetComca.getUzytavdetcomca_id());
-        dato.setUzytusuario_id_crea(modelUzyTavDetComca.getUzytusuario_id_crea());
-        dato.setUzytusuario_id_asignado(modelUzyTavDetComca.getUzytusuario_id_asignado());
-        dato.setUzytavdetcomca_fech_crea(modelUzyTavDetComca.getUzytavdetcomca_fech_crea());
-        dato.setUzytavdetcomca_estado(modelUzyTavDetComca.getUzytavdetcomca_estado());
 
-        ModelUzyTavDetComca datoActualizado = daoUzyTavDetComca.save(dato);
-        return datoActualizado;
+                entity.setUzytusuario_id_crea(dtoUzyTavDetComca.getUzytusuario_id_crea());
+                entity.setUzytusuario_id_asignado(dtoUzyTavDetComca.getUzytusuario_id_asignado());
+                entity.setUzytavdetcomca_fech_crea(dtoUzyTavDetComca.getUzytavdetcomca_fech_crea());
+                entity.setUzytavdetcomca_estado(dtoUzyTavDetComca.getUzytavdetcomca_estado());
+                ModelUzyTavDetComca updatedEntity = daoUzyTavDetComca.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) {
-        ModelUzyTavDetComca dato = daoUzyTavDetComca.findById(id).get();
-        //.orElseThrow(() -> new ControlExcepciones("No existe el registro con el ID : " + id));
-        daoUzyTavDetComca.delete(dato);
+    public boolean eliminar(Long id) {
+        ModelUzyTavDetComca entity = daoUzyTavDetComca.findById(id).orElse(null);
+        if (entity != null) {
+            daoUzyTavDetComca.delete(entity);
+            return true;
+        }
+        return false;
     }
 }