1.0.0 - msg error al crear user repetit i fix ui edit user

README.md

@@ -1,10 +1,14 @@
-# Aplicació per a fitxatges Nath v0.9.9
+# Aplicació per a fitxatges Nath v1.0.0
 #### Aleix Algueró, 2022
 ## Sobre l'aplicació
 App per a realitzar fitxatges d'inici i final de jornades i torns a l'empresa, a més de fitxatges de certes tasques concretes de magatzem. Aquests fitxatges (torns, jornades senceres i tasques) podran ser llistats pel propi treballador des de dins de l'aplicació. I l'administrador podrà llistar totes les jornades de tothom i fer consultes des d'una taula dinàmica. A més, l'administrador podrà canviar certs camps dels usuaris, crear-ne de nous i eliminar-los.
-Ara mateix, en versió 0.9.9, sembla que té una certa estabilitat.
+Ara mateix, en versió 1.0.0, sembla que té una certa estabilitat.
 
-### Update 0.9.8
+### Update 1.0.0
+- Correcció de UI a l'edició d'usuaris.
+- Missatges d'error al crear un usuari ja existent.
+
+### Update 0.9.9
 - Correcció de UI a la creació d'usuaris.
 - Millora de rendiment.
 
@@ -59,7 +63,7 @@ Ara mateix, en versió 0.9.9, sembla que té una certa estabilitat.
 - Icones d'idioma.
 
 
-### Errors i bugs (versió 0.9.9)
+### Errors i bugs (versió 1.0.0)
 - La geolocalització no funciona sense HTTPS.
 - Si s'inicia una tasca i es finalitza molt ràpid (cosa antinatural), es pot arribar a buguejar i deixar penjada una tasca inacabada mentres s'ha començat una de nova.
 - Desactivant el JS segurament es pot trencar el funcionament dels botons. És possible que es puguen iniciar varies tasques alhora i que el programa perdi la seva funcionalitat de sèrie.

app/Http/Controllers/ReportsController.php

@@ -158,6 +158,12 @@ public function updateUser(Request $request, User $usuari){
 
     public function createUser(Request $request){
 
+        $searchingU = User::where(['username'=> $request->username])->get();
+        $searchingDNI = User::where(['DNI'=> $request->dni])->get();
+        $u = count($searchingU);
+        $dni = count($searchingDNI);
+        if ($u < 1 & $dni < 1){//si username i dni no existeixen
+
         $usuari = new User();
         $usuari->username = $request->username;
         $usuari->name = $request->name;
@@ -169,10 +175,13 @@ public function createUser(Request $request){
         $usuari->save(); 
 
         $lastUser = User::where(['username'=> $request->username])->latest()->first();
-        if ($lastUser != NULL || $lastUser != '[]') {
-            return response()->json(true);
-        } else {
-            return response()->json(['message' => 'error message'], 500);
+            if ($lastUser != NULL || $lastUser != '[]') {
+                return response()->json(true);
+            } else {
+                return response()->json(['message' => 'error message'], 500);
+            }
+        } else {//si username i dni ja existeixen
+            return response()->json(false);
         }
     }
 

package.json

@@ -1,6 +1,6 @@
 {   
     "name": "nath",
-    "version": "0.9.9",
+    "version": "1.0.0",
     "private": true,
     "license" : "MIT License",
     "scripts": {

public/js/AdminUsers.js

@@ -199,7 +199,6 @@ function modalCreateUser(){
                 url: "/createUser",
                 data: $('#createUser').serialize(),
                 success: function( response ) {
-
                     if (response == true){
 
                         $("#alert-message-create-user").text(msgUserCreated);
@@ -213,9 +212,9 @@ function modalCreateUser(){
                     setInterval(function(){
                     window.location.reload();
                       }, 500)
-                    } else {
+                    } else {//user existent o repetit
 
-                        $("#alert-danger-message-create").text(msgError);
+                        $("#alert-danger-message-create").text(msgUserCreateError);
                         $("#alert-danger-create")
                         .fadeTo(4000, 1000)
                         .slideUp(1000, function () {
@@ -225,7 +224,7 @@ function modalCreateUser(){
 
                 },
                 error: function(xhr, textStatus, error){
-                    $("#alert-danger-message-create").text(msgUserCreateError);
+                    $("#alert-danger-message-create").text(msgError);
                         $("#alert-danger-create")
                         .fadeTo(4000, 1000)
                         .slideUp(2000, function () {

resources/views/admin/usuaris.blade.php

@@ -71,6 +71,12 @@
                                         type="text" autofocus>
                                 </div></div>
 
+                                <div class="row mb-3">
+                                    <label class="col-md-4 col-form-label text-md-end">DNI</label>
+                                    <div class="col-md-6"><input id="dni" name="dni" class="form-control" 
+                                        type="text" autofocus>
+                                </div></div>
+
                                 <div class="row mb-3">
                                     <label for="magatzem" class="col-md-4 col-form-label text-md-end">{{ __('messages.Warehouse') }}</label>
                                         <div class="col-md-6">
@@ -91,12 +97,6 @@
                                         </div>
                                 </div></div>
 
-                                <div class="row mb-3">
-                                    <label class="col-md-4 col-form-label text-md-end">DNI</label>
-                                    <div class="col-md-6"><input id="dni" name="dni" class="form-control" 
-                                        type="text" autofocus>
-                                </div></div>
-
                                 <div class="modal-footer">
                                     <button id="closeModalUpdate" type="button" class="btn btn-outline-secondary">{{ __('messages.Cancel') }}</button>
                                     <button id="updateUser" type="button" class="btn btn-outline-dark" {{-- onclick="updateUser()" --}}>{{ __('messages.Edit') }}</button>