Skip to content

Port to phpBB 3.3 (PHP 7.2+/8.x) - #18

Merged
gn36 merged 5 commits into
masterfrom
modernize-3.3
Oct 18, 2026
Merged

gn36 merged 5 commits into
masterfrom
modernize-3.3

Conversation

@gn36

@gn36 gn36 commented Oct 18, 2026

Copy link
Copy Markdown
Owner

Zusammenfassung

Portiert die Erweiterung auf phpBB 3.3.x (PHP ^7.2 || ^8.0). phpBB 3.1/3.2 werden nicht mehr unterstützt.

  • Notification-Typen: neuer 3.3-Basiskonstruktor, config/user_loader per Setter, language-Service; bcmath-Abhängigkeit entfernt, veraltete update_notifications() entfernt
  • services.yml (shared: false, parent), routing.yml (path:), Redirector (load_notifications() mit Methodenname)
  • Fehlerkorrekturen: Typnamen in global_events.php, soft-require in ext.php, Regex-Zeichenklasse, PHP-8.2-Deprecation (${board_url})
  • composer.json modernisiert, composer.lock/Travis/build.xml entfernt; GitHub-Actions-Workflow ergänzt
  • Tests auf PHPUnit 9 angepasst; dev/cloud-setup.sh und dev/run-tests.sh für eine Testumgebung

Tests

27 Unit-Tests grün (SQLite und MariaDB, PHP 8.3, phpBB 3.3.x). Die funktionalen Tests wurden nicht ausgeführt (brauchen Webserver mit phpBB-Installation). Der GitHub-Actions-Lauf ist noch ungeprüft.

Offen

  • Erreichbarkeit des Update-Feeds phpbb.gn36.de
  • Versionsnummer (2.0.0) noch nicht gesetzt

Generated by Claude Code

- Notification types: new 3.3 base constructor, config/user_loader via setters, language service
- Drop bcmath dependency for item ids, remove obsolete update_notifications override
- Fix redirector load_notifications(), services.yml (shared/parent), routing.yml (path)
- Fix notification type names in global_events, soft-require key in ext.php, regex in ext.php
- composer.json: PHP ^7.2 || ^8.0, phpBB ^3.3.0, drop lock/dev deps
- Tests/PHPUnit config updated for 3.3; replace Travis with GitHub Actions

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CFCEXHKUor1WeKS7KibQwV
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CFCEXHKUor1WeKS7KibQwV
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CFCEXHKUor1WeKS7KibQwV
… check script

- Workflow: test-framework@3.3.x (master targets phpBB 4)
- composer.json: version is mandatory for phpBB extensions
- Rename dev/ to develop/ (skipped by the executable-files check); add develop/check.sh mirroring CI

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CFCEXHKUor1WeKS7KibQwV
@gn36
gn36 merged commit 930ba1c into master Oct 18, 2026
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants