Ordered entity permission roles by display name

Closes #2782

diff --git a/app/Auth/Role.php b/app/Auth/Role.php
index 629cd6a955d8abf7961b67aa1c598d1d62d30658..59b1f547c13f527b7f9bd19d524525a9415e1972 100644 (file)
--- a/app/Auth/Role.php
+++ b/app/Auth/Role.php
@@ -92,7 +92,7 @@ class Role extends Model implements Loggable
     }
 
     /**
     }
 
     /**

-     * Get all visible roles
+     * Get all visible roles.

      */
     public static function visible(): Collection
     {
      */
     public static function visible(): Collection
     {


@@ -104,7 +104,10 @@ class Role extends Model implements Loggable
      */
     public static function restrictable(): Collection
     {
      */
     public static function restrictable(): Collection
     {

-        return static::query()->where('system_name', '!=', 'admin')->get();
+        return static::query()
+            ->where('system_name', '!=', 'admin')
+            ->orderBy('display_name', 'asc')
+            ->get();

     }
 
     /**
     }
 
     /**

diff --git a/resources/views/form/entity-permissions.blade.php b/resources/views/form/entity-permissions.blade.php
index 6cf5ab8bdcac9db8b02f3debb384005ddaa27c80..d3e89cc447fa07953ac30599e27d9b625ec70c27 100644 (file)
--- a/resources/views/form/entity-permissions.blade.php
+++ b/resources/views/form/entity-permissions.blade.php
@@ -25,7 +25,7 @@
     <table permissions-table class="table permissions-table toggle-switch-list" style="{{ !$model->restricted ? 'display: none' : '' }}">
         <tr>
             <th>{{ trans('common.role') }}</th>
     <table permissions-table class="table permissions-table toggle-switch-list" style="{{ !$model->restricted ? 'display: none' : '' }}">
         <tr>
             <th>{{ trans('common.role') }}</th>

-            <th @if($model->isA('page')) colspan="3" @else colspan="4" @endif>
+            <th colspan="{{ $model->isA('page') ? '3' : '4'  }}">

                 {{ trans('common.actions') }}
                 <a href="#" permissions-table-toggle-all class="text-small ml-m text-primary">{{ trans('common.toggle_all') }}</a>
             </th>
                 {{ trans('common.actions') }}
                 <a href="#" permissions-table-toggle-all class="text-small ml-m text-primary">{{ trans('common.toggle_all') }}</a>
             </th>