Future proof issues going into phpBB4#188
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #188 +/- ##
============================================
+ Coverage 59.19% 60.20% +1.01%
- Complexity 208 214 +6
============================================
Files 14 15 +1
Lines 767 789 +22
============================================
+ Hits 454 475 +21
- Misses 313 314 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
It still won't work on 4.0 because of |
That's correct, this should not be allowed to be installed on 4.0. This PR is trying to handle the situation when a 3.3.x board with Pages already installed, upgrades to phpBB 4 while Pages is still enabled/active. Yes if the reparser has gone through pages once, it won't again (unless new Pages are added I guess?) |
This should make the fatal issues Pages has when running in phpBB4. Of course there will be a proper non-backwards-compatible version of Pages after phpBB4 is eventually released. But these changes are intended to keep the current release of Pages working in both phpBB 3.3.x and 4 w/o generating fatal errors. These changes (at least for the text reparser cron task) may be removed/omitted from the non-backwards-compatible version of Pages that will be for phpBB 4 or greater.
Fatal issues when Pages is run under phpBB4:
dispatcher->dispatchno longer worksconfig_text)