Last modified: 2014-02-26 11:16:22 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 T53799, 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 51799 - Factory for registration of formatters to be used for values of a certain data type.
Factory for registration of formatters to be used for values of a certain dat...
Status: RESOLVED WORKSFORME
Product: MediaWiki extensions
Classification: Unclassified
ValueFormatters (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
Storypoints: 8
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2013-07-22 10:28 UTC by Daniel A. R. Werner
Modified: 2014-02-26 11:16 UTC (History)
3 users (show)

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

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

Description Daniel A. R. Werner 2013-07-22 10:28:23 UTC 
The ValueFormatterFactory currently only takes formatters to be used with data values of a certain value type.

We need a mechanism to register formatters to be used for values of a certain data type. If no formatter is registered for a data type, this factory should fall back to a ValueFormatterFactory and take the formatter registered for the data type's value type.

Preferably, ValueFormatterFactory would stay untouched since it should not have to care about the concept of data types.

Also have a look at the JavaScript implementation of ValueviewExpertFactory where we allow registration of jQuery.valueview.Expert constructors to be used with a specific data type, or generic for a given data value.
Comment 1 Lydia Pintscher 2014-02-26 11:16:22 UTC 
Due to the latest code changes this is not longer relevant.
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