Exception encountered, of type "InvalidArgumentException" in update.php when 'vector' is symlinked to 'Vector'
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	saper
	Dec 15 2015, 9:54 AM

Description

In order to workaround T121490 I have symlinked "vector" in the "skins" directory to "Vector".

The CLI installer went fine, but running update.php fails with

Exception encountered, of type "InvalidArgumentException"

The stacktrace:

`
[5323b2ec] [no req]   InvalidArgumentException from line 88 of /usr/home/saper/public_html/collaps/includes/config/ConfigFactory.php: Invalid callback provided
Backtrace:
#0 /usr/home/saper/public_html/collaps/includes/config/ConfigFactory.php(59): ConfigFactory->register(string, array)
#1 /usr/home/saper/public_html/collaps/maintenance/doMaintenance.php(100): ConfigFactory::getDefaultInstance()
#2 /usr/home/saper/public_html/collaps/maintenance/update.php(214): require_once(string)
#3 {main}
`

What happens is that vector is added twice to $wgConfigRegistry:

`
array(2) {
  ["vector"]=>
  array(2) {
    [0]=>
    string(28) "GlobalVarConfig::newInstance"
    [1]=>
    string(28) "GlobalVarConfig::newInstance"
  }
  ["main"]=>
  string(28) "GlobalVarConfig::newInstance"
}
`

MediaWiki core: 5b0ebd7f3a6e1c215a6af56d966ee14047a9b44f

Details

	Subject	Repo	Branch	Lines +/-
	registration: Prevent loading the same thing twice	mediawiki/core	master	+28 -2

Customize query in gerrit

Related Objects

Mentioned In: T153050: MediaWiki extension registry fails when extension and skin have same name (eg: CustomPage)
Mentioned Here: rMW5b0ebd7f3a6e: Merge "Avoid "CAS updated failed" errors on Special:Preferences double post"
T121490: vector vs Vector: 404 for collapsibleTabs.js with $wgResourceLoaderDebug = true and composer

Event Timeline

saper created this task.Dec 15 2015, 9:54 AM

saper raised the priority of this task from to Needs Triage.

saper updated the task description. (Show Details)

saper added a project: MediaWiki-Configuration.

saper subscribed.

Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptDec 15 2015, 9:54 AM

Just noticed Vector got added twice to LocalSettings.php:

wfLoadSkin( 'vector' );
wfLoadSkin( 'Vector' );

So either we should not allow stashing multiple callbacks like this into the Config Registry, or maybe the updater should learn how to deal with an array there. Or just plainly fail in the right place.

Or...maybe you shouldn't try loading the same skin twice under different names?

@Legoktm sure, but let's have an error message instead.... I see that the current configurator is very versatile, but there is no easy place to put such a check in....

Line 297 of ExtensionProcessor::extractCredits() looks like a good place, before we store it in $this->credits. Do you want to write a patch for that or should I?

Liuxinyu970226 set Security to None.Dec 18 2015, 1:42 PM

Reedy triaged this task as Low priority.Dec 18 2015, 1:44 PM

Change 261608 had a related patch set uploaded (by Legoktm):
registration: Prevent loading the same thing twice

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

gerritbot added a project: Patch-For-Review.Dec 30 2015, 1:52 AM

Legoktm claimed this task.Dec 30 2015, 1:53 AM

Paladox subscribed.Dec 30 2015, 2:09 AM

Change 261608 merged by jenkins-bot:
registration: Prevent loading the same thing twice

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

Legoktm closed this task as Resolved.Dec 31 2015, 6:51 PM

ReleaseTaggerBot added projects: MW-1.27-release (WMF-deploy-2016-01-12_(1.27.0-wmf.10)), MW-1.27-release-notes.Dec 31 2015, 9:00 PM

hashar mentioned this in T153050: MediaWiki extension registry fails when extension and skin have same name (eg: CustomPage).Dec 13 2016, 10:03 AM

Exception encountered, of type "InvalidArgumentException" in update.php when 'vector' is symlinked to 'Vector'Closed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

Exception encountered, of type "InvalidArgumentException" in update.php when 'vector' is symlinked to 'Vector'
Closed, ResolvedPublic
Actions