feat: Namespace FriendsOfREDAXO/ActivityLog, Bugfixes & Dependency-Updates (v1.0.0-beta1) by skerbis · Pull Request #47 · FriendsOfREDAXO/activity_log

skerbis · 2026-03-06T16:56:57Z

Zusammenfassung

Dieses PR bringt die lang geplante Code-Modernisierung auf Basis der offenen Issues sowie verschiedene Bugfixes und aktualisierte Abhängigkeiten.

Änderungen

🏷️ Namespace-Migration (#38)

Alle Klassen wurden in den Namespace FriendsOfREDAXO\ActivityLog\ verschoben, wie es für FriendsOfREDAXO-Addons üblich ist:

Alt	Neu
`rex_activity`	`FriendsOfREDAXO\ActivityLog\Activity`
`rex_activity_log_cronjob`	`FriendsOfREDAXO\ActivityLog\ActivityLogCronjob`
`activity_clear`	`FriendsOfREDAXO\ActivityLog\ActivityClear`
`RexActivity\EP\*`	`FriendsOfREDAXO\ActivityLog\EP\*`

Abwärtskompatibilität: Alle alten Klassennamen bleiben als leere BC-Stubs erhalten (class rex_activity extends Activity {}), bestehender Code funktioniert weiterhin.

🐛 Fix: `clang_id` Undefined array key (#46)

In EP\Meta wurde $params['clang_id'] ohne Existenzprüfung verwendet. Jetzt wird rex_clang::getCurrentId() als sicherer Fallback genutzt.

🐛 Fix: `rex_sql_exception` beim Löschen von Artikeln/Kategorien

Beim Auslösen von ART_DELETED/CAT_DELETED wurde rex_article::get()/rex_category::get() aufgerufen, obwohl der Datensatz bereits gelöscht war – das kollidierte mit dem noch laufenden DELETE-Statement in service_article.php. Die DB-Abfrage wird beim Typ delete nun vollständig übersprungen.

✅ Keine Logs für Admins (#31)

Die bereits implementierte Option disable_for_admins ist jetzt offiziell Teil der neuen Activity-Klasse und sauber dokumentiert.

📦 Dependency-Updates (ersetzt PRs #41–#45)

phpunit/phpunit 9.6.21 → 9.6.34 (CVE-2026-24765)
nightwatch 3.6.2 → 3.15.0
lodash 4.17.21 → 4.17.23
geckodriver 3.0.2 → 6.1.0

Closes / References

Closes #38, Closes #46, References #31

Checklist

Neue Klassen unter FriendsOfREDAXO\ActivityLog\
BC-Stubs für alle alten Klassennamen
use-Statements in boot.php
package.yml auf v1.0.0-beta1
CHANGELOG.md hinzugefügt
Bugfix clang_id (meta.php Undefined array key "clang_id" #46)
Bugfix rex_sql_exception beim Löschen
Dependencies aktualisiert

…g_id (#46) - Neue Klassen unter FriendsOfREDAXO\ActivityLog\ und FriendsOfREDAXO\ActivityLog\EP\ - BC-Stubs für alle alten Klassen (rex_activity, RexActivity\EP\*) via extends - boot.php nutzt use-Statements statt globale FQN-Instantiierung - rex_cronjob_manager::registerType() nutzt ActivityLogCronjob::class - fix: Undefinierter Array-Key clang_id in Meta-EP fallback auf rex_clang::getCurrentId() - Alle EP-Callbacks nutzen static::class statt hardcoded String - Version 0.2.0

- phpunit/phpunit 9.6.21 -> 9.6.34 (behebt CVE-2026-24765, schließt #45) - lodash 4.17.21 -> 4.17.23 (schließt #44) - nightwatch 3.6.2 -> 3.15.0 (schließt #41, #43) - geckodriver 3.0.2 -> 6.1.0 (schließt #42)

rex_article::get() und rex_category::get() dürfen beim delete-Typ nicht aufgerufen werden – das Objekt ist zu dem Zeitpunkt bereits aus der DB entfernt. Der interne SQL-Query kollidierte mit dem noch laufenden DELETE-Statement in service_article.php und warf eine rex_sql_exception (Unable to fetch row for statement). Lösung: Early-return bei type === 'delete', DB-Lookup vollständig überspringen.

…_admins

…xt (Cronjob)

Copilot

Pull request overview

This PR modernizes the addon by migrating core functionality into the FriendsOfREDAXO\ActivityLog\* namespace (with backward-compatible stubs), applies targeted bugfixes in extension-point handlers, and updates PHP/Node tooling dependencies in preparation for v1.0.0-beta1.

Changes:

Introduces new namespaced core classes (Activity, ActivityClear, ActivityLogCronjob) and new EP handlers under FriendsOfREDAXO\ActivityLog\EP\*.
Replaces prior implementations with BC stubs (old class names now extend the new namespaced classes/traits).
Updates dependency versions (Composer + npm) and adds CHANGELOG.md.

Reviewed changes

Copilot reviewed 36 out of 38 changed files in this pull request and generated 12 comments.

Show a summary per file

File	Description
package.yml	Bumps addon version and switches console command mapping to the new namespaced command class.
package.json	Updates Nightwatch/Geckodriver/Lodash dependency placement and versions for e2e tooling.
boot.php	Boots the new namespaced EP registry and registers the new cronjob type class.
lib/Activity.php	New namespaced Activity core (logging, callbacks, clearEntries).
lib/ActivityClear.php	New namespaced console command implementation.
lib/ActivityLogCronjob.php	New namespaced cronjob implementation.
lib/EP/ActivityExtensionPoints.php	New EP “registry” that instantiates all EP handlers (incl. optional YForm).
lib/EP/EpTrait.php	New shared trait for registering extension points and writing Activity logs.
lib/EP/Article.php	New EP handler for article events, including delete handling behavior change.
lib/EP/Category.php	New EP handler for category events, including delete handling behavior change.
lib/EP/Clang.php	New EP handler for language events.
lib/EP/Media.php	New EP handler for media events.
lib/EP/Meta.php	New EP handler for meta updates with clang fallback to address missing `clang_id`.
lib/EP/Module.php	New EP handler for module events.
lib/EP/Slice.php	New EP handler for slice events with safer clang fallback and more defensive lookups.
lib/EP/Template.php	New EP handler for template events.
lib/EP/User.php	New EP handler for user events.
lib/EP/Yform.php	New EP handler for YForm manager events (conditional activation).
lib/rex_activity.php	BC stub: `rex_activity` now extends the namespaced `Activity`.
lib/rex_activity_log_cronjob.php	BC stub: `rex_activity_log_cronjob` now extends the namespaced cronjob.
lib/console.php	BC stub: `activity_clear` now extends the namespaced console command.
lib/rex_activity_extension_points.php	BC stub: old EP registry now extends the new EP registry.
lib/extension_points/article.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/category.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/clang.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/ep_trait.php	BC stub: old trait now composes the new namespaced trait.
lib/extension_points/media.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/meta.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/module.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/slice.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/template.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/user.php	BC stub: old EP handler class now extends the new namespaced handler.
lib/extension_points/yform.php	BC stub: old EP handler class now extends the new namespaced handler.
composer.json	Updates PHPUnit constraint to the newer 9.6.x line.
composer.lock	Locks updated dev dependencies consistent with the PHPUnit bump.
CHANGELOG.md	Adds release notes for `1.0.0-beta1` documenting migration and fixes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…tenz (#17/#21) - DB: source VARCHAR(191) nullable in install.php + update.php - Activity::source() Methode + sourceListCallback - EpTrait: getSource() + source in logExtensionPoint() - Alle EP-Handler: getSource() gibt Quelle zurück (article/category/...) - Filter-Formular: Suche, Typ, Quelle, User als Bootstrap-3-Row, Selectpicker - Filter-Persistenz: list->addParam() für Pager, Hidden-Inputs in Lösch-Form - SCSS: Zeilenfarben nach Log-Level, Source-Badges mit eigenen Farben, Dark Mode - Sprachdateien: neue Keys für Spalten, Suche, Filter - README: source() dokumentiert - CHANGELOG aktualisiert

…21)

…imports entfernt

…20 für e2e

… BC-Stub rex_activity

…t apply entfernt)

…rdcodiert

skerbis added 4 commits March 6, 2026 17:41

chore(deps): Dependencies aktualisieren

4ad684d

- phpunit/phpunit 9.6.21 -> 9.6.34 (behebt CVE-2026-24765, schließt #45) - lodash 4.17.21 -> 4.17.23 (schließt #44) - nightwatch 3.6.2 -> 3.15.0 (schließt #41, #43) - geckodriver 3.0.2 -> 6.1.0 (schließt #42)

chore: Version 1.0.0-beta1, CHANGELOG.md hinzugefügt

ed47b5b

Copilot AI review requested due to automatic review settings March 6, 2026 16:56

skerbis mentioned this pull request Mar 6, 2026

meta.php Undefined array key "clang_id" #46

Closed

Copilot started reviewing on behalf of skerbis March 6, 2026 16:57 View session

This was referenced Mar 6, 2026

Namespace FOR-konform anpassen #38

Closed

Optional keine Logs bei Admins aufzeichnen #31

Closed

skerbis added 2 commits March 6, 2026 17:58

docs: README aktualisieren – neuer Namespace, BC-Hinweis, disable_for…

b1f891c

…_admins

fix: NullPointerException in Activity::log() bei fehlendem User-Konte…

c10f098

…xt (Cronjob)

Copilot AI reviewed Mar 6, 2026

View reviewed changes

skerbis added 3 commits March 6, 2026 18:22

fix: Namespace von FriendsOfREDAXO auf FriendsOfRedaxo korrigiert

294cc40

feat: Cronjob und clearEntries() um Source-Filterung erweitert (Issue #…

d2d0892

…21)

This was referenced Mar 6, 2026

Löschung über Cronjobs erweitern um Zeitraum und später vielleicht namespace_quelle #21

Closed

Namespaces um die Quellen unterscheiden zu können #17

Closed

skerbis added 13 commits March 6, 2026 18:40

fix: XSS-Schutz mit rex_escape() in EP-Handlern und Activity, unused …

b252211

…imports entfernt

fix: GitHub-Org in Workflows/package.yml, CS-Fixer-Korrekturen, Node …

36dc88c

…20 für e2e

fix: ?static → ?self in Fluent-Methoden, behebt PHPUnit-TypeError mit…

5e2f2c5

… BC-Stub rex_activity

fix: composer.lock aktualisiert

d9f6a96

fix: PHP 8.1 → 8.2 in CI-Workflows (Symfony 8.x benötigt PHP >=8.2)

bf1bd27

fix: PHP >=8.2 als Mindestanforderung in composer.json deklariert

dfa8551

Update package.yml

a628d3c

fix: CI-Workflows modernisiert (PHP 8.4, actions v4, --no-scripts, gi…

b4b7a83

…t apply entfernt)

chore: .gitignore aktualisiert (.DS_Store, redaxo_cms/, .phpunit.cache/)

b870d54

docs: README ausführlich überarbeitet, veraltete diff-Datei entfernt

b5dba83

docs: Autor- und Credits-Abschnitt hinzugefügt

448c985

fix: User-Button nur für Admins, Quelle-Spalte nach Datum verschoben

186d69b

docs: README doppelten Inhalt entfernt, CHANGELOG ergaenzt

e9d8a24

skerbis added 2 commits March 6, 2026 19:30

feat: Cronjob Source-Auswahl dynamisch aus DB laden statt statisch ha…

ff6d098

…rdcodiert

docs: CI-Badges aus README entfernt

12bed0b

skerbis merged commit 582fa09 into master Mar 6, 2026
3 checks passed

skerbis deleted the feat/namespace-friendsofredaxo branch March 6, 2026 18:38

Navigation Menu

Uh oh!

feat: Namespace FriendsOfREDAXO/ActivityLog, Bugfixes & Dependency-Updates (v1.0.0-beta1) - #47

feat: Namespace FriendsOfREDAXO/ActivityLog, Bugfixes & Dependency-Updates (v1.0.0-beta1)#47
skerbis merged 24 commits into
masterfrom
feat/namespace-friendsofredaxo

skerbis commented Mar 6, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

skerbis commented Mar 6, 2026

Zusammenfassung

Änderungen

🏷️ Namespace-Migration (#38)

🐛 Fix: clang_id Undefined array key (#46)

🐛 Fix: rex_sql_exception beim Löschen von Artikeln/Kategorien

✅ Keine Logs für Admins (#31)

📦 Dependency-Updates (ersetzt PRs #41–#45)

Closes / References

Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

🐛 Fix: `clang_id` Undefined array key (#46)

🐛 Fix: `rex_sql_exception` beim Löschen von Artikeln/Kategorien