FriendsOfREDAXO / geolocation

Tile-Proxy/Cache und Kartendarstellung mit Leaflet für REDAXO 5.13+
MIT License
26 stars 3 forks source link

trotz nicht vorhandener Berechtigungen Menü-Punkt sichtbar #111

Closed bitshiftersgmbh closed 1 year ago

bitshiftersgmbh commented 1 year ago

Weder die Tables sind in irgend einer Form freigegeben, noch die spezifischen geolocation-Einzelberechtigungen. Dennoch sieht der User das:

Screenshot 2022-12-08 150313

Klickt der Nutzer dort drauf, kommt:

image

christophboecker commented 1 year ago

Danke für den Hinweis, Peter. Ich hab mal das hier ausprobiert, also oben bei der Hauptseite ein perm: geolocation[] eingebaut. Ohne diese Berechtigung ist auch der Eintrag links im Menü weg.

page:
    title: translate:geolocation_title
    pjax: true
    icon: rex-icon fa-globe
    perm: geolocation[]
    subpages:
        config:
            title: translate:geolocation_config
            icon: fa fa-wrench
            perm: 'admin[]'
        mapset:
            title: translate:geolocation_mapset
            perm: 'geolocation[mapset]'
            icon: fa fa-map
            subPath: pages/yform.php
        layer:
            title: translate:geolocation_layer
            perm: 'geolocation[layer]'
            icon: fa fa-cloud-download
            subPath: pages/yform.php
        docs:
            title: translate:geolocation_manpage
            icon: fa fa-book
            subPath: help.php
        clear_cache:
            title: 'translate:geolocation_clear_cache'
            perm: 'geolocation[clearcache]'
            itemclass: pull-right
            linkclass: 'btn btn-delete'
            href:
                # Zur Info: Die konkreten Parameter setzt kontext-bezogen pages/index.php

Natürlich muss dann für die Zugriffsfechte neben dem Detail-Zugriffsrecht (z.B. geolocation[mapset]) immer auch das Recht 'geolocation[]` in die Rolle aufgenommen werden. Ob das so strukturell richtig ist, weiß ich nicht. Hab das nie ganz verstanden.

Probier die Lösung mal aus, bitte. Ich würde es dann so in das nächste Release aufnehmen.

Wenn das mal jemand mit Kenne richtig glatt zieht und eleganter gestaltet: gerne nen PR ("haaaaloooo, hört mich mich jemand?)

bitshiftersgmbh commented 1 year ago

Hat es vielleicht damit zu tun, dass ich mich vom Admin aus in den sehr unterpriveligierten User eingeloggt habe?

christophboecker commented 1 year ago

Ja, hat es.

bitshiftersgmbh commented 1 year ago

Gewollt oder Bug?

christophboecker commented 1 year ago

Ob das so strukturell richtig ist, weiß ich nicht. Hab das nie ganz verstanden.

Ich hab nur rudimentäre Ahnung von dem Perm-Gedöns in Redaxo. Irgendwann wird schon jemand meckern, wenn es sonderbar funktioniert. => also Bug.

bitshiftersgmbh commented 1 year ago

Die anderen AddOns sind ja auch nicht sichtbar, wenn man als Admin in einen anderen User hüpft. Frag mich, warum das hier der Fall ist, wo die .yml Perms doch sauber gesetzt scheinen ... 🤔

christophboecker commented 1 year ago

Weil die von vorneherein ne perm auf Page-Ebene, also oben haben. Bei Geolocaton nur auf Subpages. Das ist der Bug.

bitshiftersgmbh commented 1 year ago

Stimmt, da müsste geolocation[] rein, mein ich.