Commit 503f16aa by Rubén Ramírez

fix: [BibliotecaPersonalController]: Pequeño cambio en la estructuración del código

parent 7d0a7880
...@@ -32,6 +32,7 @@ public class BibliotecaPersonalController { ...@@ -32,6 +32,7 @@ public class BibliotecaPersonalController {
/** /**
* Añade un recurso a la biblioteca personal del usuario. * Añade un recurso a la biblioteca personal del usuario.
*
* Devuelve: * Devuelve:
* - 201 CREATED si se agrega correctamente. * - 201 CREATED si se agrega correctamente.
* - 404 NOT FOUND si la biblioteca o el recurso no existen. * - 404 NOT FOUND si la biblioteca o el recurso no existen.
...@@ -68,9 +69,9 @@ public class BibliotecaPersonalController { ...@@ -68,9 +69,9 @@ public class BibliotecaPersonalController {
/** /**
* Lista los recursos de una biblioteca por categoría. * Lista los recursos de una biblioteca por categoría.
*
* Devuelve: * Devuelve:
* - 200 OK con la lista de recursos si existen. * - 200 OK con la lista de recursos (vacía o no).
* - 204 NO CONTENT si no hay recursos en la categoría.
* - 400 BAD REQUEST si la categoría es inválida. * - 400 BAD REQUEST si la categoría es inválida.
* - 404 NOT FOUND si la biblioteca no existe. * - 404 NOT FOUND si la biblioteca no existe.
* - 500 INTERNAL SERVER ERROR en caso de error inesperado. * - 500 INTERNAL SERVER ERROR en caso de error inesperado.
...@@ -79,9 +80,7 @@ public class BibliotecaPersonalController { ...@@ -79,9 +80,7 @@ public class BibliotecaPersonalController {
public ResponseEntity<List<DTORecursoEnBiblioteca>> listarRecursosPorCategoria( public ResponseEntity<List<DTORecursoEnBiblioteca>> listarRecursosPorCategoria(
@PathVariable Long usuarioId, @PathVariable String categoria) { @PathVariable Long usuarioId, @PathVariable String categoria) {
try { try {
BibliotecaPersonal biblioteca = servicioUsuarios.obtenerBibliotecaDeUsuario(usuarioId); BibliotecaPersonal biblioteca = servicioUsuarios.obtenerBibliotecaDeUsuario(usuarioId);
if (biblioteca == null) return ResponseEntity.status(HttpStatus.NOT_FOUND).build(); if (biblioteca == null) return ResponseEntity.status(HttpStatus.NOT_FOUND).build();
Categoria categoriaEnum; Categoria categoriaEnum;
...@@ -91,9 +90,8 @@ public class BibliotecaPersonalController { ...@@ -91,9 +90,8 @@ public class BibliotecaPersonalController {
return ResponseEntity.status(HttpStatus.BAD_REQUEST).build(); return ResponseEntity.status(HttpStatus.BAD_REQUEST).build();
} }
List<BibliotecaPersonalRecurso> recursos = servicioBibliotecaPersonal.listarPorCategoria(biblioteca.getId(), categoriaEnum); List<BibliotecaPersonalRecurso> recursos = servicioBibliotecaPersonal
.listarPorCategoria(biblioteca.getId(), categoriaEnum);
if (recursos.isEmpty()) return ResponseEntity.status(HttpStatus.NO_CONTENT).build();
List<DTORecursoEnBiblioteca> dtoRecursos = recursos.stream() List<DTORecursoEnBiblioteca> dtoRecursos = recursos.stream()
.map(bpr -> new DTORecursoEnBiblioteca( .map(bpr -> new DTORecursoEnBiblioteca(
...@@ -105,6 +103,7 @@ public class BibliotecaPersonalController { ...@@ -105,6 +103,7 @@ public class BibliotecaPersonalController {
: null : null
)) ))
.collect(Collectors.toList()); .collect(Collectors.toList());
return ResponseEntity.ok(dtoRecursos); return ResponseEntity.ok(dtoRecursos);
} catch (Exception e) { } catch (Exception e) {
return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).build(); return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).build();
...@@ -114,6 +113,7 @@ public class BibliotecaPersonalController { ...@@ -114,6 +113,7 @@ public class BibliotecaPersonalController {
/** /**
* Elimina un recurso de la biblioteca personal del usuario. * Elimina un recurso de la biblioteca personal del usuario.
* Devuelve: * Devuelve:
*
* - 204 NO CONTENT si se elimina correctamente. * - 204 NO CONTENT si se elimina correctamente.
* - 404 NOT FOUND si la biblioteca o el recurso no existen. * - 404 NOT FOUND si la biblioteca o el recurso no existen.
* - 500 INTERNAL SERVER ERROR en caso de error inesperado. * - 500 INTERNAL SERVER ERROR en caso de error inesperado.
...@@ -138,6 +138,7 @@ public class BibliotecaPersonalController { ...@@ -138,6 +138,7 @@ public class BibliotecaPersonalController {
/** /**
* Obtiene un recurso de la biblioteca personal del usuario. * Obtiene un recurso de la biblioteca personal del usuario.
*
* Devuelve: * Devuelve:
* - 200 OK con el recurso si existe. * - 200 OK con el recurso si existe.
* - 204 NO CONTENT si el recurso no está en la biblioteca. * - 204 NO CONTENT si el recurso no está en la biblioteca.
...@@ -172,6 +173,7 @@ public class BibliotecaPersonalController { ...@@ -172,6 +173,7 @@ public class BibliotecaPersonalController {
/** /**
* Cambia la categoría de un recurso en la biblioteca personal del usuario. * Cambia la categoría de un recurso en la biblioteca personal del usuario.
*
* Devuelve: * Devuelve:
* - 200 OK si se cambia correctamente. * - 200 OK si se cambia correctamente.
* - 400 BAD REQUEST si la categoría es nula. * - 400 BAD REQUEST si la categoría es nula.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment