Skip to content
This repository was archived by the owner on Jan 3, 2023. It is now read-only.

Commit 6012686

Browse files
committed
add edit btn for menu items
… … replace tinymce jquery with js … … … … better table search optimization
1 parent 8b768f2 commit 6012686

File tree

19 files changed

+105
-56
lines changed

19 files changed

+105
-56
lines changed

src/Controllers/Admin/MenusController.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ public function update($id, Request $request)
8080
'name' => 'required|unique:menus,name,' . $id,
8181
]);
8282

83-
$menu = $this->menuModel->find($id) ?: abort(404);
83+
$menu = $this->menuModel->findOrFail($id);
8484

8585
// clear prev records
8686
$menu->pages()->detach();
@@ -116,7 +116,7 @@ public function update($id, Request $request)
116116
*/
117117
public function destroy($id, Request $request)
118118
{
119-
$menu = $this->menuModel->find($id) ?: abort(404);
119+
$menu = $this->menuModel->findOrFail($id);
120120
$menu->pages()->detach();
121121
$menu->delete();
122122

@@ -134,7 +134,7 @@ public function destroyMulti(Request $request)
134134
$ids = explode(',', $request->ids);
135135

136136
foreach ($ids as $one) {
137-
$menu = $this->menuModel->find($one);
137+
$menu = $this->menuModel->findOrFail($one);
138138
$menu->pages()->detach();
139139
$menu->delete();
140140
}

src/Controllers/Admin/PagesController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,6 @@ public function forceDelete($id)
191191
*/
192192
protected function getItem($id)
193193
{
194-
return $this->pageModel->withTrashed()->find($id) ?: abort(404);
194+
return $this->pageModel->withTrashed()->findOrFail($id);
195195
}
196196
}

src/Controllers/Admin/PermissionsController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public function store(Request $request)
6262
*/
6363
public function edit($id)
6464
{
65-
$permission = Permission::find($id);
65+
$permission = Permission::findOrFail($id);
6666

6767
return view("{$this->adminPath}.permissions.edit", compact('permission'));
6868
}
@@ -81,7 +81,7 @@ public function update($id, Request $request)
8181
'name' => 'required|unique:permissions,name,' . $id,
8282
]);
8383

84-
Permission::find($id)->update($request->all());
84+
Permission::findOrFail($id)->update($request->all());
8585

8686
$this->clearCache();
8787

src/Controllers/Admin/RolesController.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class RolesController extends BaseController
1919
*/
2020
public function index()
2121
{
22-
$roles = Role::all();
22+
$roles = Role::with('permissions')->get();
2323

2424
return view("{$this->adminPath}.roles.index", compact('roles'));
2525
}
@@ -68,7 +68,7 @@ public function store(Request $request)
6868
*/
6969
public function edit($id)
7070
{
71-
$role = Role::find($id);
71+
$role = Role::findOrFail($id);
7272
$permissions = Permission::pluck('name', 'name');
7373

7474
return view("{$this->adminPath}.roles.edit", compact('role', 'permissions'));
@@ -88,7 +88,7 @@ public function update($id, Request $request)
8888
'name' => 'required|unique:roles,name,' . $id,
8989
]);
9090

91-
$role = Role::find($id);
91+
$role = Role::findOrFail($id);
9292
$permissions = $request->input('permissions') ?: [];
9393

9494
$role->update($request->except('permissions'));

src/Controllers/Admin/Traits/MenuOps.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public function getMenuPages($id)
4040
public function removePage($id, Request $request)
4141
{
4242
// remove page from menu
43-
$menu = $this->menuModel->find($id) ?: abort(404);
43+
$menu = $this->menuModel->findOrFail($id);
4444
$menu->pages()->detach($request->page_id);
4545
$menu->touch();
4646

@@ -121,6 +121,6 @@ protected function clearSelfAndNests($id)
121121

122122
protected function findPage($id)
123123
{
124-
return $this->pageModel->find($id) ?: abort(404);
124+
return $this->pageModel->findOrFail($id);
125125
}
126126
}

src/Controllers/Admin/Traits/PageOps.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ protected function cleanEmptyTranslations($request)
7272

7373
foreach ($result as $k => $v) {
7474
if (is_array($v)) {
75-
$result[$k] = !array_filter($v) ? null : array_filter($v);
75+
$result[$k] = array_filter($v) ?: null;
7676
}
7777
}
7878

src/Controllers/Admin/UsersController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public function update($id, Request $request)
9696
'email' => 'required|email|unique:users,email,' . $id,
9797
]);
9898

99-
$user = $this->userModel->find($id) ?: abort(404);
99+
$user = $this->userModel->findOrFail($id);
100100
$roles = $request->input('roles') ?: [];
101101
$permissions = $request->input('permissions') ?: [];
102102
$img = $this->getImage($request->avatar);

src/Traits/NavigationTrait.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ protected function searchForRoute($name, $code)
146146
$routesListFile = include $this->listFileDir;
147147

148148
// check if we have a link according to that "routeName & code"
149-
return $routesListFile[$name][$code] ?? false;
149+
return $routesListFile[$name][$code] ?: false;
150150
}
151151

152152
/**

src/resources/assets/js/bulma/index.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ export default {
8080
console.error(err)
8181
})
8282
}
83-
}
83+
},
84+
render() {}
8485
}
8586
</script>

src/resources/assets/js/bulma/menu.vue

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
<script>
22
import draggable from 'vuedraggable'
3-
import MenuChild from './menu_childs.vue'
3+
import MenuChild from './menu_child.vue'
44
import menu from './mixins/menu'
55
66
export default {
77
components: {draggable, MenuChild},
8-
name: 'menu-comp',
8+
name: 'sm-menu',
99
mixins: [menu],
1010
props: ['getMenuPages', 'delPage'],
1111
data() {
@@ -153,6 +153,7 @@ export default {
153153
this.updatePages(val)
154154
})
155155
}
156-
}
156+
},
157+
render() {}
157158
}
158159
</script>

0 commit comments

Comments
 (0)