Right now once T117031 is implemented if unit conversion is configured it is automatically used. We may want to have more fine-grained configuration, i.e. some option for disabling/enabling it, accounting for PRODUCE_NORMALIZED_VALUES and maybe other configuration values.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Use flavor flags to configure builder | mediawiki/extensions/Wikibase | master | +68 -108 |
Event Timeline
Comment Actions
We probably need to configure rdf-builder-factory-callback for VT:quantity somehow depending on flavor flags. The problem is flavor flags are not accessible there and not passed to ValueSnakRdfBuilderFactory either, so we'd have to either add parameter or maybe modify $mode parameter to be more complex.
@daniel, any ideas on how to make it best?
Comment Actions
@Smalyshev maybe we can just pass the GENERATE_XXX flags as $mode, what do you think? The flags should cover the current information in $mode anyway.
Comment Actions
Yes, this should be possible, though requires changing all code that uses $mode, but it's not too much right now. I'll look into it later.
Comment Actions
Change 309227 had a related patch set uploaded (by Smalyshev):
Use flavor flags to configure builder