Page MenuHomePhabricator

Check functionality and documenation of formatter URL property configuration
Closed, ResolvedPublic3 Estimated Story Points

Description

T223552#5282000

  • how toconfigure which property to use for formatting URLs
  • whether a correct documentation exist
  • update parent task with a comment pointing factgrid where to look to check/configure properly

Event Timeline

alaa_wmde set the point value for this task to 3.
alaa_wmde moved this task from Incoming to Ready to estimate on the Wikidata-Campsite board.

I followed this documentation https://www.mediawiki.org/wiki/Wikibase/Installation/Advanced_configuration#Define_links_for_external_identifiers localy and the Formatted URL did not work. It happened just like on Factgrid. I am thinking it is a bug on our side.

I followed this documentation https://www.mediawiki.org/wiki/Wikibase/Installation/Advanced_configuration#Define_links_for_external_identifiers localy and the Formatted URL did not work. It happened just like on Factgrid. I am thinking it is a bug on our side.

Given that we cache this information in the property info table (wb_property_info) on a per-property basis, you will, after changing the formatterUrlProperty setting, need to a) edit all properties that have a "formatter URL" statement on them and then b) run the jobs currently in the job queue (see https://www.mediawiki.org/wiki/Manual:Job_queue#Manual_usage).

I tested it and it works fine, the only weird part is that if you set your property to be data type of "external-identifier" the formatter won't work, you have to put it as "string":

image.png (309×827 px, 17 KB)

The documentation says exactly this too but maybe we need to support that.

I tested it and it works fine, the only weird part is that if you set your property to be data type of "external-identifier" the formatter won't work, you have to put it as "string":

image.png (309×827 px, 17 KB)

The documentation says exactly this too but maybe we need to support that.

I got the properties wrong, it actually only works when you set the data type to "external-identifier" and won't work if it's string. I will fix the documentation.

Yeah it needs to work for external identifiers.