complete function: distribution of user menus

1. fix selector return value contain space 2. fix loading user menu problem, it will create new user-menu when it does not esist instead of showing wrong message
taoyu65 · Aug 10, 2017 · 29bd8a7 · 29bd8a7
1 parent 016d7cd
commit 29bd8a7
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 10 deletions.
diff --git a/src/Models/UserMenu.php b/src/Models/UserMenu.php
@@ -2,6 +2,9 @@
 
 namespace YM\Models;
 
+use Illuminate\Support\Facades\Cache;
+use Illuminate\Support\Facades\DB;
+
 class UserMenu extends UmiBase
 {
     use BreadOperation;
@@ -21,15 +24,22 @@ public function __construct(array $attributes = [], $openCache = true, $orderBy
     public function userJsonMenu($userId)
     {
         if ($this->openCache) {
-            return $this->cachedTable
-                ->where('user_id', $userId)
-                ->first()
-                ->json;
+            $userMenu = $this->cachedTable->where('user_id', $userId)->first();
+        } else {
+            $userMenu = self::where('user_id', $userId)->first();
         }
 
-        return self::where('user_id', $userId)
-            ->first()
-            ->json;
+        if ($userMenu) {
+            return $userMenu->json;
+        } else {
+            DB::table($this->table)
+                ->insert([
+                    'user_id'   => $userId,
+                    'json'      => '[]'
+                ]);
+            Cache::pull($this->table);
+            return '[]';
+        }
     }
 
     public function updateUserMenu($userId, $json)

diff --git a/src/resources/lang/en/menu.php b/src/resources/lang/en/menu.php
@@ -16,4 +16,6 @@
     'tip2'              => 'This list is the <strong class="red">entire menus</strong>, If you want manage different user\'s menu, you can use another function called <strong>Distribution</strong>',
     'reload'            => 'Reload',
     'save'              => 'Save',
+    'warning'           => 'Warning:',
+    'superAdminWarning' => 'is Super Admin which will load all menus regardless this setting, but still can set user menus and it will work when remove that super admin identity.'
 ];
diff --git a/src/resources/lang/zh_cn/menu.php b/src/resources/lang/zh_cn/menu.php
@@ -16,4 +16,6 @@
     'tip2'              => '这是 <strong class="red">系统菜单</strong>, 如果你想管理用户菜单, 请进入 菜单设置中的 <strong>分配菜单</strong> 中设置',
     'reload'            => '重新加载',
     'save'              => '保存',
+    'warning'           => '警告:',
+    'superAdminWarning' => '是超级管理员, 超级管理员默认加载所有菜单, 但是依然可以对其进行菜单设置, 只有在取消它的超级管理员身份时才起作用'
 ];
diff --git a/src/resources/views/common/selector.blade.php b/src/resources/views/common/selector.blade.php
@@ -57,7 +57,7 @@
                 </thead>
                 <tbody>
                     @foreach($records as $record)
-                        <tr class="tr" onclick="parent.{{$selector->functionName}}($(this).find('#{{$selector->returnField}}').html())" >
+                        <tr class="tr" onclick="parent.{{$selector->functionName}}($(this).find('#{{$selector->returnField}}').html().trim())" >
                             @foreach($selector->fields as $item)
                                 <td>
                                     <span id="{{$item}}">

diff --git a/src/resources/views/menu/distribution.blade.php b/src/resources/views/menu/distribution.blade.php
@@ -23,8 +23,9 @@
                 <i class="ace-icon fa fa-check"></i>
                 {{trans('umiTrans::menu.userFirst')}}.
             </strong>
-            {{trans('umiTrans::menu.tip1')}}
-            </p>
+            {{trans('umiTrans::menu.tip1')}} <br>
+            <span class="red"><strong>{{trans('umiTrans::menu.warning')}} </strong>"{{config('umi.system_role.super_admin')}}"
+                {{trans('umiTrans::menu.superAdminWarning')}}</span>
         </div>
     </div>