Skip to content

Commit 67bccd3

Browse files
refactor: Adapt UserController to updated commonModel->lists signature
Adjust parameters for `commonModel->lists` calls within user creation and update flows. This aligns the group fetching logic with a refined `lists` method signature, which now uses explicit `key` and `where` parameters for filtering and includes an `isReset` flag.
1 parent 9bc8983 commit 67bccd3

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

modules/Users/Controllers/UserController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ public function create_user()
144144
if (!$users->save($user)) return redirect()->route('create_user')->withInput()->with('errors', $users->errors());
145145
$new_user = $users->findById($users->getInsertID());
146146

147-
$groups = $this->commonModel->lists('auth_groups', 'group', ['group!=' => 'superadmin'], 'id ASC', 0, 0, [], ['id' => $this->request->getPost('group')]);
147+
$groups = $this->commonModel->lists('auth_groups', 'group', ['group!=' => 'superadmin'], 'id ASC', 0, 0, [], [], [], ['isReset' => false,], ['key' => 'id', 'where' => $this->request->getPost('group')]);
148148
$groupNames = array_column($groups, 'group');
149149
$new_user->syncGroups(...$groupNames);
150150

@@ -227,7 +227,7 @@ public function update_user(int $id)
227227

228228
$u->fill($data);
229229
if ($user->save($u)) {
230-
$groups = $this->commonModel->lists('auth_groups', 'group', ['group!=' => 'superadmin'], 'id ASC', 0, 0, [], ['id' => $this->request->getPost('group')]);
230+
$groups = $this->commonModel->lists('auth_groups', 'group', ['group!=' => 'superadmin'], 'id ASC', 0, 0, [], [], [], ['isReset' => false,], ['key' => 'id', 'where' => $this->request->getPost('group')]);
231231
$groupNames = array_column($groups, 'group');
232232
$u->syncGroups(...$groupNames);
233233
return redirect()->route('users')->with('message', lang('Backend.updated', [$data['username']]));

0 commit comments

Comments
 (0)