Skip to content

Commit 4a481e7

Browse files
committed
Merge remote-tracking branch 'origin/develop'
# Conflicts: # public/css/build/app.css # public/css/build/app.css.map # public/css/build/overrides.css # public/css/build/overrides.css.map # public/css/dist/all.css # public/js/dist/all.js # public/js/dist/all.js.map # public/mix-manifest.json
2 parents fff1463 + ee4abbc commit 4a481e7

File tree

26 files changed

+931
-714
lines changed

26 files changed

+931
-714
lines changed

.github/workflows/tests-mysql.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ jobs:
8282

8383
- name: Upload Laravel logs as artifacts
8484
if: always()
85-
uses: actions/upload-artifact@v4
85+
uses: actions/upload-artifact@v5
8686
with:
8787
name: laravel-logs-php-${{ matrix.php-version }}-run-${{ github.run_attempt }}
8888
path: |

.github/workflows/tests-postgres.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ jobs:
8181

8282
- name: Upload Laravel logs as artifacts
8383
if: always()
84-
uses: actions/upload-artifact@v4
84+
uses: actions/upload-artifact@v5
8585
with:
8686
name: laravel-logs-php-${{ matrix.php-version }}-run-${{ github.run_attempt }}
8787
path: |

.github/workflows/tests-sqlite.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ jobs:
6767

6868
- name: Upload Laravel logs as artifacts
6969
if: always()
70-
uses: actions/upload-artifact@v4
70+
uses: actions/upload-artifact@v5
7171
with:
7272
name: laravel-logs-php-${{ matrix.php-version }}-run-${{ github.run_attempt }}
7373
path: |

app/Http/Controllers/GroupsController.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ public function store(Request $request) : RedirectResponse
6565
$group->notes = $request->input('notes');
6666

6767
if ($group->save()) {
68+
$group->users()->sync($request->input('associated_users'));
6869
return redirect()->route('groups.index')->with('success', trans('admin/groups/message.success.create'));
6970
}
7071

@@ -88,7 +89,10 @@ public function edit(Group $group) : View | RedirectResponse
8889
$groupPermissions = [];
8990
}
9091
$selected_array = Helper::selectedPermissionsArray($permissions, $groupPermissions);
91-
return view('groups.edit', compact('group', 'permissions', 'selected_array', 'groupPermissions'));
92+
$associated_users = $group->users()->get();
93+
94+
//dd($associated_users->toArray());
95+
return view('groups.edit', compact('group', 'permissions', 'selected_array', 'groupPermissions'))->with('associated_users', $associated_users);
9296
}
9397

9498
/**
@@ -105,8 +109,10 @@ public function update(Request $request, Group $group) : RedirectResponse
105109
$group->permissions = json_encode($request->input('permission'));
106110
$group->notes = $request->input('notes');
107111

112+
108113
if (! config('app.lock_passwords')) {
109114
if ($group->save()) {
115+
$group->users()->sync($request->input('associated_users'));
110116
return redirect()->route('groups.index')->with('success', trans('admin/groups/message.success.update'));
111117
}
112118

app/Models/Ldap.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ public static function findAndBindUserLdap($username, $password)
220220
Log::debug('Filter query: '.$filterQuery);
221221

222222
// only try this if we have an Admin username set; otherwise use the 'legacy' method
223-
if ($settings->ldap_uname) {
223+
if (($settings->ldap_uname) && ($baseDn)) {
224224
// in the fallowing call, we pick a slow-failure of 0 because we might need to fall through to 'legacy'
225225
$fast_bind = self::findAndBindMultiOU($baseDn, $filterQuery, $password, 0);
226226
if ($fast_bind) {

app/Models/User.php

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,24 @@ public function isAvatarExternal() : bool
224224
return false;
225225
}
226226

227+
public function hasIndividualPermissions() {
228+
229+
if (is_object($this->permissions)) {
230+
$permissions = json_decode(json_encode($this->permissions), true);
231+
}
232+
233+
if (is_string($this->permissions)) {
234+
$permissions = json_decode($this->permissions, true);
235+
}
236+
237+
foreach ($permissions as $permission) {
238+
if ($permission != 0) {
239+
return true;
240+
}
241+
}
242+
return false;
243+
}
244+
227245
/**
228246
* Internally check the user permission for the given section
229247
*

0 commit comments

Comments
 (0)