FriendsOfREDAXO / activity_log

Activity Log für REDAXO 5
MIT License
18 stars 0 forks source link

Activity Log nur für Admins? #12

Closed fietstouring closed 1 year ago

fietstouring commented 2 years ago

Ich kann bei User Roles zwar Activity_log unter Allgemein auswählen, der Menüpunkt erscheint aber bei nicht-Admins nicht. Redaxo 5.13.3

eaCe commented 2 years ago

Ich denke ich habe da einen Fehler in der Config. Eigentlich ist dieser nur für Admins gedacht, weiter unten habe ich aber eine Permission. Diese müsste ich entfernen. https://github.com/FriendsOfREDAXO/activity_log/blob/master/package.yml#L9

fietstouring commented 1 year ago

Bekomme einen Error wenn ein nicht-Admin neue Kategorien in einer Mehrsprachigen Redaxo-Installation v 14 anlegen will

Call to a member function getName() on null …/redaxo/src/addons/activity_log/lib/extension_points/category.php:55

eaCe commented 1 year ago

@fietstouring danke. Schau ich mir die Tage an! Nun habe ich schon Tests für alles gebaut, aber nur als Admin. Das sollte ich auch erweitern...

fietstouring commented 1 year ago

ok danke, ich versuche gerade den Fehler auf meinem lokalen System zu reproduzieren. Am Produktiv-System kam er beim Anlegen einer neuen Subkategorie. Hier kommt jetzt ein neuer Fehler beim Aufruf vom Addon selbst.

Call to a member function getName() on null
…/redaxo/src/addons/activity_log/fragments/filter-form.php:35
fietstouring commented 1 year ago

hier noch mein System Report:

System report (REDAXO 5.14.1, PHP 8.0.14, MySQL 5.7.34) | REDAXO | | | ------------: | :--------- | | Version | 5.14.1 | | PHP | | | ------------: | :----------------------------- | | Version | 8.0.14 | | OPcache | no | | Xdebug | no | | Warning | The setup will continue without SSL encryption. It is recommended to use SSL encryption for each call to the front- and backend in order to ensure privacy and data protection. | | Database | | | ------------: | :----------- | | Version | MySQL 5.7.34 | | Character set | utf8mb4 | | Server | | | ------------: | :----------- | | OS | Darwin | | SAPI | cgi-fcgi | | Webserver | nginx/1.19.2 | | Request | | | ------------: | :-------------------- | | Browser | Chrome/103.0.5060.134 | | Protocol | HTTP/1.1 | | HTTPS | no | | Packages | | | --------------------: | :--------- | | activity_log | 0.0.1 | | backup | 2.8.2 | | be_password | 2.0.1 | | be_style | 3.1.0 | | be_style/customizer | 3.1.0 | | be_style/redaxo | 3.1.0 | | bloecks | 3.1.1 | | bloecks/cutncopy | 3.1.1 | | bloecks/dragndrop | 3.1.1 | | bloecks/status | 3.1.1 | | cache_warmup | 4.0.0 | | cke5 | 5.2.0 | | clear_content | 1.1.1 | | cronjob | 2.9.0 | | cropper | 1.3.0 | | developer | 3.9.0 | | email_obfuscator | 2.0.0 | | global_settings | 2.7.1 | | install | 2.10.0 | | maintenance | 2.8.5 | | massif_minify | 1.3.2 | | mblock | 3.4.7 | | media_manager | 2.13.0 | | mediapool | 2.12.0 | | metainfo | 2.9.0 | | mform | 6.1.2 | | mform/docs | 1.1 | | minibar | 2.3.1 | | multiupload | 2.1.3 | | navbuilder | 1.0.6 | | navigation_factory | 2.0.3 | | phpmailer | 2.11.2 | | project | dev | | quick_navigation | 6.0.0 | | sprog | 1.4.0 | | structure | 2.14.1 | | structure/content | 2.14.1 | | structure_tweaks | 1.4.0 | | theme | 1.3.3 | | ui_tools | 1.2.1 | | users | 2.9.0 | | watson | 2.3.0 | | xcore | 1.1.4 | | yform | 4.0.2 | | yform/docs | 3.3.1 | | yform/email | 4.0.2 | | yform/manager | 4.0.2 | | yform/tools | 4.0.2 | | yform_spam_protection | 1.2.1 | | yform_usability | 2.0.4 | | yrewrite | 2.8.3 |
eaCe commented 1 year ago

@fietstouring ich kann den Fehler leider aktuell nicht reproduzieren.

fietstouring commented 1 year ago

ok, hatte ich schon vermutet, weil er bei mir auch seltsam inkosistent auftritt. muss da nochmal genauer reinschauen.

eaCe commented 1 year ago

Ich werde einfach mal eine weitere Abfrage um Category Name hinzufügen. Im Filter scheint der User keinen Namen zu nutzen. Auch hier werde ich Name nutzen, wenn vorhanden, ansonsten den Login. Das ist ein Fehler meinerseits.

eaCe commented 1 year ago

->getName könnte wegen einer gelöschten Category Probleme verursachen. Das sollte in der aktuellen GH-Version auch nicht mehr passieren.

eaCe commented 1 year ago

Besteht das Problem in der aktuellen Version immer noch?

eaCe commented 1 year ago

Ich mach mal zu, wenn es noch Probleme gibt gerne wieder auf machen.