Trivial extension, should really just exist in core.
Will see around to getting it done
Version: unspecified
Severity: enhancement
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=53106
Description
Description
Details
Details
- Reference
- bz27841
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T28751 Extensions that should really be core functionality (tracking) | |||
| Resolved | Legoktm | T29841 Merge AssertEdit extension into MediaWiki core |
Event Timeline
Comment Actions
Change 90263 had a related patch set uploaded by Legoktm:
Merge AssertEdit extension into core
Comment Actions
(In reply to comment #5)
https://www.mediawiki.org/wiki/Extension:AssertEdit needs updating.
Done. I also created [[mw:API:Assert]] to document the core functionality but I'm not really sure where to link it from.
Comment Actions
(In reply to comment #6)
(In reply to comment #5)
https://www.mediawiki.org/wiki/Extension:AssertEdit needs updating.
Done. I also created [[mw:API:Assert]] to document the core functionality but
I'm not really sure where to link it from.
Thanks! I updated [[mw:Template:API]] and added the template to [[mw:API:Assert]]. I also linked this new API page from the extension notice at [[mw:Extension:AssertEdit]].
Comment Actions
Unintended consequence: passing a blank assert= parameter to the API now causes an unknown_assert error, whereas before it was silently ignored. Bug or feature?
Comment Actions
(In reply to comment #8)
Unintended consequence: passing a blank assert= parameter to the API now
causes
an unknown_assert error, whereas before it was silently ignored. Bug or
feature?
I don't consider this to be a bug since you shouldn't be passing blank parameters like that so I guess it can be considered a "feature".
Comment Actions
(In reply to comment #9)
(In reply to comment #8)
Unintended consequence: passing a blank assert= parameter to the API now
causes
an unknown_assert error, whereas before it was silently ignored. Bug or
feature?I don't consider this to be a bug since you shouldn't be passing blank
parameters like that so I guess it can be considered a "feature".
Fair enough. Can we have a "blank_assert" (or somesuch) rather than "unknown_assert" then? This is a breaking change to some reusers, so giving them more debug information may be helpful.
Comment Actions
The error is coming from the API parameter parsing, before it ever gets to the point where the assert code sees it. I'm inclined to say "no" here, since the documented values are "bot" or "user" ("" is not included) and the error message is correct.
Comment Actions
(In reply to comment #11)
The error is coming from the API parameter parsing, before it ever gets to
the point where the assert code sees it.
I don't really care a great deal, I just noted that it was an (unintended?) change in behaviour that has already forced several API users to rewrite (admittedly lazy) code. Incidentally https://www.mediawiki.org/wiki/API:Assert could do with expansion on this point.