Ghuron added a comment.
Apparently implementation of //wikibase.getSitelink()// in //client/includes/DataAccess/Scribunto/mw.wikibase.lua// is utilizing //trackUsageForTitleOrSitelink()// in //client/includes/DataAccess/Scribunto/WikibaseLanguageIndependentLuaBindings.php// Implementation of //methodtable.getSitelink()// in //client/includes/DataAccess/Scribunto/mw.wikibase.entity.lua// is much more brutal, it just fires //addSiteLinksUsage// on any access to //sitelinks// table It looks very strange that 2 implementations of //getSitelink()// functions that are supposed to work identically based on 2 completely different pieces of code. If they are not going to be merged somehow, I believe the easiest way to fix this would be to expose //addTitleOrSitelinksUsage()// for lua and use it instead of //php.addSiteLinksUsage// Technically //maskEntityTable()// always passes both //entityId// and //key// (//globalSiteId// here) to //usageFunc()//, it just addSiteLinksUsage happens to ignore second parameter TASK DETAIL https://phabricator.wikimedia.org/T295356 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/ To: Ghuron Cc: putnik, Lucas_Werkmeister_WMDE, Ghuron, Aklapper, Invadibot, maantietaja, Akuckartz, Nandana, lucamauri, Lahi, Gq86, GoranSMilovanovic, QZanden, LawExplorer, _jensen, rosalieper, Scott_WUaS, Wikidata-bugs, aude, Mbch331
_______________________________________________ Wikidata-bugs mailing list -- [email protected] To unsubscribe send an email to [email protected]
