Page MenuHomePhabricator

convertExtensionToRegistration skips config params set to empty array
Closed, ResolvedPublic

Description

When I generated extension.json for Graph extension, git aced8e127238a25ac8ddae38f0697693811570db, it skipped the GraphDataDomains configuration parameter that was set to the array(). This is very dangerous, as it may be accidentally forgotten when converting other extensions, and cause run-time errors.

Event Timeline

Yurik assigned this task to Legoktm.
Yurik raised the priority of this task from to High.
Yurik updated the task description. (Show Details)
Yurik added a project: MediaWiki-Configuration.
Yurik subscribed.
Paladox added a subscriber: Legoktm.
Paladox subscribed.

Hi could the problem be because It is set like

$$var = array();

where as shoulden it be like

$var = array();

Change 210365 had a related patch set uploaded (by Paladox):
Remove $ from $$var in convertExtensionToRegistration.php

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

Change 210365 abandoned by Paladox:
Remove $ from $$var in convertExtensionToRegistration.php

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

Change 210389 had a related patch set uploaded (by Legoktm):
registration: Don't ignore empty array config settings when converting

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

Change 210389 merged by jenkins-bot:
registration: Don't ignore empty array config settings when converting

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

Change 212575 had a related patch set uploaded (by Legoktm):
registration: Don't ignore empty array config settings when converting

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

Change 212575 merged by jenkins-bot:
registration: Don't ignore empty array config settings when converting

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