Skip to content
Snippets Groups Projects

hide booking-plan-icon if the user has no rights

Merged David Siegfried requested to merge (removed):biest-53 into main
3 unresolved threads

closes #53 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
30 30 Icon::create('timetable'),
31 31 ['target' => '_blank']
32 32 );
33 } else {
34 33 $actions->addLink(
  • added 1 commit

    Compare with previous version

  • Moritz Strohm
    Moritz Strohm @strohm started a thread on an outdated change in commit c806c340
  • 17 17 Icon::create('info-circle'),
    18 18 ['data-dialog' => '']
    19 19 );
    20 if ($room->userHasPermission($current_user, 'autor')) {
    20 if ($room->userHasPermission($current_user, 'autor') || $room->booking_plan_is_public) {
    • Es sollte noch eine Prüfung darauf, ob RESOURCES_SHOW_PUBLIC_ROOM_PLANS angeschaltet ist, hinzugefügt werden, also:

      ($room->booking_plan_is_public && Config::get()->RESOURCES_SHOW_PUBLIC_ROOM_PLANS)

      Wie im Ticket #53 (closed) bereits geschrieben wurde, funktioniert es dann ja.

    • Please register or sign in to reply
  • Moritz Strohm
    Moritz Strohm @strohm started a thread on commit ac0e9904
  • 30 30 Icon::create('timetable'),
    31 31 ['target' => '_blank']
    32 32 );
    33 $actions->addLink(
    34 $room->getActionLink('booking_plan'),
    35 _('Belegungsplan'),
    36 Icon::create('timetable'),
    37 ['data-dialog' => 'size=big']
    38 );
    33 } else {
  • added 1 commit

    Compare with previous version

  • Die grundlegende Änderung, die die Lösung dieses BIESt bewirkt, erscheint mir unproblematisch.

    Edited by Moritz Strohm
  • Elmar Ludwig approved this merge request

    approved this merge request

  • David Siegfried mentioned in commit 39b6263f

    mentioned in commit 39b6263f

  • Please register or sign in to reply
    Loading