Last modified: 2013-11-14 14:32:24 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T46057, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 44057 - Content interface should specify a validate() method
Content interface should specify a validate() method
Status: ASSIGNED
Product: MediaWiki
Classification: Unclassified
ContentHandler (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Ori Livneh
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2013-01-17 05:45 UTC by Ori Livneh
Modified: 2013-11-14 14:32 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Ori Livneh 2013-01-17 05:45:58 UTC 
Via Wikitech-l:

On Sunday, January 13, 2013 at 4:36 AM, Daniel Kinzler wrote:

> On 13.01.2013 01:35, Ori Livneh wrote:

> > * The way validation is handled is a bit mysterious. Content defines
> > an isValid interface and (if I recall correctly) a return value of
> > false would prevent the content from getting saved. But in such
> > cases you want a helpful error.

> You are right, it would be better to have a validate() method that
> returns a Status object. isValid() could then just call that and return
> $status->isOK(), for compatibility. If you like, file a bug for that -
> or just write it :)
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links