From e7ad1a7a5abd6a900493198ed241029be3e3b922 Mon Sep 17 00:00:00 2001 From: efernandez Date: Thu, 3 Jan 2019 16:38:06 -0300 Subject: [PATCH] code review --- src/js/controllerModal.js | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/src/js/controllerModal.js b/src/js/controllerModal.js index e3f4953..49e3d34 100644 --- a/src/js/controllerModal.js +++ b/src/js/controllerModal.js @@ -25,7 +25,7 @@ angular.module('focaModal') $scope.selectedEntidad = -1; $scope.busquedaPress = function(key) { - if (key === 13) { + if(key === 13) { $scope.searchLoading = true; focaModalService.getEntidad($scope.filters, query).then( function(res) { @@ -49,7 +49,7 @@ angular.module('focaModal') $scope.filteredEntidades.length / $scope.numPerPage ); $scope.resetPage(); - }else if(pressed){ + }else if(pressed) { $timeout(function() { angular.element('#search')[0].focus(); $scope.filters = ''; @@ -89,15 +89,12 @@ angular.module('focaModal') if (key === 38) { anterior(key); } - if (key === 40) { siguiente(key); } - if (key === 37) { retrocederPagina(); } - if (key === 39) { avanzarPagina(); } @@ -114,22 +111,18 @@ angular.module('focaModal') paginas.unshift(paginaActual - 2); } } - if (paginaActual + 1 < $scope.lastPage) { paginas.push(paginaActual + 1); if (paginaActual + 2 < $scope.lastPage) { paginas.push(paginaActual + 2); } } - if (paginaActual !== 1) { paginas.unshift(1); } - if (paginaActual !== $scope.lastPage) { paginas.push($scope.lastPage); } - return paginas; } @@ -138,7 +131,7 @@ angular.module('focaModal') } function anterior() { - if ($scope.selectedEntidad === 0 && $scope.currentPage > 1) { + if($scope.selectedEntidad === 0 && $scope.currentPage > 1) { retrocederPagina(); } else { $scope.selectedEntidad--; @@ -146,7 +139,7 @@ angular.module('focaModal') } function siguiente() { - if ($scope.selectedEntidad < $scope.currentPageEntidades.length - 1) { + if($scope.selectedEntidad < $scope.currentPageEntidades.length - 1) { $scope.selectedEntidad++; } else { avanzarPagina(); @@ -161,7 +154,7 @@ angular.module('focaModal') } function avanzarPagina() { - if ($scope.currentPage < $scope.lastPage) { + if($scope.currentPage < $scope.lastPage) { $scope.selectPage($scope.currentPage + 1); $scope.selectedEntidad = 0; } -- 1.9.1