Last modified: 2014-07-02 07:58:15 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T69142, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67142 - ZeroPortal unit tests throws Warning: JsonConfig: Invalid $wgJsonConfigs['JsonZeroConfig']['url']: API URL is not set ...
ZeroPortal unit tests throws Warning: JsonConfig: Invalid $wgJsonConfigs['Jso...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
ZeroPortal (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 67216
  Show dependency treegraph
 
Reported: 2014-06-26 17:28 UTC by Antoine "hashar" Musso (WMF)
Modified: 2014-07-02 07:58 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Warning: JsonConfig: Invalid $wgJsonConfigs['JsonZeroConfig']['url']: API URL is not set (2.99 KB, text/plain)
2014-06-26 17:28 UTC, Antoine "hashar" Musso (WMF)
Details

Description Antoine "hashar" Musso (WMF) 2014-06-26 17:28:19 UTC
I am working on a Jenkins jobs that runs phpunit against all extensions.  I noticed ZeroPortal throws a warning.

I used the master version of core and of the following extensions:

$ /bin/ls -1 extensions
ContributionTracking
DonationInterface
JsonConfig
MobileFrontend
README
ZeroBanner
ZeroPortal
$

$ cd tests/phpunit/
$ php phpunit.php --testsuite extensions

Warning: JsonConfig: Invalid $wgJsonConfigs['JsonZeroConfig']['url']: API URL is not set, and this config is not being stored locally [Called from JsonConfig\JCSingleton::init in /extensions/JsonConfig/includes/JCSingleton.php at line 106] in /core/includes/debug/Debug.php on line 303


Will attach stacktrace.
Comment 1 Antoine "hashar" Musso (WMF) 2014-06-26 17:28:43 UTC
Created attachment 15752 [details]
Warning: JsonConfig: Invalid $wgJsonConfigs['JsonZeroConfig']['url']: API URL is not set
Comment 2 Yuri Astrakhan 2014-06-26 19:32:18 UTC
Can be fixed by adding ['isLocal']=true. Will try to get to it shortly or feel free to submit a patch or even +2 it
Comment 3 Antoine "hashar" Musso (WMF) 2014-06-26 20:52:23 UTC
There is no hurry. It only impacts a proof of concept I am working on.  But would be nice to have the issue dropped soonish :-)
Comment 4 Antoine "hashar" Musso (WMF) 2014-06-30 14:08:40 UTC
We really need tests to pass when all wmf extensions are installed together. That is preventing us from progression toward the HHVM migrating. Raising priority to High.
Comment 5 Gerrit Notification Bot 2014-07-01 21:53:46 UTC
Change 143491 had a related patch set uploaded by Yurik:
Fixed defaults to allow non-configured extension to work per bug 67142

https://gerrit.wikimedia.org/r/143491
Comment 6 Gerrit Notification Bot 2014-07-01 21:54:03 UTC
Change 143492 had a related patch set uploaded by Yurik:
Matching default ZeroPortal config changes per bug 67142

https://gerrit.wikimedia.org/r/143492
Comment 7 Gerrit Notification Bot 2014-07-01 21:57:08 UTC
Change 143491 merged by jenkins-bot:
Fixed defaults to allow non-configured extension to work per bug 67142

https://gerrit.wikimedia.org/r/143491
Comment 8 Gerrit Notification Bot 2014-07-01 21:59:11 UTC
Change 143492 merged by MaxSem:
Matching default ZeroPortal config changes per bug 67142

https://gerrit.wikimedia.org/r/143492
Comment 9 Antoine "hashar" Musso (WMF) 2014-07-02 07:58:15 UTC
Thank you very much everyone. That hides the issued warning :-}

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links