Page MenuHomePhabricator

Skins should no longer output HTML and head tag
Open, HighPublicBUG REPORT

Description

NOTE: Impacted skins and extensions are listed here https://codesearch.wmcloud.org/search/?q=(bottomscripts%7Cheadelement)&i=nope&files=&excludeFiles=&repos= NOTE: To address this problem please stop calling these functions. You can see https://phabricator.wikimedia.org/rSCAVab2b4414a0f405318b5ccea89225f1aa50b90591 for an example.

In 1.39 skins following T62846 using SkinTemplate will trigger a deprecation notice Deprecated: Use of QuickTemplate::(get/html/text/haveData) with parameter bottomscripts` was deprecated in MediaWiki 1.39. [Called from QuickTemplate::get in /vagrant/mediawiki/includes/skins/QuickTemplate.php at line 144] in /vagrant/mediawiki/includes/debug/MWDebug.php on line 380`. On the long term this will ensure that all extension code is rendered by a skin and head tags are consistent.

There are no plans on the short term to break these skins, but a lot of skins are impacted and we should endeavor to fix as many of them as possible.

Any help is greatly appreciated and will be reviewed within 48hrs provided it's associated with this ticket.

The fix is relatively simple:

  1. Skins should add skin option bodyOnly: true to skin.json
  2. Skins should remove any echo statements that echo bottomscripts, headelement or getTrail

Examples:

Impacted skins

Last updated May 13th 2022:

  • neptune
  • material
  • osmfoundation
  • strapping (mediawiki-strapping)
  • tyrian
  • bluell
  • pure
  • hive
  • p2wiki
  • cosmos
  • hassomecolours
  • amethyst
  • nimbus
  • metrolook
  • gamepress
  • woogleshades
  • poncho
  • bluesky
  • greystuff
  • pivot
  • foreground
  • tweeki
  • truglass
  • gumaxdd
  • splash
  • wptouch
  • deskmessmirrored
  • bouquet
  • dusk
  • dusktodawn
  • refreshed
  • medik
  • liberty
  • darkvector
  • aether https://invent.kde.org/websites/aether-mediawiki/-/merge_requests/4
  • apex
  • anisa
  • erudite
  • mask
  • tempo
  • wikimediaapiportal https://gerrit.wikimedia.org/r/c/mediawiki/skins/WikimediaApiPortal/+/786397
  • webplatform
  • jony
  • modernskylight
  • darkcosmos
  • scratchwikiskin2
  • onyx
  • simpletext
  • s2018
  • monaco
  • cavendish
  • cavendish-brown
  • cavendish-green
  • mediawikibootstrap
  • peruna
  • collegeinsider
  • cemublue
  • library
  • t29v7
  • atlasmuseum
  • eveskin
  • wisky

TODO

Details

SubjectRepoBranchLines +/-
mediawiki/coremaster+60 -113
mediawiki/skins/mediawiki-strappingREL1_39+2 -7
mediawiki/skins/MaterialREL1_39+21 -24
mediawiki/skins/mediawiki-strappingmaster+2 -7
mediawiki/skins/MetrolookREL1_39+28 -12
mediawiki/skins/Metrolookmaster+28 -12
mediawiki/skins/Materialmaster+21 -24
mediawiki/skins/DeskMessMirroredmaster+23 -47
mediawiki/skins/DuskToDawnmaster+22 -30
mediawiki/skins/Duskmaster+20 -21
mediawiki/skins/Bouquetmaster+26 -36
mediawiki/skins/DeskMessMirroredREL1_39+23 -47
mediawiki/skins/BouquetREL1_39+26 -36
mediawiki/skins/DuskREL1_39+20 -21
mediawiki/skins/DuskToDawnREL1_39+22 -30
mediawiki/skins/GamepressREL1_39+9 -46
mediawiki/skins/Gamepressmaster+9 -46
mediawiki/skins/Nimbusmaster+28 -12
mediawiki/skins/NimbusREL1_39+28 -12
mediawiki/skins/RefreshedREL1_39+22 -34
mediawiki/skins/Refreshedmaster+22 -34
mediawiki/skins/apexmaster+40 -69
mediawiki/skins/apexREL1_39+40 -69
mediawiki/skins/PonchoREL1_39+3 -9
mediawiki/skins/Ponchomaster+3 -9
mediawiki/skins/GuMaxDDmaster+28 -469
mediawiki/skins/TruglassREL1_39+17 -38
mediawiki/skins/Truglassmaster+17 -38
mediawiki/skins/CosmosREL1_39+4 -8
mediawiki/skins/GreyStuffREL1_39+14 -57
mediawiki/skins/GreyStuffmaster+14 -57
mediawiki/skins/SplashREL1_39+5 -11
mediawiki/skins/HasSomeColoursREL1_39+14 -21
mediawiki/skins/AnisaREL1_39+13 -21
mediawiki/skins/Anisamaster+13 -21
mediawiki/skins/HasSomeColoursmaster+14 -21
mediawiki/skins/WoOgLeShadesmaster+12 -12
mediawiki/skins/Splashmaster+5 -11
mediawiki/skins/Cosmosmaster+4 -8
mediawiki/skins/AmethystREL1_39+12 -4
mediawiki/skins/Amethystmaster+12 -4
mediawiki/skins/BlueSkymaster+14 -12
mediawiki/skins/Tempomaster+2 -5
mediawiki/skins/Maskmaster+2 -5
mediawiki/skins/eruditemaster+2 -8
mediawiki/skins/Cavendishmaster+4 -7
Show related patches Customize query in gerrit

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 831854 merged by jenkins-bot:

