From 595b0ed6b4d28ea434fa170ca32feb44aa1a9a93 Mon Sep 17 00:00:00 2001
From: Celestino Rey <creylopez@yahoo.es>
Date: Thu, 1 Aug 2024 10:26:23 +0200
Subject: [PATCH] Ajustes para mejorar la experiencia

---
 LibrosPy/servicios/libros/paginas.py          |  8 ++-
 .../servicios/libros/templates/autor.html     |  2 +-
 .../servicios/libros/templates/index.html     |  4 +-
 .../servicios/libros/templates/libros.html    | 59 -------------------
 .../{searchrautor.html => searchautor.html}   |  2 +-
 .../libros/templates/searchlibro.html         |  8 +--
 6 files changed, 14 insertions(+), 69 deletions(-)
 delete mode 100644 LibrosPy/servicios/libros/templates/libros.html
 rename LibrosPy/servicios/libros/templates/{searchrautor.html => searchautor.html} (90%)

diff --git a/LibrosPy/servicios/libros/paginas.py b/LibrosPy/servicios/libros/paginas.py
index 36ebf4c..aa7c5f8 100644
--- a/LibrosPy/servicios/libros/paginas.py
+++ b/LibrosPy/servicios/libros/paginas.py
@@ -195,17 +195,21 @@ def uploaded_file(filename):
 @bp.route('/searchautor')
 def searchautor():
     query = request.args.get('query', '')
+
     if query:
-        autores = Autores.query.filter(autor.name.contains(query)).all()
+        autores = Autores.query.filter(Autores.apellido.contains(query)).all()
     else:
         autores = []
+        
     return render_template('searchautor.html', query=query, autores=autores)
 
 @bp.route('/searchlibro')
 def searchlibro():
     query = request.args.get('query', '')
+
     if query:
-        libros = Libros.query.filter(libro.name.contains(query)).all()
+        libros = Libros.query.filter(Libros.titulo.contains(query)).all()
     else:
         libros = []
+
     return render_template('searchlibro.html', query=query, libros=libros)
diff --git a/LibrosPy/servicios/libros/templates/autor.html b/LibrosPy/servicios/libros/templates/autor.html
index d1b1262..8222ce8 100644
--- a/LibrosPy/servicios/libros/templates/autor.html
+++ b/LibrosPy/servicios/libros/templates/autor.html
@@ -40,7 +40,7 @@
                         <tr>
                             <td class="cell">{{ libro.id }}</td>
                             <td class="cell">{{ libro.anno }}</td>
-                            <td class="cell">{{ libro.titulo }}</td>
+                            <td class="cell"><a href="{{ url_for('paginas.libro', libro_id=libro.id) }}">{{ libro.titulo }}</a></td>
                             {% if libro.portada %}
                             <td class="cell"><img src="{{ url_for('paginas.uploaded_file', filename=libro.portada) }}" alt="{{ autor.apellido }}, {{ autor.nombre}}" style="width:200px;height:200px;"></td>
                             {% else %}
diff --git a/LibrosPy/servicios/libros/templates/index.html b/LibrosPy/servicios/libros/templates/index.html
index 89b94db..1149d1e 100644
--- a/LibrosPy/servicios/libros/templates/index.html
+++ b/LibrosPy/servicios/libros/templates/index.html
@@ -32,8 +32,8 @@
                                 <tr>
                                     <td class="cell">{{ libro.id }}</td>
                                     <td class="cell">{{ libro.anno }}</td>
-                                    <td class="cell">{{ libro.titulo }}</td>
-                                    <td class="cell">{{ libro.autores.apellido }}, {{ libro.autores.nombre }}</td>
+                                    <td class="cell"><a href="{{ url_for('paginas.libro', libro_id=libro.id) }}">{{ libro.titulo }}</a></td>
+                                    <td class="cell"><a href="{{ url_for('paginas.autor', autor_id=libro.autor_id) }}">{{ libro.autores.apellido }}, {{ libro.autores.nombre }}</a></td>
                                     {% if libro.portada %}
                                     <td class="cell"><img src="{{ url_for('paginas.uploaded_file', filename=libro.portada) }}" alt="{{ libro.titulo }}" style="width:200px;height:200px;"></td>
                                     {% else %}
