Merge pull request #147 from rrafaelc/feat/listagem-perguntas-sistema

feat: listar perguntas e correcoes
rrafaelc · Nov 17, 2023 · bac30b7 · bac30b7
2 parents 2a86b36 + 2971973
commit bac30b7
Show file tree

Hide file tree

Showing 8 changed files with 120 additions and 443 deletions.
diff --git a/api/src/Controllers/PerguntaController.php b/api/src/Controllers/PerguntaController.php
@@ -46,16 +46,6 @@ private function processResourceRequest(string $method, string $id): void
           return;
         }
 
-        if ($usuarioLogado["cargo"] == CargoEnum::MODERADOR && $pergunta["criado_por"] != $usuarioLogado["id"]) {
-          http_response_code(403);
-          echo json_encode([
-            "status" => "error",
-            "errors" => ["Não permitido alterar pergunta de outro usuário"]
-          ]);
-
-          return;
-        }
-
         $data = (array) json_decode(file_get_contents("php://input"), true);
         $data["usuarioId"] = $usuarioLogado["id"];
 
@@ -87,16 +77,6 @@ private function processResourceRequest(string $method, string $id): void
           return;
         }
 
-        if ($usuarioLogado["cargo"] == CargoEnum::MODERADOR && $pergunta["criado_por"] != $usuarioLogado["id"]) {
-          http_response_code(403);
-          echo json_encode([
-            "status" => "error",
-            "errors" => ["Não permitido deletar pergunta de outro usuário"]
-          ]);
-
-          return;
-        }
-
         $this->gateway->delete($id);
 
         http_response_code(204);

diff --git a/api/src/Gateways/PerguntaGateway.php b/api/src/Gateways/PerguntaGateway.php
@@ -22,7 +22,7 @@ public function getAll(array $ordenacao): array
       ue.foto_uri AS foto_usuario_editado
       FROM pergunta p
       LEFT JOIN usuario u ON p.criado_por = u.id
-      LEFT JOIN pergunta_editada_por pe ON u.id = pe.usuario_id
+      LEFT JOIN pergunta_editada_por pe ON p.id = pe.pergunta_id
       LEFT JOIN usuario ue ON pe.usuario_id = ue.id";
 
     if (isset($ordenacao["MaisAlta"]) && $ordenacao["MaisAlta"]) {
@@ -37,7 +37,7 @@ public function getAll(array $ordenacao): array
         ue.foto_uri AS foto_usuario_editado
         FROM pergunta p
         LEFT JOIN usuario u ON p.criado_por = u.id
-        LEFT JOIN pergunta_editada_por pe ON u.id = pe.usuario_id
+        LEFT JOIN pergunta_editada_por pe ON p.id = pe.pergunta_id
         LEFT JOIN usuario ue ON pe.usuario_id = ue.id
         ORDER BY
           CASE
@@ -59,7 +59,7 @@ public function getAll(array $ordenacao): array
         ue.foto_uri AS foto_usuario_editado
         FROM pergunta p
         LEFT JOIN usuario u ON p.criado_por = u.id
-        LEFT JOIN pergunta_editada_por pe ON u.id = pe.usuario_id
+        LEFT JOIN pergunta_editada_por pe ON p.id = pe.pergunta_id
         LEFT JOIN usuario ue ON pe.usuario_id = ue.id
         ORDER BY curtidas DESC";
     }

diff --git a/sistema/colaboradores/script.js b/sistema/colaboradores/script.js
@@ -316,6 +316,7 @@ const execute = async () => {
         showCancelButton: true,
         confirmButtonText: 'Sim, confirmar!',
         cancelButtonText: 'Não',
+        allowEnterKey: false,
         icon: 'question',
       }).then(async (result) => {
         if (result.isConfirmed) {
@@ -325,6 +326,7 @@ const execute = async () => {
             showCancelButton: true,
             confirmButtonText: 'Sim, irei anotar!',
             cancelButtonText: 'Não',
+            allowEnterKey: false,
             icon: 'info',
           }).then(async (result) => {
             if (result.isConfirmed) {

diff --git a/sistema/curtidas/script.js b/sistema/curtidas/script.js
@@ -60,10 +60,12 @@ const execute = async () => {
       <td>
         <div id="colaborador">
           <div class="avatar">
-          <img src="${pergunta.foto_usuario ?? '../../img/userFallback.jpg'}" />
+          <img title="${pergunta.nome_usuario ?? 'N/A'}" src="${
+            pergunta.foto_usuario ?? '../../img/userFallback.jpg'
+          }" />
           </div>
           <div class="nome">
-          <span>${pergunta.nome_usuario ?? 'Sem usuário'}</span>
+          <span>${pergunta.nome_usuario ?? 'N/A'}</span>
           </div>
         </div>
       </td>
@@ -76,19 +78,12 @@ const execute = async () => {
         <div id="curtidas"><span>${pergunta.curtidas}</span></div>
       </td>
       <td>
-        ${
-          user.cargo === 'Administrador'
-            ? `<div id="acao">
+        <div id="acao">
           <a href="../../sistema/perguntas/editar/"><i class="fas fa-pencil"></i></a>
-          <button class='click' data-id=${pergunta.id} href="#"><i class="fas fa-trash-can"></i></button>
-        </div>`
-            : pergunta.criado_por === user.id
-            ? `<div id="acao">
-            <a href="../../sistema/perguntas/editar/"><i class="fas fa-pencil"></i></a>
-            <button class='click' data-id=${pergunta.id} href="#"><i class="fas fa-trash-can"></i></button>
-          </div>`
-            : ''
-        }
+          <button class='click' data-id=${
+            pergunta.id
+          } href="#"><i class="fas fa-trash-can"></i></button>
+        </div>
       </td>
   </tr>
   `,
@@ -144,10 +139,12 @@ const execute = async () => {
           <td>
             <div id="colaborador">
               <div class="avatar">
-                <img src="${pergunta.foto_usuario ?? '../../img/userFallback.jpg'}" />
+                <img title="${pergunta.nome_usuario ?? 'N/A'}" src="${
+                  pergunta.foto_usuario ?? '../../img/userFallback.jpg'
+                }" />
               </div>
               <div class="nome">
-                <span>${pergunta.nome_usuario ?? 'Sem usuário'}</span>
+                <span>${pergunta.nome_usuario ?? 'N/A'}</span>
               </div>
             </div>
           </td>
@@ -160,19 +157,12 @@ const execute = async () => {
             <div id="curtidas"><span>${pergunta.curtidas}</span></div>
           </td>
           <td>
-          ${
-            user.cargo === 'Administrador'
-              ? `<div id="acao">
+          <div id="acao">
             <a href="../../sistema/perguntas/editar/"><i class="fas fa-pencil"></i></a>
-            <button class='click' data-id=${pergunta.id} href="#"><i class="fas fa-trash-can"></i></button>
-          </div>`
-              : pergunta.criado_por === user.id
-              ? `<div id="acao">
-              <a href="../../sistema/perguntas/editar/"><i class="fas fa-pencil"></i></a>
-              <button class='click' data-id=${pergunta.id} href="#"><i class="fas fa-trash-can"></i></button>
-            </div>`
-              : ''
-          }
+            <button class='click' data-id=${
+              pergunta.id
+            } href="#"><i class="fas fa-trash-can"></i></button>
+          </div>
         </td>
       </tr>
       `,