[mediawiki/skins/GreyStuff@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831854

Change 831853 merged by jenkins-bot:

[mediawiki/skins/Cosmos@REL1_39] Fix deprecated use of headelement

https://gerrit.wikimedia.org/r/831853

Change 831950 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Truglass@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831950

Change 831858 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Truglass@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831858

Change 831950 merged by jenkins-bot:

[mediawiki/skins/Truglass@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831950

Change 831953 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/GuMaxDD@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831953

Change 831858 merged by jenkins-bot:

[mediawiki/skins/Truglass@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831858

Change 831953 merged by jenkins-bot:

[mediawiki/skins/GuMaxDD@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/831953

Change 831958 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Poncho@master] Skins should no longer output HTML and head tag

https://gerrit.wikimedia.org/r/831958

Change 831864 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Poncho@REL1_39] Skins should no longer output HTML and head tag

https://gerrit.wikimedia.org/r/831864

Change 831958 merged by jenkins-bot:

[mediawiki/skins/Poncho@master] Skins should no longer output HTML and head tag

https://gerrit.wikimedia.org/r/831958

Change 831864 merged by jenkins-bot:

[mediawiki/skins/Poncho@REL1_39] Skins should no longer output HTML and head tag

https://gerrit.wikimedia.org/r/831864

Change 832296 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/apex@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832296

Change 832308 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/apex@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832308

Change 832308 merged by jenkins-bot:

[mediawiki/skins/apex@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832308

Change 832296 merged by jenkins-bot:

[mediawiki/skins/apex@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832296

Change 832527 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Refreshed@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832527

Change 832528 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Nimbus@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832528

Change 832325 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Refreshed@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832325

Change 832325 merged by jenkins-bot:

[mediawiki/skins/Refreshed@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832325

Change 832527 merged by jenkins-bot:

[mediawiki/skins/Refreshed@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832527

Change 832546 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Nimbus@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832546

Change 832546 merged by jenkins-bot:

[mediawiki/skins/Nimbus@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832546

Change 832528 merged by Jdlrobson:

[mediawiki/skins/Nimbus@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832528

Change 832645 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Gamepress@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832645

Change 832666 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Gamepress@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832666

Change 832648 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/DuskToDawn@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832648

Change 832649 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Dusk@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832649

Change 832651 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/DeskMessMirrored@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832651

Change 832653 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Bouquet@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832653

Change 832645 merged by jenkins-bot:

[mediawiki/skins/Gamepress@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832645

Change 832667 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Bouquet@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832667

Change 832668 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Dusk@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832668

Change 832669 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/DuskToDawn@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832669

Change 832666 merged by jenkins-bot:

[mediawiki/skins/Gamepress@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832666

Change 832669 merged by jenkins-bot:

[mediawiki/skins/DuskToDawn@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832669

Change 832668 merged by jenkins-bot:

[mediawiki/skins/Dusk@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832668

Change 832667 merged by jenkins-bot:

[mediawiki/skins/Bouquet@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832667

Change 832670 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/DeskMessMirrored@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832670

Change 832670 merged by jenkins-bot:

[mediawiki/skins/DeskMessMirrored@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832670

Change 832653 merged by jenkins-bot:

[mediawiki/skins/Bouquet@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832653

Change 832649 merged by jenkins-bot:

[mediawiki/skins/Dusk@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832649

Change 832648 merged by jenkins-bot:

[mediawiki/skins/DuskToDawn@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832648

Change 832651 merged by jenkins-bot:

[mediawiki/skins/DeskMessMirrored@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832651

Change 832664 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Metrolook@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832664

Change 832686 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Material@master] Upgrade Material for 1.39

https://gerrit.wikimedia.org/r/832686

Change 832676 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Material@REL1_39] Upgrade Material for 1.39

https://gerrit.wikimedia.org/r/832676

Change 832693 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/mediawiki-strapping@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832693

Change 832686 merged by jenkins-bot:

[mediawiki/skins/Material@master] Upgrade Material for 1.39

https://gerrit.wikimedia.org/r/832686

Change 832697 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/p2wiki@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832697

Change 832677 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Metrolook@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832677

Change 832678 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/mediawiki-strapping@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832678

Change 832677 merged by jenkins-bot:

[mediawiki/skins/Metrolook@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832677

Change 832664 merged by jenkins-bot:

[mediawiki/skins/Metrolook@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832664

Change 832693 merged by jenkins-bot:

[mediawiki/skins/mediawiki-strapping@master] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832693

Change 832676 merged by Jdlrobson:

[mediawiki/skins/Material@REL1_39] Upgrade Material for 1.39

https://gerrit.wikimedia.org/r/832676

Change 832678 merged by jenkins-bot:

[mediawiki/skins/mediawiki-strapping@REL1_39] Upgrade for 1.39

https://gerrit.wikimedia.org/r/832678

(This is not a blocker - this is more a message to skin developers who will see deprecation messages in 1.40 release)

Change 930261 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] WIP: BREAKING CHANGE: Remove support for skins rendering outside body element

https://gerrit.wikimedia.org/r/930261

Jdlrobson edited projects, added MW-1.41-release; removed MW-1.40-release.
Jdlrobson updated the task description. (Show Details)

Note that these have missed the boat for 1.41, so should be moved/re-evaluated for 1.42.

(This is actually trickier - as it is a breaking change for skins that return content other than HTML e.g. JSON.