[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 524202 abandoned by Tarrow: Use URL for ajax EntityData Reason: can't use URL yet https://gerrit.wikimedia.org/r/524202 TASK DETAIL https://phabricator.wikimedia.org/T85499 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/e

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Ladsgroup added a comment. In T85499#5358738 , @gerritbot wrote: > Change 525142 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup): > [operations/puppet@production] varnish: Do not strip the cache out of Special:EntityD

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Quiddity added a comment. In T85499#5352677 , @Lea_Lacroix_WMDE wrote: > Announced today on the mailing-lists, Project Chat and Weekly Summary, and I sent an individual message to most of the people in the list. example links for pos

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 525142 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup): [operations/puppet@production] varnish: Do not strip the cache out of Special:EntityData if revision is set https://gerrit.wikimedia.org/r/525142 TASK DETAIL https://phabricat

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Lea_Lacroix_WMDE added a comment. Announced today on the mailing-lists, Project Chat and Weekly Summary, and I sent an individual message to most of the people in the list. TASK DETAIL https://phabricator.wikimedia.org/T85499 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/p

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 524202 had a related patch set uploaded (by Tarrow; owner: Tarrow): [mediawiki/extensions/Wikibase@master] Use URL for ajax EntityData https://gerrit.wikimedia.org/r/524202 TASK DETAIL https://phabricator.wikimedia.org/T85499 EMAIL PREFERENCES http

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 523737 **merged** by jenkins-bot: [mediawiki/extensions/Wikibase@master] Replace wbEntity config variable with ajax call https://gerrit.wikimedia.org/r/523737 TASK DETAIL https://phabricator.wikimedia.org/T85499 EMAIL PREFERENCES https://phabricat

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 522609 abandoned by Ladsgroup: Stop serving wbEntity config variable Reason: In favor of I9a96f290f6719cc82767d6d0024daed3f25153ba https://gerrit.wikimedia.org/r/522609 TASK DETAIL https://phabricator.wikimedia.org/T85499 EMAIL PREFERENCES h

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 523737 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup): [mediawiki/extensions/Wikibase@master] Replace wbEntity config variable with ajax call https://gerrit.wikimedia.org/r/523737 TASK DETAIL https://phabricator.wikimedia.org/T854

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 522609 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup): [mediawiki/extensions/Wikibase@master] Stop serving wbEntity config variable https://gerrit.wikimedia.org/r/522609 TASK DETAIL https://phabricator.wikimedia.org/T85499 EMAIL P

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 361652 merged by jenkins-bot: [mediawiki/extensions/Wikibase@master] Improve documentation of "wikibase.entityPage.entityLoaded" hook https://gerrit.wikimedia.org/r/361652TASK DETAILhttps://phabricator.wikimedia.org/T85499EMAIL PREFERENCEShttps://phabricator.wikim

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 361652 had a related patch set uploaded (by Thiemo Mättig (WMDE); owner: Thiemo Mättig (WMDE)): [mediawiki/extensions/Wikibase@master] Improve documentation of "wikibase.entityPage.entityLoaded" hook https://gerrit.wikimedia.org/r/361652TASK DETAILhttps://phabrica

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 353366 merged by jenkins-bot: [mediawiki/extensions/Wikibase@master] Replacement for 'wbEntity' config variable To be able to migrate from 'wbEntity' in future replacing it with an API call. https://gerrit.wikimedia.org/r/353366TASK DETAILhttps://phabricator.wikim

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Aleksey_WMDE added a comment. I've downloaded all the scripts that present in the former list and analyzed them manually here summary of the analysis. Total: 58 scripts Discovered use cases: Check only for existence of wbEntity: 11 scripts Check only for existence of wbEntity.sitelinks: 10 scri

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Aleksey_WMDE added a comment. Script to download all found scripts: #!/usr/bin/env bash curl 'https://www.wikidata.org/w/index.php?title=MediaWiki:Gadget-AuthorityControl.js&action=""> > 'MediaWiki:Gadget-AuthorityControl.js' curl 'https://www.wikidata.org/w/index.php?title=MediaWiki:Gadget-Merge

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Aleksey_WMDE added a comment. To check correctness: MediaWiki:Gadget-AuthorityControl.js should be in the listTASK DETAILhttps://phabricator.wikimedia.org/T85499EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Aleksey_WMDECc: Ladsgroup, gerritbot, Aleksey_WMDE,

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Aleksey_WMDE added a comment. @Ladsgroup What I've noticed that this pattern is wrong. For example: MediaWiki:Gadget-Merge.js contains access to wbEntityId but not to wbEntity. Can you try it with regexp: \bwbEntity\b ? Or may be like: :~$ mwgrep '\'wbEntity\''TASK DETAILhttps://phabricator.wikime

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Aleksey_WMDE added a comment. Also, you've made a mistake in your command, this one is correct: $ curl --compressed -so /dev/null https://www.wikidata.org/wiki/Q183 -w '%{size_download}' 260312TASK DETAILhttps://phabricator.wikimedia.org/T85499EMAIL PREFERENCEShttps://phabricator.wikimedia.org/set

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Aleksey_WMDE added a comment. I downloaded the Q183 entity and removed wbEntity manually, and then compared sizes: $ curl https://www.wikidata.org/wiki/Q183 > Q183.html $ # Manually removed wbEntity and saved as Q183_no_wbEntity.html $ cat Q183.html | wc -c 3316781 $ gzip -c Q183.html | wc -c 2600

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Ladsgroup added a comment. One quick thing: As I checked the response page, it's gzipped causing it to downsize dramatically. For example you said Q183 is 2.9MB which is not true: amir-GL552VW:~$ curl --compressed -so /dev/null https://www.wikidata.oki/Q183 -w '%{size_download}' 42418 amir-GL552VW

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

gerritbot added a comment. Change 353366 had a related patch set uploaded (by Aleksey Bekh-Ivanov (WMDE); owner: Aleksey Bekh-Ivanov (WMDE)): [mediawiki/extensions/Wikibase@master] [WIP] Replacement for 'wbEntity' config variable To be able to migrate from 'wbEntity' in future replacing it with an

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

hoo added a comment. All usages except for AuthorityControl can probably be easily switched without having to worry much. In case of AuthorityControl we need to be more careful: It's enabled per default, thus needs to be handled with care regarding potential performance issues. TASK DETAIL

[Wikidata-bugs] [Maniphest] [Commented On] T85499: wbEntity shouldn't be served on every page load

Jdlrobson added a comment. I have no doubt it is useful and convenient to have it in the page but having the entire json embedded is an unacceptable page load increase. Every user is subjected to a page larger than it needs to be. It should be retrieved via the api, stored in a more minimal for