Infra-Mirrors/BookStack
1
0
Fork 0
mirror of https://github.com/BookStackApp/BookStack.git synced 2024-07-04 18:41:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Added image view access notice to role form

Browse Source 
Added to clarify the role permission in scenarios where users may have
not read the docs site to understand image access control.

Related to #3688
This commit is contained in:
Dan Brown 2022-09-06 17:20:35 +01:00
parent 6955b2fd5a
commit d8672944a5
No known key found for this signature in database
GPG Key ID: 46D9F943C24A2EF9
3 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
resources/lang/en/settings.php
View File

@ -161,6 +161,7 @@ return [
'roles_system_warning' => 'Be aware that access to any of the above three permissions can allow a user to alter their own privileges or the privileges of others in the system. Only assign roles with these permissions to trusted users.',
'role_asset_desc' => 'These permissions control default access to the assets within the system. Permissions on Books, Chapters and Pages will override these permissions.',
'role_asset_admins' => 'Admins are automatically given access to all content but these options may show or hide UI options.',
'role_asset_image_view_note' => 'This relates to visibility within the image manager. Actual access of uploaded image files will be dependant upon system image storage option.',
'role_all' => 'All',
'role_own' => 'Own',
'role_controlled_by_asset' => 'Controlled by the asset they are uploaded to',

8
resources/views/settings/roles/parts/form.blade.php
View File

@ -172,7 +172,7 @@
<a href="#" permissions-table-toggle-all-in-row class="text-small text-primary">{{ trans('common.toggle_all') }}</a>
</td>
<td>@include('settings.roles.parts.checkbox', ['permission' => 'image-create-all', 'label' => ''])</td>
<td style="line-height:1.2;"><small class="faded">{{ trans('settings.role_controlled_by_asset') }}</small></td>
<td style="line-height:1.2;"><small class="faded">{{ trans('settings.role_controlled_by_asset') }}<sup>1</sup></small></td>
<td>
@include('settings.roles.parts.checkbox', ['permission' => 'image-update-own', 'label' => trans('settings.role_own')])
<br>
@ -221,5 +221,11 @@
</td>
</tr>
</table>
<div>
<p class="text-muted text-small px-m">
<sup>1</sup> {{ trans('settings.role_asset_image_view_note') }}
</p>
</div>
</div>
</div>

8
tests/Permissions/RolesTest.php
View File

@ -163,6 +163,14 @@ class RolesTest extends TestCase
$this->assertEquals($this->user->id, $roleA->users()->first()->id);
}
public function test_image_view_notice_shown_on_role_form()
{
/** @var Role $role */
$role = Role::query()->first();
$this->asAdmin()->get("/settings/roles/{$role->id}")
->assertSee('Actual access of uploaded image files will be dependant upon system image storage option');
}
public function test_copy_role_button_shown()
{
/** @var Role $role */