[Wikidata-bugs] [Maniphest] [Retitled] T188086: [Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.4.0

2018-03-08 Thread thiemowmde
thiemowmde renamed this task from "[Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.3.0" to "[Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.4.0".thiemowmde updated the task description. (Show Details)
CHANGES TO TASK DESCRIPTIONList of code bases that currently use the Wikibase CodeSniffer <0.3.0, and should be updated to at least 0.3.0, better 0.4.0, if possible:...[x] #MediaWiki-extensions-InterwikiSorting https://gerrit.wikimedia.org/r/410261...[x] #Wikibase-Quality-External-Validation https://gerrit.wikimedia.org/r/409861
[] #Wikidata.org https://gerrit.wikimedia.org/r/413730
[] WikimediaBadges https://gerrit.wikimedia.org/r/413731
[] #DataValues components:...TASK DETAILhttps://phabricator.wikimedia.org/T188086EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: gerritbot, Jonas, WMDE-leszek, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T143706: Implement TermLookup based on Elastic

2018-03-05 Thread thiemowmde
thiemowmde added a comment.

@Smalyshev wrote:
I don't see where term types are defined […]


That's exactly the issue I referred to. These "term types" being strings is mostly an implementation detail of how the wb_terms table is designed. The currently specified term types can be found in TermIndexEntry. Please use these constants to make it easier to find places working with these term types.


TermIndexEntry::TYPE_LABEL = "label"
TermIndexEntry::TYPE_ALIAS = "alias"
TermIndexEntry::TYPE_DESCRIPTION = "description"


but could there be more?

We played around with ideas to have other types for #lexicographical_data, but we are most probably not going to do this. So these 3 types are all that exist.

Is Elastic implementation supposed to support them […]

Please either return no result if a caller asks for an unknown term type, or throw an exception.

Also, there's an overlap of coverage - you can use getLabel or you can use getPrefetchedTerm(... 'label' ...).

This is just one of the many problems of this combination of older and newer interfaces. Please focus on the more narrow methods (everything without $termType) if possible.

code dealing with prefetching, buffering, resolving different repos […] should all this code be reimplemented for ElasticSearch implementation of prefetchTerms or can it be somehow reused? All that code relies on TermBuffer/TermIndex now.

We might need to split the existing implementations. But I believe it is easier to start over and not touch the existing implementations. For example, you can start by implementing nothing but a TermLookup. If this works, you can add a small TermBuffer implementation on top that allows for prefetching. Since TermBuffer alone is not very useful, you might need to use the PrefetchingTermLookup interface as a decorator instead:

class PrefetchingElasticTermLookup implements PrefetchingTermLookup {

	private $lookup;

	public function __construct( ElasticTermLookup $lookup ) {
		$this->lookup = $lookup;
	}

	public function prefetchTerms( array $entityIds, array $termTypes = null, array $languageCodes = null ) {
		/* TBD */
	}

	public function getPrefetchedTerm( EntityId $entityId, $termType, $languageCode ) {
		switch ( $termType ) {
			case TermIndexEntry::TYPE_LABEL:
return $this->getLabel( $entityId, $languageCode );
			case TermIndexEntry::TYPE_DESCRIPTION:
return $this->getDescription( $entityId, $languageCode );
			default:
throw new \InvalidArgumentException( "Not defined for \$termType \"$termType\"" );
		}
	}

	public function getLabel( EntityId $entityId, $languageCode ) {
		$this->lookup->getLabel( $entityId, $languageCode );
	}

	public function getLabels( EntityId $entityId, array $languageCodes ) {
		$this->lookup->getLabels( $entityId, $languageCodes );
	}

	public function getDescription( EntityId $entityId, $languageCode ) {
		$this->lookup->getDescription( $entityId, $languageCode );
	}

	public function getDescriptions( EntityId $entityId, array $languageCodes ) {
		$this->lookup->getDescriptions( $entityId, $languageCodes );
	}

}



TermLookup methods are supposed to be dealing with entities from federated repos too, and also working on the WikibaseClient side […]

This might indeed be needed. However, I suggest to not care for these for the moment. We can add such functionality later via dispatchers, decorators, and such.TASK DETAILhttps://phabricator.wikimedia.org/T143706EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: WMDE-leszek, Smalyshev, hoo, Liuxinyu970226, Aklapper, aude, JeroenDeDauw, Tobi_WMDE_SW, thiemowmde, adrianheine, Lydia_Pintscher, Ricordisamoa, daniel, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, QZanden, LawExplorer, Vali.matei, Volker_E, Wikidata-bugs, GWicke, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Retitled] T188802: String values with leading/trailing whitespace are (inconsistently) reported as invalid

2018-03-05 Thread thiemowmde
thiemowmde renamed this task from "Adding property field Property:P742 (pseudonym) needs to ignore leading spaces" to "String values with leading/trailing whitespace are (inconsistently) reported as invalid".
TASK DETAILhttps://phabricator.wikimedia.org/T188802EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lydia_Pintscher, Aklapper, Billinghurst, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188802: Adding property field Property:P742 (pseudonym) needs to ignore leading spaces

2018-03-05 Thread thiemowmde
thiemowmde added projects: DataValues, MediaWiki-extensions-WikibaseRepository, Easy.thiemowmde triaged this task as "Low" priority.thiemowmde moved this task from incoming to needs discussion or investigation on the Wikidata board.thiemowmde added a comment.
Labels, descriptions, aliases, as well as monolingual text values are trimmed. For monolingual text values this happens in the backend via MonolingualTextParser.

For string values this is not done. Instead a validator blocks strings with leading or trailing spaces, tabs, newlines, or any other vertical whitespace. The code for this can be seen in ValidatorBuilders::getCommonStringValidators.

We did not wanted to enforce trimming on string values because whitespace can be significant and meaningful. Think of https://www.wikidata.org/wiki/Property:P487, the string property holding a Unicode character. Shouldn't it be possible to add a statement to https://www.wikidata.org/wiki/Q380933 (the Item describing the space character) that holds the space character? Because of the validator this is currently not possible. I believe having this validator in place was a good thing. But nowadays we have plenty of alternatives, most notably constrain checks. Maybe it's time to lift the hard-coded limitation?TASK DETAILhttps://phabricator.wikimedia.org/T188802WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lydia_Pintscher, Aklapper, Billinghurst, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T186726: Security review WikibaseLexeme extension

2018-03-05 Thread thiemowmde
thiemowmde triaged this task as "Normal" priority.
TASK DETAILhttps://phabricator.wikimedia.org/T186726EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Bawolff, thiemowmdeCc: gerritbot, Aklapper, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Lydia_Pintscher, WMDE-leszek, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, Cinemantique, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, EBjune, LawExplorer, Lewizho99, Maathavan, dpatrick, Luke081515, Wikidata-bugs, aude, JanZerebecki, Darkdadaah, csteipp, Mbch331, Jay8g, Legoktm___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T185697: mwext-phpunit-coverage-publish for WikibaseQualityExternalValidation fails

2018-03-05 Thread thiemowmde
thiemowmde triaged this task as "Lowest" priority.thiemowmde added a project: Need-volunteer.
TASK DETAILhttps://phabricator.wikimedia.org/T185697EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Legoktm, gerritbot, greg, Lucas_Werkmeister_WMDE, Aklapper, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, Scott_WUaS, abian, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T185609: Invalid callback WikibaseQuality\ExternalValidation\WikibaseQualityExternalValidationHooks::onCreateSchema

2018-03-05 Thread thiemowmde
thiemowmde triaged this task as "Lowest" priority.thiemowmde added a project: Need-volunteer.
TASK DETAILhttps://phabricator.wikimedia.org/T185609EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: gerritbot, Lucas_Werkmeister_WMDE, Umherirrender, Legoktm, Aklapper, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, Scott_WUaS, abian, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T186726: Security review WikibaseLexeme extension

2018-03-05 Thread thiemowmde
thiemowmde added a project: Wikidata-Sprint-2018-02-28.thiemowmde moved this task from ready to go to in progress on the Wikidata board.
TASK DETAILhttps://phabricator.wikimedia.org/T186726WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Bawolff, thiemowmdeCc: gerritbot, Aklapper, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Lydia_Pintscher, WMDE-leszek, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, Cinemantique, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, EBjune, LawExplorer, Lewizho99, Maathavan, dpatrick, Luke081515, Wikidata-bugs, aude, JanZerebecki, Darkdadaah, csteipp, Mbch331, Jay8g, Legoktm___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T186726: Security review WikibaseLexeme extension

2018-03-05 Thread thiemowmde
thiemowmde added a comment.
I uploaded fixes for two issues:


 https://gerrit.wikimedia.org/r/416406
 https://github.com/wmde/php-vuejs-templating/pull/6
 FormIdFormatter and SenseIdFormatter are dummy implementations. The escaping is fine. I created T188899: [Task] Remove hard-coded demo data from Lexeme code base to properly deal with these later.
 comma-separator is a message from core, and intentionally contains HTML (but no wiki syntax). As far as I can see we are using it correctly. Yes, we are aware messages with HTML are problematic, and we avoid it whenever possible.
 Click-jacking prevention is a general issue for all of #Wikidata and should have it's own ticket.
TASK DETAILhttps://phabricator.wikimedia.org/T186726EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Bawolff, thiemowmdeCc: gerritbot, Aklapper, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Lydia_Pintscher, WMDE-leszek, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, Cinemantique, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, EBjune, LawExplorer, Lewizho99, Maathavan, dpatrick, Luke081515, Wikidata-bugs, aude, JanZerebecki, Darkdadaah, csteipp, Mbch331, Jay8g, Legoktm___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188899: [Task] Remove hard-coded demo data from Lexeme code base

2018-03-05 Thread thiemowmde
thiemowmde added a parent task: T168260: Deploy WikibaseLexeme extension on Wikimedia cluster.
TASK DETAILhttps://phabricator.wikimedia.org/T188899EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: RazShuty, Ladsgroup, Jonas, Aleksey_WMDE, WMDE-leszek, Lydia_Pintscher, thiemowmde, Aklapper, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T168260: Deploy WikibaseLexeme extension on Wikimedia cluster

2018-03-05 Thread thiemowmde
thiemowmde added a subtask: T188899: [Task] Remove hard-coded demo data from Lexeme code base.
TASK DETAILhttps://phabricator.wikimedia.org/T168260EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Aklapper, daniel, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Created] T188899: [Task] Remove hard-coded demo data from Lexeme code base

2018-03-05 Thread thiemowmde
thiemowmde created this task.thiemowmde added projects: Wikidata, Lexicographical data, Technical-Debt.Herald added a subscriber: Aklapper.
TASK DESCRIPTIONIn T186726: Security review WikibaseLexeme extension the #security-reviews team stumbled across hard-coded demo data in the Lexeme code base. Before T168260: Deploy WikibaseLexeme extension on Wikimedia cluster these must all be removed, and possibly replaced with proper implementations:


Everything in the directory src/DemoData:
AskOut1Populator, AskOut2Populator, AskOut3Populator, HardLexemePopulator, LeiterLexemePopulator, and usages:
Currently exclusively used in LexemeContent.

Id.php is a set of constants.
Used in above populators.
Also directly used in LexemeContent.


FormIdFormatter.php and SenseIdFormatter.php are basically nothing but dummies with hard-coded demo-data.
resources/experts/Form.js and resources/experts/Sense.js contain the same demo data.

TASK DETAILhttps://phabricator.wikimedia.org/T188899EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: RazShuty, Ladsgroup, Jonas, Aleksey_WMDE, WMDE-leszek, Lydia_Pintscher, thiemowmde, Aklapper, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Raised Priority] T188378: Error for some Lua modules "The time allocated for running scripts has expired"

2018-03-05 Thread thiemowmde
thiemowmde raised the priority of this task from "Lowest" to "Normal".thiemowmde moved this task from incoming to needs discussion or investigation on the Wikidata board.thiemowmde added a project: Performance.thiemowmde added a comment.
What generally changed since November 2015 is:


The Items are all bigger now. Here are the diffs for two randomly picked example Items from the California maintenance list: https://www.wikidata.org/w/index.php?title=Q400522=626131991=310722154, https://www.wikidata.org/w/index.php?title=Q805375=594954459=275088526.
Way more pages use way more information from Wikidata, which means the available CPU resources need to be shared between way more processes often running at the same time.


The next step in this task should be to recreate an example use case on a local system, and profile the code involved to find bottlenecks. Maybe it's wb_terms again? Quite possible.TASK DETAILhttps://phabricator.wikimedia.org/T188378WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ladsgroup, happy5214, Rschen7754, Mr.Ibrahem, hoo, thiemowmde, Lydia_Pintscher, Lea_Lacroix_WMDE, Aklapper, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, LawExplorer, Vali.matei, Volker_E, Wikidata-bugs, aude, GWicke, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T74157: [Story] Use rc_source and drop RC_TYPE

2018-03-05 Thread thiemowmde
thiemowmde added a project: Wikidata-Sprint-2018-02-28.thiemowmde moved this task from needs discussion or investigation to in progress on the Wikidata board.
TASK DETAILhttps://phabricator.wikimedia.org/T74157WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ladsgroup, gerritbot, PokestarFan, Liuxinyu970226, Ricordisamoa, Krenair, Lydia_Pintscher, aude, Aklapper, bd808, EBernhardson, Quiddity, Spage, Mattflaschen-WMF, DannyH, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, 45Jayjay1969, Th3d3v1ls, Ramalepe, Liugev6, QZanden, EnricoCNC, LawExplorer, Lewizho99, Maathavan, Trizek-WMF, SBisson, Wikidata-bugs, Gryllida, jeblad, Catrope, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T179793: Consider dropping the "wb_items_per_site.wb_ips_site_page" index

2018-03-04 Thread thiemowmde
thiemowmde added a project: Wikidata-Sprint-2018-02-28.thiemowmde moved this task from incoming to in progress on the Wikidata board.
TASK DETAILhttps://phabricator.wikimedia.org/T179793WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: hoo, thiemowmdeCc: jcrespo, Stashbot, Marostegui, Aklapper, Ladsgroup, Multichill, daniel, hoo, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Minhnv-2809, Maathavan, Luke081515, Wikidata-bugs, aude, Mbch331, Jay8g, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188182: Special:Nearby no longer shows descriptions on Wikidata

2018-03-04 Thread thiemowmde
thiemowmde added subscribers: pmiazga, Lea_Lacroix_WMDE.thiemowmde added a comment.
Looks like said MFEnableWikidataDescriptions flag was introduced via https://gerrit.wikimedia.org/r/402126 (T182362) and https://gerrit.wikimedia.org/r/408357 (T182217) some weeks ago. But since this is disabled by default and the configuration not updated, the feature was effectively disabled. Was this intentional? Why is one feature hidden behind two almost identical configuration flags now?TASK DETAILhttps://phabricator.wikimedia.org/T188182EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lea_Lacroix_WMDE, pmiazga, Jdlrobson, thiemowmde, Lydia_Pintscher, aude, Aklapper, Lahi, Gq86, mojoaxel, GoranSMilovanovic, Jrbranaa, QZanden, LawExplorer, Winter, Wong128hk, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188182: Special:Nearby no longer shows descriptions on Wikidata

2018-03-02 Thread thiemowmde
thiemowmde added subscribers: Lydia_Pintscher, thiemowmde.thiemowmde added projects: Regression, MobileFrontend, Wikidata.org.thiemowmde triaged this task as "Low" priority.thiemowmde moved this task from incoming to needs discussion or investigation on the Wikidata board.thiemowmde added a comment.
The API query that runs at the moment looks like this: https://www.wikidata.org/w/api.php?action="">

Notice the parameter wbptterms=label does miss the …|description. The global configuration wgMFDisplayWikibaseDescriptions is supposed to enable this. See MobileFrontend/resources/mobile.search.util/extendSearchParams.js for the code responsible for this. As far as I can tell from what I see in the mediawiki-config repository it is currently enabled, but mw.config.get( 'wgMFDisplayWikibaseDescriptions' ) on wikidata.org shows it is not. Further investigation is needed to find this configuration bug.TASK DETAILhttps://phabricator.wikimedia.org/T188182WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lydia_Pintscher, aude, Aklapper, Lahi, Gq86, mojoaxel, GoranSMilovanovic, Jrbranaa, QZanden, LawExplorer, Wong128hk, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T184948: limit page creation rate on Wikidata

2018-03-02 Thread thiemowmde
thiemowmde triaged this task as "Normal" priority.
TASK DETAILhttps://phabricator.wikimedia.org/T184948EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: hoo, Sjoerddebruin, Addshore, Ladsgroup, gerritbot, Aklapper, Lydia_Pintscher, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188629: Completely remove support for detail and detailHTML in wbcheckconstraints response

2018-03-02 Thread thiemowmde
thiemowmde triaged this task as "Low" priority.thiemowmde added a project: Technical-Debt.thiemowmde moved this task from incoming to ready to go on the Wikidata board.
TASK DETAILhttps://phabricator.wikimedia.org/T188629WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lucas_Werkmeister_WMDE, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Agabi10, Wikidata-bugs, aude, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T150690: Special:Pages using property/P

2018-03-02 Thread thiemowmde
thiemowmde triaged this task as "Low" priority.thiemowmde added a project: Need-volunteer.
TASK DETAILhttps://phabricator.wikimedia.org/T150690EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Mikey641, IKhitron, Aklapper, eranroz, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, Rafisamra2016, QZanden, Tal-WMIL, Zoranzoki21, LawExplorer, Avner, Tzafrir, Wikidata-bugs, aude, Amire80, TheDJ, Matanya, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T134731: Mark changes to non-local sitelinks as minor

2018-03-02 Thread thiemowmde
thiemowmde triaged this task as "Low" priority.thiemowmde added a project: Need-volunteer.
TASK DETAILhttps://phabricator.wikimedia.org/T134731EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Aklapper, jeblad, Zppix, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188728: Replace claim with violation message in Special:ConstraintReport

2018-03-02 Thread thiemowmde
thiemowmde added subscribers: Jan_Dittrich, Lydia_Pintscher, thiemowmde.thiemowmde triaged this task as "Normal" priority.thiemowmde added a project: WMDE-Design.thiemowmde added a comment.Herald added a project: Design.
I'm very happy with the proposed change for multiple reasons:


The (in my opinion) misplaced question mark goes away.
The actual error becomes visible right away.
What the special page shows becomes much more similar to what can be seen on the Entity pages.
Removing the claim allows us to get rid of a lot of special-case code that exists only for this special page.


I would love to have a brief look by @Jan_Dittrich's #wmde-design team. Is this a good direction that fits to the designs previously done for #wikibase-quality-constraints?TASK DETAILhttps://phabricator.wikimedia.org/T188728EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lydia_Pintscher, Jan_Dittrich, Aklapper, Lucas_Werkmeister_WMDE, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Agabi10, JGirault, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188722: Remove Moldovan Wikipedia from WDQS

2018-03-02 Thread thiemowmde
thiemowmde added subscribers: daniel, thiemowmde, Addshore.thiemowmde moved this task from incoming to needs discussion or investigation on the Wikidata board.thiemowmde triaged this task as "Low" priority.thiemowmde added a project: MediaWiki-extensions-WikibaseRepository.thiemowmde added a comment.
From what I see, the following conditions must be met for this triple to appear in a dump:


wikidata.org still contains an Item with a sitelink that actually points to a page on mo.wikipedia.org. Weirdly enough I can't find any: https://query.wikidata.org/#SELECT%20%3Fitem%20%3Fpage%20WHERE%20%7B%0A%3Fpage%20schema%3Aabout%20%3Fitem%3B%0Aschema%3AisPartOf%20%3Chttps%3A%2F%2Fmo.wikipedia.org%2F%3E%0A%7D%20LIMIT%2010
The wiki is still in the sites table.


The actual bug here seems to be that the Wikibase code still thinks "mowiki" is a valid wiki, while it is a redirect in reality. You can see that "mo" can still be found when trying to add a sitelink, but doesn't work properly.

There are 10 (!) calls to \SiteLookup::getSites in the Wikibase code, and they all have this issue. Unfortunately it seems the sites table does not contain the information if a domain is a redirect. Where is this stored?TASK DETAILhttps://phabricator.wikimedia.org/T188722WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Addshore, thiemowmde, daniel, Strainu, Aklapper, VIGNERON, WikidataFacts, Lahi, Gq86, Darkminds3113, Lucas_Werkmeister_WMDE, GoranSMilovanovic, QZanden, EBjune, merbst, LawExplorer, Avner, Gehel, Jonas, FloNight, Xmlizer, jkroll, Smalyshev, Wikidata-bugs, Jdouglas, aude, Tobias1984, Manybubbles, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188381: Avoid joining three tables just to get the latest revision ID

2018-03-02 Thread thiemowmde
thiemowmde triaged this task as "Low" priority.thiemowmde added a project: Technical-Debt.
TASK DETAILhttps://phabricator.wikimedia.org/T188381EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDE, thiemowmdeCc: gerritbot, daniel, Lucas_Werkmeister_WMDE, Aklapper, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, Agabi10, Wikidata-bugs, aude, ArielGlenn, He7d3r, Mbch331, Jay8g, greg___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Closed] T188735: Violation messages on Special:ConstraintReport don’t include links

2018-03-02 Thread thiemowmde
thiemowmde added a project: Wikidata-Sprint-2018-02-28.thiemowmde closed this task as "Resolved".thiemowmde moved this task from incoming to in progress on the Wikidata board.thiemowmde assigned this task to Lucas_Werkmeister_WMDE.
TASK DETAILhttps://phabricator.wikimedia.org/T188735WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDE, thiemowmdeCc: gerritbot, Lucas_Werkmeister_WMDE, Aklapper, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, Agabi10, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188279: Investigate optimzing wb_terms

2018-03-02 Thread thiemowmde
thiemowmde triaged this task as "Normal" priority.thiemowmde added a comment.

The term_entity_type is not needed any more in Wikibase code, as far as I'm aware of. Third parties can extract the entity type from the first letter of term_full_entity_id, if needed.
The mapping for term_type should live in code, in my opinion. E.g. as constants in the TermIndex interface.
I would not "normalize" term_lang further. It's a language code, it's short, and the allowed values potentially change constantly. If anything, this should be a primary key to the sites table. But the primary key there is also a string, and it's even longer, so there would be no benefit with regards of size.
TASK DETAILhttps://phabricator.wikimedia.org/T188279EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lucas_Werkmeister_WMDE, Aklapper, WMDE-leszek, RazShuty, daniel, Lydia_Pintscher, Ladsgroup, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Changed Subscribers] T143706: Implement TermLookup based on Elastic

2018-03-02 Thread thiemowmde
thiemowmde edited subscribers, added: WMDE-leszek; removed: Rzuwig, JanZerebecki.thiemowmde added a comment.
Yea, there is an interface violation in WikibaseClient::getPrefetchingTermLookup as well as WikibaseRepo::getPrefetchingTermLookup. I uploaded https://gerrit.wikimedia.org/r/415816 as a reminder. For what this ticket here is about you should ignore this and the separate TermBuffer interface and only care about classes implementing PrefetchingTermLookup.

Please try to avoid fiddling with TermIndex, if possible. It is way to heavily bound to the wb_terms table. I feel it exposes to much of the wb_terms table structure, as well as details of the WHERE clause.

When looking at classes like BufferingTermLookup I have similar concerns. It is way to heavily bound to TermIndex, and not worth fiddling with.

I would recommend sticking to the PrefetchingTermLookup interface, and possibly creating a new caching layer that is different from BufferingTermLookup.

@daniel, what do you think?TASK DETAILhttps://phabricator.wikimedia.org/T143706EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: WMDE-leszek, Smalyshev, hoo, Liuxinyu970226, Aklapper, aude, JeroenDeDauw, Tobi_WMDE_SW, thiemowmde, adrianheine, Lydia_Pintscher, Ricordisamoa, daniel, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, QZanden, LawExplorer, Vali.matei, Volker_E, Wikidata-bugs, GWicke, Mbch331, Jay8g, Rzuwig, JanZerebecki___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T186726: Security review WikibaseLexeme extension

2018-03-02 Thread thiemowmde
thiemowmde added a comment.
I added the WIP originally. The only open dependency is a separate security review of https://github.com/wmde/php-vuejs-templating, which needs a separate #security-reviews ticket. Everything else is resolved, so this is ready to go from my point of view. :-)TASK DETAILhttps://phabricator.wikimedia.org/T186726EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Bawolff, thiemowmdeCc: Aklapper, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Lydia_Pintscher, WMDE-leszek, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, EBjune, LawExplorer, dpatrick, Luke081515, Wikidata-bugs, aude, JanZerebecki, Darkdadaah, csteipp, Mbch331, Jay8g, Legoktm___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T71753: [Story] Wikibase / Wikidata support on Wikiquote

2018-03-01 Thread thiemowmde
thiemowmde closed subtask T71752: Refactor EntityContent (tracking) as "Invalid".
TASK DETAILhttps://phabricator.wikimedia.org/T71753EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Yair_rand, PokestarFan, Liuxinyu970226, Micru, Ash_Crow, Nemo_bis, Ricordisamoa, Aklapper, Wikidata-bugs, Bene, Lydia_Pintscher, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Closed] T71752: Refactor EntityContent (tracking)

2018-03-01 Thread thiemowmde
thiemowmde closed this task as "Invalid".thiemowmde added a comment.
Yes and no. Refactoring EntityContent is an ongoing task that will never end. We don't need this ticket to do this, especially since this tickets description is quite vague and not really actionable.TASK DETAILhttps://phabricator.wikimedia.org/T71752EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Aklapper, GPHemsley, Bene, JeroenDeDauw, Lydia_Pintscher, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T188599: Add monolingual language code sju

2018-03-01 Thread thiemowmde
thiemowmde added a comment.
Adding language codes generally needs consensus by the language committee, which is a part of the community, as far as I'm aware of.

Yes, the patch is already uploaded – by a volunteer. Therefor the tag.TASK DETAILhttps://phabricator.wikimedia.org/T188599EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: jhsoby-WMNO, thiemowmdeCc: thiemowmde, Nikerabbit, gerritbot, jhsoby-WMNO, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, Luke081515, SimmeD, Wikidata-bugs, Snowolf, aude, Shizhao, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188378: Error for some Lua modules "The time allocated for running scripts has expired"

2018-03-01 Thread thiemowmde
thiemowmde triaged this task as "Lowest" priority.thiemowmde added projects: Need-volunteer, MediaWiki-extensions-WikibaseClient.thiemowmde added a comment.
Scratching the limits is basically the point of these maintenance pages. Whenever something like this is done, it is pretty much guaranteed there will be a few extreme outliers. The first example provided loads 300+ Wikidata items, and for each:


Loads all labels in all languages
Checks if 13 specific properties are present, and filled in properly


The configuration for this can be seen in https://www.wikidata.org/wiki/Module:Roadtable/models.

The main code is in https://www.wikidata.org/wiki/Module:Roadtable. It contains a problematic mw.wikibase.getEntityObject that might be worth replacing with more specific methods like getBestStatements or the new getAllStatements (T166056).TASK DETAILhttps://phabricator.wikimedia.org/T188378EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: hoo, thiemowmde, Lydia_Pintscher, Lea_Lacroix_WMDE, Aklapper, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T114903: Migrate wb_terms to using prefixed entity IDs instead of numeric IDs

2018-03-01 Thread thiemowmde
thiemowmde removed a project: Patch-For-Review.
TASK DETAILhttps://phabricator.wikimedia.org/T114903EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ladsgroup, Stashbot, gerritbot, PokestarFan, Pasleim, Ricordisamoa, JanZerebecki, hoo, daniel, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, Marostegui, LawExplorer, Minhnv-2809, Luke081515, Wikidata-bugs, aude, Mbch331, Jay8g, Krenair, Giuliamocci, Adrian1985, Cpaulf30, Baloch007, Darkminds3113, Lordiis, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, Lewizho99, Maathavan___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T114903: Migrate wb_terms to using prefixed entity IDs instead of numeric IDs

2018-03-01 Thread thiemowmde
thiemowmde triaged this task as "Normal" priority.thiemowmde added a project: Technical-Debt.
TASK DETAILhttps://phabricator.wikimedia.org/T114903EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ladsgroup, Stashbot, gerritbot, PokestarFan, Pasleim, Ricordisamoa, JanZerebecki, hoo, daniel, Aklapper, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, Marostegui, LawExplorer, Lewizho99, Minhnv-2809, Maathavan, Luke081515, Wikidata-bugs, aude, Mbch331, Jay8g, Krenair___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Retitled] T188534: [Bug] Format constraint incorrectly interprets regexes containing unwrapped "|"

2018-03-01 Thread thiemowmde
thiemowmde renamed this task from "Format constraint incorrectly interprets regexes containing unwrapped "|"" to "[Bug] Format constraint incorrectly interprets regexes containing unwrapped "|"".thiemowmde added a subscriber: Jonas.
TASK DETAILhttps://phabricator.wikimedia.org/T188534EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, thiemowmde, Lucas_Werkmeister_WMDE, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, Agabi10, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188534: Format constraint incorrectly interprets regexes containing unwrapped "|"

2018-03-01 Thread thiemowmde
thiemowmde triaged this task as "Normal" priority.thiemowmde added a project: Easy.thiemowmde moved this task from incoming to consider for next sprint on the Wikidata board.thiemowmde added a comment.
Yea, simply add (?:…) around all regexes, no matter what they contain. This is always fine.

The code for this can be found in \WikibaseQuality\ConstraintReport\ConstraintCheck\Helper\SparqlHelper::matchesRegularExpressionWithSparql.TASK DETAILhttps://phabricator.wikimedia.org/T188534WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lucas_Werkmeister_WMDE, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, Agabi10, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188580: Add monolingual language code smn

2018-03-01 Thread thiemowmde
thiemowmde triaged this task as "Low" priority.thiemowmde added projects: Easy, Community-consensus-needed, Need-volunteer, MediaWiki-extensions-WikibaseRepository.
TASK DETAILhttps://phabricator.wikimedia.org/T188580EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: jhsoby-WMNO, thiemowmdeCc: gerritbot, jhsoby-WMNO, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, Luke081515, SimmeD, Wikidata-bugs, Snowolf, aude, Shizhao, Nikerabbit, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188579: Add monolingual language code sms

2018-03-01 Thread thiemowmde
thiemowmde added projects: Easy, Community-consensus-needed, Need-volunteer, MediaWiki-extensions-WikibaseRepository.thiemowmde triaged this task as "Low" priority.
TASK DETAILhttps://phabricator.wikimedia.org/T188579EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: jhsoby-WMNO, thiemowmdeCc: gerritbot, jhsoby-WMNO, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, Luke081515, SimmeD, Wikidata-bugs, Snowolf, aude, Shizhao, Nikerabbit, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188596: Add monolingual language code sjd

2018-03-01 Thread thiemowmde
thiemowmde triaged this task as "Low" priority.thiemowmde added projects: Need-volunteer, Easy, Community-consensus-needed, MediaWiki-extensions-WikibaseRepository.
TASK DETAILhttps://phabricator.wikimedia.org/T188596EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: jhsoby-WMNO, thiemowmdeCc: Nikerabbit, gerritbot, jhsoby-WMNO, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, Luke081515, SimmeD, Wikidata-bugs, Snowolf, aude, Shizhao, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188599: Add monolingual language code sju

2018-03-01 Thread thiemowmde
thiemowmde added a project: MediaWiki-extensions-WikibaseRepository.
TASK DETAILhttps://phabricator.wikimedia.org/T188599EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: jhsoby-WMNO, thiemowmdeCc: Nikerabbit, gerritbot, jhsoby-WMNO, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, Luke081515, SimmeD, Wikidata-bugs, Snowolf, aude, Shizhao, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188599: Add monolingual language code sju

2018-03-01 Thread thiemowmde
thiemowmde added projects: Community-consensus-needed, Easy, Need-volunteer.thiemowmde triaged this task as "Low" priority.
TASK DETAILhttps://phabricator.wikimedia.org/T188599EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: jhsoby-WMNO, thiemowmdeCc: Nikerabbit, gerritbot, jhsoby-WMNO, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, Luke081515, SimmeD, Wikidata-bugs, Snowolf, aude, Shizhao, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T139228: Better match language names when creating Wikidata items

2018-03-01 Thread thiemowmde
thiemowmde added a comment.
Oh, absolutely right. Thanks! It's just an other approach: I was more thinking in the line of teaching all suggesters to understand word boundaries, no matter what the content of the suggester is. Reusing the ULS API when searching for languages is definitely worth looking into!TASK DETAILhttps://phabricator.wikimedia.org/T139228EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Nikerabbit, gerritbot, thiemowmde, Aklapper, Zppix, Nikola_Smolenski, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, JGirault, ferdbold, Wikidata-bugs, aude, Gryllida, Ricordisamoa, TheDJ, Jdforrester-WMF, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T139228: Better match language names when creating Wikidata items

2018-03-01 Thread thiemowmde
thiemowmde added a comment.
Sure. That's not what I wanted to imply. What OOUI can do is to look at the UX pattern and apply it via it's own, independent client-side code. What a word-boundary is is defined in RegExp. OOUI can reuse this definition (being aware it will not work in languages where the idea of word-boundaries is a little different).TASK DETAILhttps://phabricator.wikimedia.org/T139228EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Nikerabbit, gerritbot, thiemowmde, Aklapper, Zppix, Nikola_Smolenski, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, JGirault, ferdbold, Wikidata-bugs, aude, Gryllida, Ricordisamoa, TheDJ, Jdforrester-WMF, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T139228: Better match language names when creating Wikidata items

2018-02-28 Thread thiemowmde
thiemowmde added projects: OOUI, WMDE-Design, Need-volunteer.thiemowmde triaged this task as "Normal" priority.thiemowmde added a comment.Herald added a project: Design.
Special:NewItem is using OOUI now, which means this needs to be filled as an upstream feature request to improve the ComboBoxInputWidget we use.

The jquery.wikibase.siteselector _javascript_ module mentioned is still used on the client, as well as sitelinkview on Item pages. That selector could also benefit from the same update.

I just noticed that the language selector in #universallanguageselector already does exactly what is suggested here. I love it and would love to see it applied to OOUI.TASK DETAILhttps://phabricator.wikimedia.org/T139228EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Aklapper, Zppix, Nikola_Smolenski, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, JGirault, ferdbold, Wikidata-bugs, aude, Gryllida, Ricordisamoa, Nikerabbit, TheDJ, Jdforrester-WMF, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Closed] T139227: Enable sorting of interwiki links in Wikibase Client

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Invalid".thiemowmde added a project: MediaWiki-extensions-InterwikiSorting.thiemowmde added a comment.
To clean our backlogs I'm going to close this for now, for the following reasons:


I don't see a problem on https://en.wikipedia.org/wiki/Main_Page.
Interwiki links on the clients are already sorted. Therefore I wonder what "enable sorting" refers to. The code was previously in the Wikibase code base, and recently moved to #mediawiki-extensions-interwikisorting. Is there an issue with that?
I don't want us to maintain an other parser function for a rare edge-case like this. Different requirements for the order should be configuration per wiki, and already are as far as I'm aware of.


If the issue is still there, please provide more information and feel free to reopen this ticket.TASK DETAILhttps://phabricator.wikimedia.org/T139227EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, aude, Aklapper, Zppix, Nikola_Smolenski, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T92365: [Story] Centralize Wikibase documentation process and assign responsibilities

2018-02-28 Thread thiemowmde
thiemowmde added subscribers: Lea_Lacroix_WMDE, thiemowmde.thiemowmde added a project: Documentation.thiemowmde added a comment.
@Lea_Lacroix_WMDE, do you think this ticket is helpful? I suggest to close it as invalid because it is barely actionable. Closing this would not mean we don't care about a better documentation process, the contrary. It's just that I think this ticket here does not help us finding a better one.TASK DETAILhttps://phabricator.wikimedia.org/T92365EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Lea_Lacroix_WMDE, Liuxinyu970226, Jc3s5h, Tobi_WMDE_SW, JeroenDeDauw, Abraham, Snaterlicious, Aklapper, Cpaulf30, Lahi, Gq86, GoranSMilovanovic, Ivana_Isadora, Jayprakash12345, QZanden, LawExplorer, Wikidata-bugs, aude, Dinoguy1000, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T89350: Impose standard on capitalizing concept terms in messages

2018-02-28 Thread thiemowmde
thiemowmde added projects: Documentation, Easy.thiemowmde added a subscriber: Lea_Lacroix_WMDE.thiemowmde added a comment.
I also vote for capitalization. This is especially important for (intentionally) vague concepts like "Entity", "Item", "Property", and "Form" (part of #lexicographical_data). Think of a form to edit a Form. Think of a property of a Property. Think of a list item linking to an Item. All these sentences start making a tiny little bit more sense if the first letter is an uppercase one.TASK DETAILhttps://phabricator.wikimedia.org/T89350EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lea_Lacroix_WMDE, Jan_Dittrich, Abraham, adrianheine, thiemowmde, Lydia_Pintscher, Aklapper, Snaterlicious, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, GoranSMilovanovic, Ivana_Isadora, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, Dinoguy1000, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T110335: [Task] Merging WikibaseQuality branches "reverted" a lot of relevant changes

2018-02-28 Thread thiemowmde
thiemowmde added a comment.
I would not walk through the diffs, but through the individual commits. As I said in this tasks description I did not spotted all changes that still might be worth cherry-picking. Spotting these is what this ticket here is about.TASK DETAILhttps://phabricator.wikimedia.org/T110335EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lucas_Werkmeister_WMDE, Lydia_Pintscher, JanZerebecki, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Agabi10, Scott_WUaS, abian, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T145932: [Bug] Special:NewItem does not validate maximum label length

2018-02-28 Thread thiemowmde
thiemowmde moved this task from incoming to ready to go on the Wikidata board.thiemowmde triaged this task as "Normal" priority.thiemowmde added a project: Technical-Debt.thiemowmde added a comment.
This is an actual bug, still relevant.TASK DETAILhttps://phabricator.wikimedia.org/T145932WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: hoo, thiemowmdeCc: Gstupp, Liuxinyu970226, Thryduulf, Mbch331, Sjoerddebruin, Aklapper, hoo, aude, daniel, Lydia_Pintscher, thiemowmde, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T118860: [RFC] Use Role Object Pattern to represent derived data in the data model

2018-02-28 Thread thiemowmde
thiemowmde closed subtask T132443: [Task] Discuss design of (De)Serializer interfaces as "Declined".
TASK DETAILhttps://phabricator.wikimedia.org/T118860EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: daniel, thiemowmdeCc: Tobi_WMDE_SW, Bene, Lydia_Pintscher, mkroetzsch, adrianheine, hoo, thiemowmde, aude, Jonas, JanZerebecki, JeroenDeDauw, Aklapper, StudiesWorld, daniel, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, TomT0m, Wong128hk, Luke081515, Wikidata-bugs, fbstj, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T132443: [Task] Discuss design of (De)Serializer interfaces

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde added a comment.
I close this for now to reduce our backlog. This does not mean these unspecific "mixed" interfaces do not need discussion any more. But we can continue working on this code whenever the need arises.TASK DETAILhttps://phabricator.wikimedia.org/T132443EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Tobi_WMDE_SW, Lydia_Pintscher, aude, hoo, adrianheine, JeroenDeDauw, daniel, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T56098: [Epic][UI] Front-end performance: Improve loading time of items

2018-02-28 Thread thiemowmde
thiemowmde closed subtask T124783: [Task] Investigate how to improve loading time of individual widgets as "Declined".
TASK DETAILhttps://phabricator.wikimedia.org/T56098EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Agabi10, Multichill, hoo, Sjoerddebruin, Jonas, Liuxinyu970226, Aklapper, tommorris, Wikidata-bugs, Bene, matmarex, Nemo_bis, thiemowmde, Tobi_WMDE_SW, He7d3r, ori, Micru, adrianheine, Snaterlicious, Ricordisamoa, MZMcBride, Quiddity, Lydia_Pintscher, Stryn, Ltrlg, daniel, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, QZanden, LawExplorer, Vali.matei, Volker_E, aude, GWicke, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T124783: [Task] Investigate how to improve loading time of individual widgets

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde added a comment.
No expectations are given. In general it is very unlikely people are ever working on the jQueryUI based parts of the code again.TASK DETAILhttps://phabricator.wikimedia.org/T124783EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Aklapper, Lydia_Pintscher, adrianheine, Jonas, thiemowmde, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, QZanden, LawExplorer, Vali.matei, Volker_E, Wikidata-bugs, aude, GWicke, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T56098: [Epic][UI] Front-end performance: Improve loading time of items

2018-02-28 Thread thiemowmde
thiemowmde closed subtask T118968: [Task] Investigate if removing of events improves loading times as "Declined".
TASK DETAILhttps://phabricator.wikimedia.org/T56098EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Agabi10, Multichill, hoo, Sjoerddebruin, Jonas, Liuxinyu970226, Aklapper, tommorris, Wikidata-bugs, Bene, matmarex, Nemo_bis, thiemowmde, Tobi_WMDE_SW, He7d3r, ori, Micru, adrianheine, Snaterlicious, Ricordisamoa, MZMcBride, Quiddity, Lydia_Pintscher, Stryn, Ltrlg, daniel, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, QZanden, LawExplorer, Vali.matei, Volker_E, aude, GWicke, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T118968: [Task] Investigate if removing of events improves loading times

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde added a comment.
I'm afraid there is nothing actionable here. That part of the jQueryUI based UI is very, very unlikely to ever be touched again.TASK DETAILhttps://phabricator.wikimedia.org/T118968EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lydia_Pintscher, adrianheine, thiemowmde, Bene, Jonas, Aklapper, Lahi, Gq86, Darkminds3113, GoranSMilovanovic, QZanden, LawExplorer, Vali.matei, Volker_E, Wikidata-bugs, aude, GWicke, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Closed] T110533: [Bug] The ValueView CI does not run its own QUnit tests

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Resolved".thiemowmde claimed this task.thiemowmde added a comment.
Fixed via T177816: Turn pseudo-extension data-values/value-view into a npm package.TASK DETAILhttps://phabricator.wikimedia.org/T110533EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ricordisamoa, Addshore, aude, adrianheine, Tobi_WMDE_SW, Lydia_Pintscher, JeroenDeDauw, JanZerebecki, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Lowered Priority] T110335: [Task] Merging WikibaseQuality branches "reverted" a lot of relevant changes

2018-02-28 Thread thiemowmde
thiemowmde lowered the priority of this task from "High" to "Low".thiemowmde added a subscriber: Lucas_Werkmeister_WMDE.thiemowmde added a comment.
@Lucas_Werkmeister_WMDE, you are currently most familiar with these code bases. Do you think it's worth looking at the diff links provided in this tasks description and see if there is anything still worth cherry-picking to the most recent version of the code base?TASK DETAILhttps://phabricator.wikimedia.org/T110335EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lucas_Werkmeister_WMDE, Lydia_Pintscher, JanZerebecki, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Agabi10, Scott_WUaS, abian, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T107864: [Task] Create ValueParserFactory integration test for the global wgValueParsers setting

2018-02-28 Thread thiemowmde
thiemowmde added a comment.
@Lydia_Pintscher, is it worth spending time on a test that makes sure …action="">, …parser=globecoordinate, and …parser=null will work forever? I mean, even if somebody removes these three lines of configuration some day, the worst thing that will happen is that some third-party services relying on our parsers might start failing. Do we care? The parser=… parameter in question is marked as deprecated for almost 3 years. Services relying on it are outdated anyway.

We are currently not tracking what datatypes a.k.a. parsers are used from the wbparsevalue module. That might be required before removing anything.TASK DETAILhttps://phabricator.wikimedia.org/T107864EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lydia_Pintscher, Tobi_WMDE_SW, daniel, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T103086: [Story] Have an "always fail" feature in WikibaseJavaScriptApi for testing purposes

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde removed a project: Story.thiemowmde added a comment.
This ticket is proposing a solution disconnected from a story, and is more of an RfC than anything else because of this. Closing for now as there are no plans to work on this. Even if, the solution would most probably be a different one then.TASK DETAILhttps://phabricator.wikimedia.org/T103086EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, adrianheine, WMDE-Fisch, Lydia_Pintscher, aude, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Lowered Priority] T99907: [RFC] Human-readable serialization of TimeValue precisions in RDF

2018-02-28 Thread thiemowmde
thiemowmde lowered the priority of this task from "High" to "Low".
TASK DETAILhttps://phabricator.wikimedia.org/T99907EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Liuxinyu970226, mkroetzsch, Smalyshev, daniel, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, TomT0m, Wong128hk, Luke081515, Wikidata-bugs, aude, fbstj, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T98468: [Story] Include example JSON blobs in API module descriptions

2018-02-28 Thread thiemowmde
thiemowmde added a project: Documentation.thiemowmde added a subscriber: Lea_Lacroix_WMDE.
TASK DETAILhttps://phabricator.wikimedia.org/T98468EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Lea_Lacroix_WMDE, Addshore, Lucie, hoo, aude, JanZerebecki, Lydia_Pintscher, thiemowmde, Aklapper, Cpaulf30, Lahi, Gq86, GoranSMilovanovic, Ivana_Isadora, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, Wikidata-bugs, Dinoguy1000, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T96659: Force or fix Packagist update trigger or get rid of Packagist dependency

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde added a comment.
I'm still running into this on a regular basis. Luckily I can work around it now because I became administrator for all packages I care about (poke me if you need to become administrator too). Since getting rid of Packagist is more unlikely than ever I'm closing this ticket for now.TASK DETAILhttps://phabricator.wikimedia.org/T96659EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Bene, Addshore, JanZerebecki, JeroenDeDauw, Tobi_WMDE_SW, Lydia_Pintscher, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T77977: [Epic] Unit support

2018-02-28 Thread thiemowmde
thiemowmde closed subtask T95425: [Bug] Quantity formatter rounding causes significant data loss as "Resolved".
TASK DETAILhttps://phabricator.wikimedia.org/T77977EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: PokestarFan, Ash_Crow, Popcorndude, Darkdadaah, Mbch331, Jc3s5h, Mike_Peel, Luke081515, Wolfvoll, -jem-, Lucie, Izno, JanZerebecki, Smalyshev, Filceolaire, jeblad, Tobias1984, Kelson, Ricordisamoa, Sylvain_WMFr, DSGalaktos, AmaryllisGardener, Liuxinyu970226, Aklapper, Snipre, daniel, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Johan, Wikidata-bugs, aude, TheDJ, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Closed] T95425: [Bug] Quantity formatter rounding causes significant data loss

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Resolved".thiemowmde claimed this task.thiemowmde added a comment.
Since https://github.com/DataValues/Number/pull/68 we are not applying artificial rounding any more when the precision is shown. \o/TASK DETAILhttps://phabricator.wikimedia.org/T95425EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Wostr, Srittau, Nikki, Thryduulf, Addshore, Lydia_Pintscher, Jc3s5h, Snipre, mgrabovsky, daniel, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T56318: Quantity datatype (tracking)

2018-02-28 Thread thiemowmde
thiemowmde closed subtask T95425: [Bug] Quantity formatter rounding causes significant data loss as "Resolved".
TASK DETAILhttps://phabricator.wikimedia.org/T56318EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: TerraCodes, jeblad, DixonD, Darkdadaah, Aklapper, Wolfvoll, Klortho, MSGJ, Kelson, Ricordisamoa, Wikidata-bugs, Bene, He7d3r, Denny, Micru, Ash_Crow, Event, Snipre, Liuxinyu970226, Lydia_Pintscher, Stryn, daniel, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T93867: [Task] Consolidate "Created a new item" summary line messages

2018-02-28 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde added a comment.Herald removed a subscriber: Liuxinyu970226.
The first two message keys are not used in the code any more, but must stay because there are still edit summaries with these message keys in the database. qqq.json describes these messages as such. Therefore consolidation is not easily possible (well, it is, but we would lose localization for some of these old summaries, and should only do so if we are sure this is ok). Closing for now as this is a really minor issue probably not worth anybodies time.TASK DETAILhttps://phabricator.wikimedia.org/T93867EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: PokestarFan, Ricordisamoa, aude, Lucie, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, Jayprakash12345, QZanden, LawExplorer, Srdjan_m, MuhammadShuaib, LNDDYL, Psychoslave, Wikidata-bugs, Gryllida, Shizhao, Arrbee, Mbch331, Jay8g, Liuxinyu970226___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T90435: [Epic] Wikidata watchlist improvements (client)

2018-02-28 Thread thiemowmde
thiemowmde closed subtask T93867: [Task] Consolidate "Created a new item" summary line messages as "Declined".
TASK DETAILhttps://phabricator.wikimedia.org/T90435EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Zache, Noella94, Elitre, PokestarFan, Tbayer, Aschroet, IKhitron, Esh77, TomT0m, Oliv0, abian, Daniel_Mietchen, Soisyc_Croisic, H4stings, Ricordisamoa, matej_suchanek, MGChecker, Sitic, JanZerebecki, Conny, Snaterlicious, Liuxinyu970226, Quiddity, Rschen7754, Ainali, MisterSynergy, Lydia_Pintscher, Aklapper, daniel, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T68651: Make last character optional in globe/geo/lat-long parsers

2018-02-28 Thread thiemowmde
thiemowmde added a comment.
I'm a big fan of parsers that do their very best to figure out what the user means. The examples provided here and not unambiguous, as far as I can tell, and can not lead to problematic edge cases. Personally I would be willing to work on this any time when approved by PM.TASK DETAILhttps://phabricator.wikimedia.org/T68651EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Wikidata-bugs, thiemowmde, Snaterlicious, Lydia_Pintscher, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Lowered Priority] T66515: [Story] Direct link to download the raw JSON data of each Entity

2018-02-28 Thread thiemowmde
thiemowmde lowered the priority of this task from "Normal" to "Low".thiemowmde added a project: Easy.thiemowmde added a comment.
I still believe this would be helpful. Even if the only thing it would do is to trigger the explorer gene in some curious people. That alone would be awesome.

This also got PMs "Go!" already.TASK DETAILhttps://phabricator.wikimedia.org/T66515EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: hoo, Aklapper, Wikidata-bugs, thiemowmde, Lydia_Pintscher, daniel, JohnLewis, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188354: Remove phpunit/phpunit dependencies from extensions's composer.json

2018-02-28 Thread thiemowmde
thiemowmde triaged this task as "Normal" priority.thiemowmde moved this task from incoming to consider for next sprint on the Wikidata board.
TASK DETAILhttps://phabricator.wikimedia.org/T188354WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, Ladsgroup, WMDE-leszek, Lydia_Pintscher, Aklapper, Umherirrender, Reedy, thiemowmde, Legoktm, Lahi, SilvioAryneutro, Gq86, Timoavax79, Cinemantique, GoranSMilovanovic, Jayprakash12345, QZanden, LawExplorer, Wikidata-bugs, aude, Ricordisamoa, Darkdadaah, zeljkofilipin, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188354: Remove phpunit/phpunit dependencies from extensions's composer.json

2018-02-28 Thread thiemowmde
thiemowmde added projects: Wikidata, Lexicographical data, MediaWiki-extensions-WikibaseMediaInfo.thiemowmde added subscribers: Lydia_Pintscher, WMDE-leszek, Ladsgroup, daniel.
TASK DETAILhttps://phabricator.wikimedia.org/T188354EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, Ladsgroup, WMDE-leszek, Lydia_Pintscher, Aklapper, Umherirrender, Reedy, thiemowmde, Legoktm, Lahi, SilvioAryneutro, Gq86, Timoavax79, Cinemantique, GoranSMilovanovic, Jayprakash12345, QZanden, LawExplorer, Wikidata-bugs, aude, Ricordisamoa, Darkdadaah, zeljkofilipin, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T188108: Investigation: Measure load times when the complete lexeme data is loaded to display lexeme in the statement

2018-02-28 Thread thiemowmde
thiemowmde added a comment.
Looks good, thanks! Since this was brought up, here are some numbers from the Wikidata Item "Germany" for comparison:


622 statements
684 qualifiers
579 non-empty references, containing 1789 snaks (that's an average of 3 snaks per reference)
3095 value snaks, including qualifiers and references snaks
TASK DETAILhttps://phabricator.wikimedia.org/T188108EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszek, thiemowmdeCc: thiemowmde, Lydia_Pintscher, daniel, Jonas, Ladsgroup, Aklapper, WMDE-leszek, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T114482: [Task] Do a "clean up open pull requests on GitHub" day

2018-02-27 Thread thiemowmde
thiemowmde closed this task as "Declined".
TASK DETAILhttps://phabricator.wikimedia.org/T114482EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ricordisamoa, hoo, aude, daniel, JanZerebecki, Jonas, Bene, JeroenDeDauw, Tobi_WMDE_SW, Lydia_Pintscher, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T68651: Make last character optional in globe/geo/lat-long parsers

2018-02-27 Thread thiemowmde
thiemowmde added projects: Need-volunteer, DataValues.thiemowmde moved this task from incoming to ready to go on the Wikidata board.
TASK DETAILhttps://phabricator.wikimedia.org/T68651WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Wikidata-bugs, thiemowmde, Snaterlicious, Lydia_Pintscher, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Declined] T65932: Entity does not exist message should link to Special:Search

2018-02-27 Thread thiemowmde
thiemowmde closed this task as "Declined".thiemowmde added a comment.
To clean our backlogs I'm going to decline my own wish. The search is always visible in the top-right corner. As far as I know nobody ever run into the described issue again, not even myself. Adding an other link to that message comes with maintenance costs that don't seem to be worth it.TASK DETAILhttps://phabricator.wikimedia.org/T65932EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ricordisamoa, Aklapper, Wikidata-bugs, thiemowmde, Lydia_Pintscher, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Unblock] T96040: Wikibase special pages (tracking)

2018-02-27 Thread thiemowmde
thiemowmde closed subtask T65932: Entity does not exist message should link to Special:Search as "Declined".
TASK DETAILhttps://phabricator.wikimedia.org/T96040EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: PokestarFan, Luke081515, Liuxinyu970226, aude, Aklapper, Lahi, Gq86, Lsherwinforone, GoranSMilovanovic, Jayprakash12345, QZanden, Ptolusque, LawExplorer, Sethakill, Wong128hk, Wikidata-bugs, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-27 Thread thiemowmde
thiemowmde added a comment.

@Legoktm wrote:
Please don't do this until someone fixes the structure test […]


As argued in https://gerrit.wikimedia.org/r/413731 I feel asking people to not do stuff isn't the best approach. The least thing needed would be a PHPCS sniff that enforces all code bases to always use the full qualified \PHPUnit\Framework\TestCase.

I can see the value of StructureTest, but I disagree with it being a technical blocker for said reason. It's not like a code base is somehow degenerating just because StructureTest does not understand it's code style any more.TASK DETAILhttps://phabricator.wikimedia.org/T188011EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Legoktm, gerritbot, Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Retitled] T178994: [Task] Consistently use interfaces in the WikibaseLexeme code base

2018-02-27 Thread thiemowmde
thiemowmde renamed this task from "[Task] Necessary refactorings to restore consistency in the WikibaseLexeme code base" to "[Task] Consistently use interfaces in the WikibaseLexeme code base".thiemowmde removed thiemowmde as the assignee of this task.thiemowmde added a subscriber: RazShuty.thiemowmde updated the task description. (Show Details)
CHANGES TO TASK DESCRIPTIONPrevious changes (e.g. https://gerrit.wikimedia.org/r/376705, which was created for and assigned to the otherwise unrelated ticket T173742) left the code base in an inconsistent state that is hard to read,Numerous previous changes left the Lexeme codebase in an inconsistent state by simply "avoiding" interfaces the Wikibase codebase and components provide for various reasons. hard to maintain,This makes the implementations in the Lexeme codebase considerably different from the established ones in the Wikibase codebase. and more and more problematic to continue working on.

The goal is to restore the consistency of the code base back to a state that enables us to work on closely related tasks from the backlogSuch code becomes harder to read and maintain, most notably editing of forms and senses and the elementsand harder to reuse in situations that type hint against one of they are made ofse interfaces.

#patch-for-review and TODOs:...[] https://gerrit.wikimedia.org/r/383812
[x] https://gerrit.wikimedia.org/r/386140
[] https://gerrit.wikimedia.org/r/386140Several classes implementing Wikibase\Lexeme\DataModel\Services\Diff\FormDiff do not fully conform to the DiffOp interface they implement. Content is hold twice in two private properties where it can even conflict with each other.TASK DETAILhttps://phabricator.wikimedia.org/T178994EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: RazShuty, gerritbot, Ladsgroup, Jakob_WMDE, Jonas, WMDE-leszek, daniel, Lydia_Pintscher, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, Cinemantique, GoranSMilovanovic, Adik2382, Th3d3v1ls, Ramalepe, Liugev6, QZanden, LawExplorer, Lewizho99, Maathavan, Wikidata-bugs, aude, Darkdadaah, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188395: [Task] Make sure all Wikibase codebases run PHPCS during `composer test`

2018-02-27 Thread thiemowmde
thiemowmde moved this task from incoming to ready to go on the Wikidata board.thiemowmde triaged this task as "Normal" priority.
TASK DETAILhttps://phabricator.wikimedia.org/T188395WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ladsgroup, Jonas, Lucas_Werkmeister_WMDE, Rzuwig, thiemowmde, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Created] T188395: [Task] Make sure all Wikibase codebases run PHPCS during `composer test`

2018-02-27 Thread thiemowmde
thiemowmde created this task.thiemowmde added projects: Wikidata, Technical-Debt, Wikibase-DataModel, Easy, Need-volunteer.Herald added a subscriber: Aklapper.
TASK DESCRIPTIONIn some Wikibase components, the shortcut composer test is set up to only run PHPUnit and a small subset of tests, but not PHPCS. This was probably an unintentional, arbitrary decision we should be able to question when other needs arise. It turned out to be unexpected because MediaWiki, other extensions and components run composer test during CI. This is expected to always include PHPCS.

These components currently miss to run PHPCS (and sometimes PHPMD) via composer test:


Diff
#DataModel
#wikibase-datamodel-serialization https://github.com/wmde/WikibaseDataModelSerialization/pull/230
#wikibase-datamodel-services
#wikibase-datamodel-services
#wikibase-internal-serialization
Purtle
TASK DETAILhttps://phabricator.wikimedia.org/T188395EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Ladsgroup, Jonas, Lucas_Werkmeister_WMDE, Rzuwig, thiemowmde, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Reopened] T185486: [Bug] wblexemeeditformelements API requires to resubmit unrelated values

2018-02-27 Thread thiemowmde
thiemowmde reopened this task as "Open".thiemowmde added a comment.
There is no such thing as a non-public API. All APIs are public. The only thing we can do is mark one as unstable (and we did), but this does not make it private. So to close this ticket I would like to know how we are going to make it obvious to users they should not use this API, and how we are possibly going to enforce this?TASK DETAILhttps://phabricator.wikimedia.org/T185486EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, WMDE-leszek, Ladsgroup, Jonas, Lydia_Pintscher, thiemowmde, Aklapper, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T156220: Wikidata autocomplete breaks with URL ...?uselang=en-US

2018-02-23 Thread thiemowmde
thiemowmde added projects: Mediawiki-extensions-PropertySuggester, Need-volunteer.thiemowmde triaged this task as "Lowest" priority.thiemowmde moved this task from incoming to needs discussion or investigation on the Wikidata board.thiemowmde added a comment.
The reason for this behavior can be seen here: https://www.wikidata.org/w/api.php?action="">. There is no language "en-US" in MediaWiki. Most code falls back to the default "en". But the API module that is responsible for suggesting properties does not.

I can see how confusing this is. Unfortunately I don't have an idea for an easy fix. The error message is neither in Wikibase nor in the PropertySuggester, but in the basic MediaWiki core API code.


We could try to teach the GetSuggestions.php API module to accept unknown language codes, but that alone would not fix the issue. The API would not fail any more, but still return nothing. It would need to fall back to English, and I don't know if this is a good idea.
We could try to validate the language code in the frontend _javascript_ code before passing it to GetSuggestions.php. But since core seems to be fine with any string, I don't know how such a validation (against what?) would look like.
TASK DETAILhttps://phabricator.wikimedia.org/T156220WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, PokestarFan, Mbch331, Lydia_Pintscher, daniel, Aklapper, Yurik, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, Wikidata-bugs, aude, Sjoerddebruin, TheDJ___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T187855: Undefined index in WikibaseClient $this->getRepositoryDefinitions()->getDatabaseNames()['']

2018-02-23 Thread thiemowmde
thiemowmde added a comment.
@Tgr, how old is this Vagrant box you are using? I suspect the error might already be fixed via https://gerrit.wikimedia.org/r/410133. Can you please have a look?TASK DETAILhttps://phabricator.wikimedia.org/T187855EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, hoo, Aklapper, Tgr, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Luke081515, Wikidata-bugs, aude, Mbch331, jeremyb___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Edited] T188086: [Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.3.0

2018-02-23 Thread thiemowmde
thiemowmde updated the task description. (Show Details)
CHANGES TO TASK DESCRIPTIONList of code bases that currently use the Wikibase CiCodeSniffer <0.3.0:...[] #Wikidata.org https://gerrit.wikimedia.org/r/413730
[x] WikimediaBadges https://gerrit.wikimedia.org/r/4102643731
[] #DataValues components:...[x] #wikibase-datamodel-serialization https://github.com/wmde/WikibaseDataModelSerialization/pull/242...TASK DETAILhttps://phabricator.wikimedia.org/T188086EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: gerritbot, Jonas, WMDE-leszek, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Changed Project Column] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-23 Thread thiemowmde
thiemowmde moved this task from Unsorted to Migrate / Replace on the Technical-Debt board.thiemowmde added a project: WorkType-Maintenance.
TASK DETAILhttps://phabricator.wikimedia.org/T188011WORKBOARDhttps://phabricator.wikimedia.org/project/board/609/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-23 Thread thiemowmde
thiemowmde added a project: Patch-For-Review.thiemowmde updated the task description. (Show Details)
CHANGES TO TASK DESCRIPTION...List of components that need an update:
* TBD[] #ArticlePlaceholder
[] #Cognate
[] #MediaWiki-extensions-InterwikiSorting
[] Wikibase
[x] WikibaseLexeme https://gerrit.wikimedia.org/r/410795
[] WikibaseMediaInfo
[] #Wikibase-Quality
[] #Wikibase-Quality-Constraints
[] #Wikibase-Quality-External-Validation
[] WikimediaBadges
[] #DataValues components:
** [] common
** [] data-values
** [] geo
** [] interfaces
** [] number
** [] serialization
** [] time
[] [serialization/serialization](https://github.com/wmde/Serialization)
[] #wikibase-datamodel
[] #wikibase-datamodel-serialization https://github.com/wmde/WikibaseDataModelSerialization/pull/244
[] #wikibase-datamodel-services
[] #wikibase-internal-serializationTASK DETAILhttps://phabricator.wikimedia.org/T188011EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T187775: Investigation: Constraints for a database schema to store representations of a Lexeme

2018-02-23 Thread thiemowmde
thiemowmde added a comment.
Estimated table sizes:


wbl_lexemes
The latest Item ID is currently Q49977198. Thats 9 bytes.
9 * 3 = 27 bytes per row.
27 * 1 million Lexemes = 26 megabytes.

wbl_lemmata
Lexeme IDs will be similar to Item IDs, so 9 bytes again.
Lets say language codes are 5 bytes on average (e.g. stuff like "en-gb").
Lets say lemmas are 15 characters on average (see http://www.ravi.io/language-word-lengths).
Lemmas will use multi-byte UTF-8 characters in many cases. I suggest to assume a factor of 4 bytes per character, just to be sure.
Lets say a Lexeme does have 2 lemmas on average.
( 9 + 5 + ( 15 * 4 ) ) * 2 * 1 million Lexemes = 141 megabytes.

TASK DETAILhttps://phabricator.wikimedia.org/T187775EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, Lucas_Werkmeister_WMDE, Ladsgroup, WMDE-leszek, thiemowmde, Aklapper, Lydia_Pintscher, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T187775: Investigation: Constraints for a database schema to store representations of a Lexeme

2018-02-23 Thread thiemowmde
thiemowmde added a comment.

wb_terms is plural. Most MediaWiki core tables are plural. I also like plural names for tables more. But in the end it really does not matter.
I used VARBINARY and VARCHAR BINARY as they currently are on other Wikibase tables. From https://dev.mysql.com/doc/refman/5.7/en/binary-varbinary.html: "The […] VARBINARY data types [is] distinct from the […] VARCHAR BINARY data type. […] the BINARY attribute does not cause the column to be treated as a binary string column. Instead, it causes the binary (_bin) collation for the column character set to be used, and the column itself contains nonbinary character strings rather than binary byte strings." https://dev.mysql.com/doc/refman/5.7/en/charset-binary-collations.html explains this in much more detail. Based on this I believe whats suggested above is correct: Use VARBINARY for Item IDs that are known to not contain multi-byte characters, but VARCHAR BINARY for values that will contain multi-byte characters.
TASK DETAILhttps://phabricator.wikimedia.org/T187775EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, Lucas_Werkmeister_WMDE, Ladsgroup, WMDE-leszek, thiemowmde, Aklapper, Lydia_Pintscher, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188086: [Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.3.0

2018-02-23 Thread thiemowmde
thiemowmde added a subtask: T167436: [Task] Use Wikibase CodeSniffer standard in all Wikibase code repositories.
TASK DETAILhttps://phabricator.wikimedia.org/T188086EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, WMDE-leszek, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T167436: [Task] Use Wikibase CodeSniffer standard in all Wikibase code repositories

2018-02-23 Thread thiemowmde
thiemowmde added a parent task: T188086: [Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.3.0.
TASK DETAILhttps://phabricator.wikimedia.org/T167436EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: PokestarFan, gerritbot, Ladsgroup, Aklapper, TerraCodes, aude, hoo, Aleksey_WMDE, Lydia_Pintscher, thiemowmde, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, MuhammadShuaib, SimmeD, Wikidata-bugs, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188086: [Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.3.0

2018-02-23 Thread thiemowmde
thiemowmde moved this task from incoming to ready to go on the Wikidata board.thiemowmde triaged this task as "Low" priority.thiemowmde added projects: Patch-For-Review, User-thiemowmde, Need-volunteer, Easy.
TASK DETAILhttps://phabricator.wikimedia.org/T188086WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, WMDE-leszek, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Aklapper, Giuliamocci, Adrian1985, Asad_Ali_Palijo, Cpaulf30, Lahi, Gq86, Baloch007, Darkminds3113, Lordiis, GoranSMilovanovic, Adik2382, Soteriaspace, RazeSoldier, Jayprakash12345, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, QZanden, Zoranzoki21, LawExplorer, Lewizho99, Maathavan, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Created] T188086: [Task] Update Wikibase code bases to use Wikibase CodeSniffer 0.3.0

2018-02-23 Thread thiemowmde
thiemowmde created this task.thiemowmde added projects: Wikidata, DataValues, WorkType-Maintenance.Herald added a subscriber: Aklapper.
TASK DESCRIPTIONList of code bases that currently use the Wikibase CideSniffer <0.3.0:


 #ArticlePlaceholder https://gerrit.wikimedia.org/r/410258
 #Cognate
 #MediaWiki-extensions-InterwikiSorting https://gerrit.wikimedia.org/r/410261
 #PropertySuggester
 Wikibase https://gerrit.wikimedia.org/r/405735
 WikibaseLexeme https://gerrit.wikimedia.org/r/409859
 WikibaseMediaInfo
 #Wikibase-Quality https://gerrit.wikimedia.org/r/409865
 #Wikibase-Quality-Constraints https://gerrit.wikimedia.org/r/409869
 #Wikibase-Quality-External-Validation https://gerrit.wikimedia.org/r/409861
 #Wikidata.org
 WikimediaBadges https://gerrit.wikimedia.org/r/410264
 #DataValues components:
 common
 data-values
 geo
 interfaces
 number https://github.com/DataValues/Number/pull/116
 serialization
 time

 serialization/serialization
 #wikibase-datamodel https://github.com/wmde/WikibaseDataModel/pull/773
 #wikibase-datamodel-serialization https://github.com/wmde/WikibaseDataModelSerialization/pull/242
 #wikibase-datamodel-services https://github.com/wmde/WikibaseDataModelServices/pull/192
 #wikibase-internal-serialization
TASK DETAILhttps://phabricator.wikimedia.org/T188086EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, WMDE-leszek, Lucas_Werkmeister_WMDE, Ladsgroup, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Closed] T167436: [Task] Use Wikibase CodeSniffer standard in all Wikibase code repositories

2018-02-23 Thread thiemowmde
thiemowmde closed this task as "Resolved".thiemowmde removed a project: User-thiemowmde.thiemowmde updated the task description. (Show Details)
CHANGES TO TASK DESCRIPTION...* [x] WikimediaBadges
* [ * [x] #datavalues components:...* [ ] ~~diff~~
* [ ] s* [x] serialization https://github.com/wmde/Serialization/pull/19
* [x] #wikibase-datamodel https://github.com/wmde/WikibaseDataModel/pull/745
* [ ] #wikibase-datamodel-_javascript_
* [ * [x] #wikibase-datamodel-sserialization https://github.com/wmde/WikibaseDataModelSerialization/pull/228
* [x] #wikibase-datamodel-services https://github.com/wmde/WikibaseDataModelServices/pull/179 and https://github.com/wmde/WikibaseDataModelServices/pull/180
* [ ] #wikibase-internal-serialization https://github.com/wmde/WikibaseInternalSerialization/pull/121
* [ ] #wikibase-_javascript_-api
* [ ] #wikibase-serialization-_javascript_
* [ ] #valueview* [x] #wikibase-internal-serialization https://github.com/wmde/WikibaseInternalSerialization/pull/121TASK DETAILhttps://phabricator.wikimedia.org/T167436EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: PokestarFan, gerritbot, Ladsgroup, Aklapper, TerraCodes, aude, hoo, Aleksey_WMDE, Lydia_Pintscher, thiemowmde, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, MuhammadShuaib, SimmeD, Wikidata-bugs, TheDJ, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Edited] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-22 Thread thiemowmde
thiemowmde updated the task description. (Show Details)
CHANGES TO TASK DESCRIPTION...If a code base requires it's own PHPUnit version (this is the case for all PHP components that are not MediaWiki extensions), the suggestion is to go for this dependency in composer.json:dependency in composer.json needs to be updated like this:
```lang=JSON
"phpunit/phpunit": "^4.8.35|^6",...TASK DETAILhttps://phabricator.wikimedia.org/T188011EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-22 Thread thiemowmde
thiemowmde moved this task from incoming to ready to go on the Wikidata board.thiemowmde triaged this task as "Low" priority.thiemowmde added projects: Need-volunteer, Easy.thiemowmde added a comment.
I already uploaded https://gerrit.wikimedia.org/r/410795 and https://github.com/wmde/WikibaseDataModelSerialization/pull/244 as a first proof-of-concept, especially to see if the …|^6 idea works.TASK DETAILhttps://phabricator.wikimedia.org/T188011WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Updated] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-22 Thread thiemowmde
thiemowmde added a project: User-thiemowmde.
TASK DETAILhttps://phabricator.wikimedia.org/T188011EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, SimmeD, Wikidata-bugs, aude, TheDJ, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Created] T188011: [Task] Use PHPUnit 6 forward compatibility aliases in all Wikibase code bases

2018-02-22 Thread thiemowmde
thiemowmde created this task.thiemowmde added projects: Wikidata, DataValues, Technical-Debt.Herald added a subscriber: Aklapper.
TASK DESCRIPTIONPHPUnit backported namespaced class names from PHPUnit 6 back to the older versions 4 and 5. We can start using these right now. This will make the later migration to PHPUnit 6 quite a lot easier, as the necessary patches will be much smaller.

In contrast to what the changelog states, PHPUnit 4.8.35 introduced these aliases:


PHPUnit_Framework_Assert → Assert
PHPUnit_Framework_BaseTestListener → BaseTestListener
PHPUnit_Framework_TestCase → TestCase
PHPUnit_Framework_TestListener → TestListener


In addition, PHPUnit 4.8.36 introduced these aliases:


PHPUnit_Framework_AssertionFailedError → AssertionFailedError
PHPUnit_Framework_Test → Test
PHPUnit_Framework_TestSuite → TestSuite


The suggestion is to not care about any of these, but exclusively focus on replacing PHPUnit_Framework_TestCase with TestCase. This is the vast majority of replacements we need to do (about 84% when I count appearances in my local codebase, with the next one being 7% of PHPUnit_Framework_MockObject_MockObject). Focusing on this one class will make reviewing patches easier. All the other updates can be done later when actually updating to PHPUnit 6.

The suggestion is to not replace … extends \PHPUnit_Framework_TestCase with … extends \PHPUnit\Framework\TestCase, but with … extends TestCase and an import in the use section above. This fits better to the code style the Wikidata team is used to, where full qualified class names are avoided.

If a code base requires it's own PHPUnit version (this is the case for all PHP components that are not MediaWiki extensions), the suggestion is to go for this dependency in composer.json:

"phpunit/phpunit": "^4.8.35|^6",

List of components that need an update:


TBD
TASK DETAILhttps://phabricator.wikimedia.org/T188011EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: Jonas, Lucas_Werkmeister_WMDE, WMDE-leszek, Ladsgroup, thiemowmde, Aklapper, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T187912: Usernames with apostrophes are not well escaped in edit summaries

2018-02-21 Thread thiemowmde
thiemowmde added projects: MediaWiki-extensions-WikibaseRepository, Easy, Regression.thiemowmde moved this task from incoming to ready to go on the Wikidata board.thiemowmde triaged this task as "Normal" priority.thiemowmde added a comment.
Seems to be closely related to T187755: Double quotes in title are displayed as HTML entity in comments. Could potentially be the same cause, but should be tracked separately.TASK DETAILhttps://phabricator.wikimedia.org/T187912WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, matej_suchanek, Aklapper, Asad_Ali_Palijo, Lahi, Gq86, GoranSMilovanovic, Soteriaspace, RazeSoldier, Jayprakash12345, Jrbranaa, JakeTheDeveloper, QZanden, Zoranzoki21, LawExplorer, TerraCodes, MuhammadShuaib, Wong128hk, SimmeD, Wikidata-bugs, aude, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Triaged] T187888: "Failed to dump Q12129 (Value must be at most 127 characters long.)" when dumping Wikidata as TTL

2018-02-21 Thread thiemowmde
thiemowmde edited projects, added DataValues, Technical-Debt; removed MediaWiki-extensions-WikibaseRepository.thiemowmde moved this task from incoming to needs discussion or investigation on the Wikidata board.thiemowmde triaged this task as "Normal" priority.thiemowmde added a comment.
The message is from the DecimalValue constructor, which is used in the QuantityValue constructor. This situation can happen when an edit is made via the API, and a quantity is submitted as a floating point number instead of a string. The code in the DecimalValue that converts floats to strings, but does this in a way so it can violate it's own limitations. Basically: The float is converted to a string with 100 decimal places. If the number before the decimal point is longer than 27 characters, the conversion fails with said error message.

This is very closely related to T155910: Erroneous digits in QuantityValue. I already prepared a fix in https://github.com/DataValues/Number/pull/115, which might solve both issues.TASK DETAILhttps://phabricator.wikimedia.org/T187888WORKBOARDhttps://phabricator.wikimedia.org/project/board/71/EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: thiemowmde, Aklapper, hoo, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Mbch331, Jay8g___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


[Wikidata-bugs] [Maniphest] [Commented On] T187775: Investigation: Constraints for a database schema to store representations of a Lexeme

2018-02-21 Thread thiemowmde
thiemowmde added a comment.
@WMDE-leszek, something like this would be my draft:

CREATE TABLE IF NOT EXISTS wbl_lexemes (
  lex_lexeme_id VARBINARY(20) NOT NULL PRIMARY KEY,
  lex_lexical_category_id VARBINARY(20) NOT NULL,
  lex_language_item_id VARBINARY(20) NOT NULL
);

CREATE TABLE IF NOT EXISTS wbl_lemmata (
  lem_lexeme_id VARBINARY(20) NOT NULL,
  lem_language_code VARBINARY(32) NOT NULL,
  lem_lemma VARCHAR(768) BINARY NOT NULL
);
CREATE UNIQUE INDEX wbl_lem_lexeme_id_language_code ON wbl_lemmata (lem_lexeme_id, lem_language_code);TASK DETAILhttps://phabricator.wikimedia.org/T187775EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: thiemowmdeCc: daniel, Lucas_Werkmeister_WMDE, Ladsgroup, WMDE-leszek, thiemowmde, Aklapper, Lydia_Pintscher, Lahi, Gq86, Cinemantique, GoranSMilovanovic, QZanden, LawExplorer, Wikidata-bugs, aude, Darkdadaah, Mbch331___
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs


<    1   2   3   4   5   6   7   8   9   10   >