From 750847e3b6119c4e9b03d654762cad9a67675f70 Mon Sep 17 00:00:00 2001
From: unknown <ignacio@iaa.es>
Date: Thu, 3 Apr 2025 21:14:39 +0200
Subject: [PATCH] Revisado roles Compras

---
 ci4/app/Controllers/Compras/Proveedores.php   |   8 +-
 .../themes/vuexy/main/menus/compras_menu.php  |  74 +++---
 .../vuexy/main/menus/configuracion_menu.php   | 230 +++++++++---------
 3 files changed, 162 insertions(+), 150 deletions(-)

diff --git a/ci4/app/Controllers/Compras/Proveedores.php b/ci4/app/Controllers/Compras/Proveedores.php
index 0ac9be3e..cca3fe73 100755
--- a/ci4/app/Controllers/Compras/Proveedores.php
+++ b/ci4/app/Controllers/Compras/Proveedores.php
@@ -50,6 +50,8 @@ class Proveedores extends \App\Controllers\BaseResourceController {
 
 
     public function index() {
+
+        checkPermission('proveedores.menu');
         
         $viewData = [
                 'currentModule' => static::$controllerSlug,
@@ -67,9 +69,7 @@ class Proveedores extends \App\Controllers\BaseResourceController {
 
     public function add() {
         
-        
-
-        
+        checkPermission('proveedores.create');        
 
         if ($this->request->getPost()) :
 
@@ -138,6 +138,8 @@ class Proveedores extends \App\Controllers\BaseResourceController {
     } // end function add()
 
     public function edit($requestedId = null) {
+
+        checkPermission('proveedores.edit');
         
         if ($requestedId == null) :
             return $this->redirect2listView();
diff --git a/ci4/app/Views/themes/vuexy/main/menus/compras_menu.php b/ci4/app/Views/themes/vuexy/main/menus/compras_menu.php
index 8af7bd89..cc2dd5e8 100644
--- a/ci4/app/Views/themes/vuexy/main/menus/compras_menu.php
+++ b/ci4/app/Views/themes/vuexy/main/menus/compras_menu.php
@@ -2,35 +2,45 @@
 /**
  * MENU COMPRAS
  */
-if (auth()->user()->inGroup('beta')) {
-?>
-<!-- BUY -->
-<li class="menu-item">
-    <a href="javascript:void(0);" class="menu-link menu-toggle">
-        <i class="menu-icon tf-icons ti ti-shopping-cart"></i>
-        <?= lang("App.menu_compras") ?>
-    </a>
-    <ul class="menu-sub">
-        <li class="menu-item">
-            <a href="<?= site_url("servicios/compras") ?>" class="menu-link">
-                <?= lang("App.menu_compras") ?>
-            </a>
-        </li>
-        <li class="menu-item">
-            <a href="<?= site_url("servicios/productos") ?>" class="menu-link">
-                <?= lang("App.menu_productos") ?>
-            </a>
-        </li>
-        <li class="menu-item">
-            <a href="<?= site_url("compras/proveedores") ?>" class="menu-link">
-                <?= lang("App.menu_proveedores") ?>
-            </a>
-        </li>
-        <li class="menu-item">
-            <a href="<?= site_url("servicios/compras/ajustes") ?>" class="menu-link">
-                <?= lang("App.menu_compras_ajustes") ?>
-            </a>
-        </li>
-    </ul>
-</li>
-<?php } ?>
+if (
+    auth()->user()->can('proveedores.menu') ||
+    auth()->user()->inGroup('beta')
+) {
+
+    ?>
+    <!-- COMPRAS -->
+    <li class="menu-item">
+        <a href="javascript:void(0);" class="menu-link menu-toggle">
+            <i class="menu-icon tf-icons ti ti-shopping-cart"></i>
+            <?= lang("App.menu_compras") ?>
+        </a>
+        <ul class="menu-sub">
+            <?php if (auth()->user()->can('proveedores.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("compras/proveedores") ?>" class="menu-link">
+                        <?= lang("App.menu_proveedores") ?>
+                    </a>
+                </li>
+            <?php } ?>
+
+            <?php if (auth()->user()->inGroup('beta')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("servicios/compras") ?>" class="menu-link">
+                        <?= lang("App.menu_compras") ?>
+                    </a>
+                </li>
+                <li class="menu-item">
+                    <a href="<?= site_url("servicios/productos") ?>" class="menu-link">
+                        <?= lang("App.menu_productos") ?>
+                    </a>
+                </li>
+
+                <li class="menu-item">
+                    <a href="<?= site_url("servicios/compras/ajustes") ?>" class="menu-link">
+                        <?= lang("App.menu_compras_ajustes") ?>
+                    </a>
+                </li>
+            <?php } ?>
+        </ul>
+    </li>
+<?php } ?>
\ No newline at end of file
diff --git a/ci4/app/Views/themes/vuexy/main/menus/configuracion_menu.php b/ci4/app/Views/themes/vuexy/main/menus/configuracion_menu.php
index 5a3621f9..169ae9c3 100644
--- a/ci4/app/Views/themes/vuexy/main/menus/configuracion_menu.php
+++ b/ci4/app/Views/themes/vuexy/main/menus/configuracion_menu.php
@@ -6,127 +6,127 @@
 if (
     auth()->user()->can('paises.menu') ||
     auth()->user()->can('formas-pago.menu') ||
-    auth()->user()->can('papel-genericosk   .menu') ||
+    auth()->user()->can('papel-generico.menu') ||
     auth()->user()->can('papel-impresion.menu') ||
     auth()->user()->can('maquinas.menu') ||
     auth()->user()->can('maquinas-defecto.menu') ||
     auth()->user()->can('usuarios.menu') ||
     auth()->user()->can('roles-permisos.menu')
 ) {
-?>
-<li class="menu-item">
-    <a href="javascript:void(0);" class="menu-link menu-toggle">
-        <i class="menu-icon tf-icons ti ti-adjustments-horizontal"></i>
-        <?= lang("App.menu_configuration") ?>
-    </a>
-    <ul class="menu-sub">
+    ?>
+    <li class="menu-item">
+        <a href="javascript:void(0);" class="menu-link menu-toggle">
+            <i class="menu-icon tf-icons ti ti-adjustments-horizontal"></i>
+            <?= lang("App.menu_configuration") ?>
+        </a>
+        <ul class="menu-sub">
 
-        <?php if (auth()->user()->can('paises.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to('paisList') ?>" class="menu-link">
-                <?= lang("App.menu_paises") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('formas-pago.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to('formasPagoList') ?>" class="menu-link">
-                <?= lang("App.menu_formas_pago") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('papel-generico.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= site_url("configuracion/papelesgenericos") ?>" class="menu-link">
-                <?= lang("App.menu_papelgenerico") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('papel-impresion.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= site_url("configuracion/papelesimpresion") ?>" class="menu-link">
-                <?= lang("App.menu_papelimpresion") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('maquinas.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= site_url("configuracion/maquinas") ?>" class="menu-link">
-                <?= lang("App.menu_maquina") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('maquinas-defecto.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= site_url("configuracion/maquinasdefecto") ?>" class="menu-link">
-                <?= lang("App.menu_maquina_defecto") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->inGroup('admin') || auth()->user()->inGroup('beta')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to("maquinaTareaList") ?>" class="menu-link">
-                <?= lang("App.menu_maquina_tareas") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('usuarios.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= site_url("configuracion/users") ?>" class="menu-link">
-                <?= lang("App.menu_users") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('roles-permisos.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= site_url("configuracion/group") ?>" class="menu-link">
-                <?= lang("App.menu_permission_group") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('proveedores.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to("proveedorList") ?>" class="menu-link">
-                <?= lang("App.menu_proveedores") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('ubicaciones.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to("ubicacionesList") ?>" class="menu-link">
-                <?= lang("App.menu_ubicaciones") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->can('series-facturas.menu')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to("seriesFacturasList") ?>" class="menu-link">
-                <?= lang("App.menu_series_facturas") ?>
-            </a>
-        </li>
-        <?php } ?>
+            <?php if (auth()->user()->can('paises.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to('paisList') ?>" class="menu-link">
+                        <?= lang("App.menu_paises") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('formas-pago.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to('formasPagoList') ?>" class="menu-link">
+                        <?= lang("App.menu_formas_pago") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('papel-generico.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("configuracion/papelesgenericos") ?>" class="menu-link">
+                        <?= lang("App.menu_papelgenerico") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('papel-impresion.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("configuracion/papelesimpresion") ?>" class="menu-link">
+                        <?= lang("App.menu_papelimpresion") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('maquinas.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("configuracion/maquinas") ?>" class="menu-link">
+                        <?= lang("App.menu_maquina") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('maquinas-defecto.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("configuracion/maquinasdefecto") ?>" class="menu-link">
+                        <?= lang("App.menu_maquina_defecto") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->inGroup('admin')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to("maquinaTareaList") ?>" class="menu-link">
+                        <?= lang("App.menu_maquina_tareas") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('usuarios.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("configuracion/users") ?>" class="menu-link">
+                        <?= lang("App.menu_users") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('roles-permisos.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= site_url("configuracion/group") ?>" class="menu-link">
+                        <?= lang("App.menu_permission_group") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('proveedores.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to("proveedorList") ?>" class="menu-link">
+                        <?= lang("App.menu_proveedores") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('ubicaciones.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to("ubicacionesList") ?>" class="menu-link">
+                        <?= lang("App.menu_ubicaciones") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->can('series-facturas.menu')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to("seriesFacturasList") ?>" class="menu-link">
+                        <?= lang("App.menu_series_facturas") ?>
+                    </a>
+                </li>
+            <?php } ?>
 
-        <?php if (auth()->user()->inGroup('admin') || auth()->user()->inGroup('beta')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to('erroresPresupuestoIndex') ?>" class="menu-link">
-                <?= lang("App.menu_error_presupuesto") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->inGroup('admin') || auth()->user()->inGroup('beta')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to('variablesIndex') ?>" class="menu-link">
-                <?= lang("App.menu_variables") ?>
-            </a>
-        </li>
-        <?php } ?>
-        <?php if (auth()->user()->inGroup('admin') || auth()->user()->inGroup('beta')) { ?>
-        <li class="menu-item">
-            <a href="<?= route_to('configMessagesIndex') ?>" class="menu-link">
-                <?= lang("App.menu_config_messages") ?>
-            </a>
-        </li>
-        <?php } ?>
-    </ul>
-</li>
+            <?php if (auth()->user()->inGroup('admin')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to('erroresPresupuestoIndex') ?>" class="menu-link">
+                        <?= lang("App.menu_error_presupuesto") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->inGroup('admin')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to('variablesIndex') ?>" class="menu-link">
+                        <?= lang("App.menu_variables") ?>
+                    </a>
+                </li>
+            <?php } ?>
+            <?php if (auth()->user()->inGroup('admin')) { ?>
+                <li class="menu-item">
+                    <a href="<?= route_to('configMessagesIndex') ?>" class="menu-link">
+                        <?= lang("App.menu_config_messages") ?>
+                    </a>
+                </li>
+            <?php } ?>
+        </ul>
+    </li>
 <?php } ?>
\ No newline at end of file