Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Add our ZObject title requirements using the getUserPermissionsErrors hook | mediawiki/extensions/WikiLambda | master | +37 -0 |
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | Jdforrester-WMF | T277077 Z0 should never be stored as a ZID in a ZObject | |||
Resolved | gengh | T273541 Provide a ZObjectStore DAO service based on Wikipage and use it exclusively |
Event Timeline
So I can confirm that edits through action=edit or the API both result in the wikilambda-invalidzobjecttitle error, as expected.
When we move this code into the DAO service we might use a secondary hook like EditFilter to sit alongside MultiContentSave, but that seems like it shouldn't be needed?
Whilst I'm here, though, let's add it to the permission ban list via getUserPermissionsErrors.
Change 670582 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/WikiLambda@master] Add our ZObject title requirements using the getUserPermissionsErrors hook
Change 670582 merged by jenkins-bot:
[mediawiki/extensions/WikiLambda@master] Add our ZObject title requirements using the getUserPermissionsErrors hook
Maybe more fundamentally the regex matches should all be adjusted so it's looking for Z[1-9]\d* rather than Z\d+ as a valid Z id?
All the PHP-side regexs do this already. I think we're a little looser on the Vue side, and we actively use Z0 as a placeholder for new ZObject creation there…