diff --git a/LibrosPy/servicios/libros/templates/libros.html b/LibrosPy/servicios/libros/templates/libros.html
deleted file mode 100644
index c67b5e5..0000000
--- a/LibrosPy/servicios/libros/templates/libros.html
+++ /dev/null
@@ -1,59 +0,0 @@
-{% extends 'base.html' %}
-
-{% block content %}
-		    <div class="container-xl">
-			    
-			    <div class="row g-3 mb-4 align-items-center justify-content-between">
-				    <div class="col-auto">
-			            <h1 class="app-page-title mb-0">Libros</h1>
-				    </div>
-			    </div><!--//row-->
-			   
-			    
-			    <nav id="orders-table-tab" class="orders-table-tab app-nav-tabs nav shadow-sm flex-column flex-sm-row mb-4">
-				    <a class="flex-sm-fill text-sm-center nav-link active" id="libros-tab" data-bs-toggle="tab" href="#libros" role="tab" aria-controls="libros" aria-selected="true">Libros</a>
-				</nav>
-				
-				
-				<div class="tab-content" id="orders-table-tab-content">
-			        <div class="tab-pane fade show active" id="libros" role="tabpanel" aria-labelledby="libros-tab">
-					    <div class="app-card app-card-orders-table shadow-sm mb-5">
-						    <div class="app-card-body">
-							    <div class="table-responsive">
-							        <table class="table app-table-hover mb-0 text-left">
-										<thead>
-											<tr>
-												<th class="cell">#</th>
-												<th class="cell">anno</th>
-												<th class="cell">Título</th>
-												<th class="cell">Autor</th>
-												<th class="cell">Portada</th>
-											</tr>
-										</thead>
-										<tbody>
-											{% for libro in libros %}
-											<tr>
-												<td class="cell">{{ libro.id }}</td>
-												<td class="cell">{{ libro.anno }}</td>
-												<td class="cell">{{ libro.titulo }}</td>
-												<td class="cell">{{ libro.autores.apellido }}, {{ libro.autores.nombre }}</td>
-												{% if libro.portada %}
-												<td class="cell"><img src="{{ url_for('paginas.uploaded_file', filename=libro.portada) }}" alt="Sin imagen" style="width:200px;height:200px;"></td>
-												{% else %}
-												<p>Sin imagen</p>
-												{% endif %}
-												<td><a class="dropdown-item" href="{{ url_for('paginas.uploaded_file', filename=libro.epub) }}"><svg width="1em" height="1em" viewBox="0 0 16 16" class="bi bi-download me-2" fill="currentColor" xmlns="http://www.w3.org/2000/svg">
-													<path fill-rule="evenodd" d="M.5 9.9a.5.5 0 0 1 .5.5v2.5a1 1 0 0 0 1 1h12a1 1 0 0 0 1-1v-2.5a.5.5 0 0 1 1 0v2.5a2 2 0 0 1-2 2H2a2 2 0 0 1-2-2v-2.5a.5.5 0 0 1 .5-.5z"/>
-													<path fill-rule="evenodd" d="M7.646 11.854a.5.5 0 0 0 .708 0l3-3a.5.5 0 0 0-.708-.708L8.5 10.293V1.5a.5.5 0 0 0-1 0v8.793L5.354 8.146a.5.5 0 1 0-.708.708l3 3z"/>
-												  </svg>Descargar</a></td>
-											</tr>
-											{% endfor %}		
-										</tbody>
-									</table>
-						        </div><!--//table-responsive-->
-						    </div><!--//app-card-body-->		
-						</div><!--//app-card-->						
-			        </div><!--//tab-pane-->
-				</div><!--//tab-content--> 
-		    </div><!--//container-fluid-->
-{% endblock %}
\ No newline at end of file
diff --git a/LibrosPy/servicios/libros/templates/searchrautor.html b/LibrosPy/servicios/libros/templates/searchautor.html
similarity index 90%
rename from LibrosPy/servicios/libros/templates/searchrautor.html
rename to LibrosPy/servicios/libros/templates/searchautor.html
index 0618e3b..7489f36 100644
--- a/LibrosPy/servicios/libros/templates/searchrautor.html
+++ b/LibrosPy/servicios/libros/templates/searchautor.html
@@ -9,7 +9,7 @@
     <ul>
         {% for autor in autores %}
         <li>
-            <a href="{{ url_for('paginas.autor', autor_id=autor.id) }}">{{ autor.matricula }}</a>
+            <a href="{{ url_for('paginas.autor', autor_id=autor.id) }}">{{ autor.apellido }}, {{ autor.nombre }}</a>
         </li>
         {% endfor %}
     </ul>
diff --git a/LibrosPy/servicios/libros/templates/searchlibro.html b/LibrosPy/servicios/libros/templates/searchlibro.html
index 57d4392..bcbe536 100644
--- a/LibrosPy/servicios/libros/templates/searchlibro.html
+++ b/LibrosPy/servicios/libros/templates/searchlibro.html
@@ -5,16 +5,16 @@
 
 <h2>Resultados de la búsqueda para "{{ query }}"</h2>
 
-{% if songs %}
+{% if libros %}
     <ul>
-        {% for song in songs %}
+        {% for libro in libros %}
         <li>
-            <a href="{{ url_for('paginas.song', song_id=song.id) }}">{{ song.title }} by {{ song.author }}</a>
+            <a href="{{ url_for('paginas.libro', libro_id=libro.id) }}">'{{ libro.titulo }}' por {{ libro.autores.apellido }}, {{ libro.autores.nombre }}</a>
         </li>
         {% endfor %}
     </ul>
 {% else %}
-    <p>No se han encontrado canciones que coindidan con ese criterio.</p>
+    <p>No se han encontrado libros que coindidan con ese criterio.</p>
 {% endif %}
 
 <a href="{{ url_for('paginas.index') }}">Volver al inicio</a>