Repository navigation
Rename to phpbbmodders/wiki - #14
Merged
Merged
Conversation
Move from the tas2580 vendor to phpbbmodders: namespace, service IDs, route names, template namespace, language loading and the notification type. The new ext.php refuses to enable while tas2580/wiki is enabled, and on enable moves an existing install's migration history, notification type (both spellings), users' notification settings and CLI reparser resume data to the new names before the normal migration run. The wiki tables never included the vendor name and are used as they are. Also: standard phpBB Modders header on every PHP file (original tas2580 and Crizz0 notices kept; three files added on 09/23 were wrongly stamped "(c) 2026 tas2580" and now carry the phpBB Modders line only), composer metadata, PHP >=7.1.3 to match phpBB 3.3, and README upgrade steps. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tas2580/wiki→phpbbmodders/wiki: PHP namespace, service IDs (phpbbmodders.wiki.*), route names, the@phpbbmodders_wiki/template namespace, language loading, the notification type, CIEXTNAMEandcomposer.json. Page URLs (/wiki/...) don't change.ext.phphandles existing installs:tas2580/wikiis still enabledarticle_edit, or the older misspelledarticke_edit); its ID is kept, so existing notifications still workreparser:reparseCLI runphpbb_wiki_article) and permissions (u_wiki_*,m_wiki_*) never included the vendor name, so they are used as they are. Old migrations only change their namespace; their SQL describes what already ran and is left alone.add_article_description,rename_article_edit_notification, and its test) had been wrongly stamped(c) 2026 tas2580and now carry the phpBB Modders line only.composer.json: phpBB Modders asExtension Developer, tas2580 and Crizz0 →Past Developer, display namephpBB Modders Wiki, PHP>=7.1.3(phpBB 3.3's own minimum; the new code needs PHP 7), and an extra keyword.Upgrading: disable the old extension (keep its data), delete
ext/tas2580/wiki, upload this version toext/phpbbmodders/wiki, enable it, then purge the cache. If you disable from the command line, purge the cache before enabling; the CLI disable doesn't clear it.Test plan
Claude tested on a phpBB 3.3 board (SQLite, PHP 8.4) with the old extension from
maininstalled. The board had 7 articles, notification type 9 with 2 notifications, and 10 permissions. Claude added test data: subscriptions under both notification-type spellings (including duplicates for the same user and method), a leftover misspelled type row, and interrupted reparser resume data.\phpbbmodders\wiki\phpbbmodders.wiki.notification.type.article_editand enabledphpbbmodders/wikiremains/wiki, an article page and the index return 200, with noext/tas2580asset paths.phpbbmodders\wiki\notification\article_edit.reparser:reparse phpbbmodders_wiki_article --resumeresumes from the moved data.master) board: articles and migrations kept, subscriptions merged, index renders.php -lon every PHP file andcomposer validatepass.Investigated and written by Claude on behalf of William Jacoby (bonelifer).