Last modified: 2014-08-28 09:35:24 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 T71459, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69459 - PHP Warning: Invalid argument
PHP Warning: Invalid argument
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
CirrusSearch (Other open bugs)
master
All Linux
: Lowest minor (vote)
: ---
Assigned To: Chad H.
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-13 07:41 UTC by keyler
Modified: 2014-08-28 09:35 UTC (History)
3 users (show)

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


Attachments

Description keyler 2014-08-13 07:41:32 UTC
When using CirrusSearch (Master) we get the following with mw1.23.2. I know we should use CirrusSearch 1.23 but I wanted to try the newest changes and also Elasticserach 1.3.1. 
While running "php forceSearchIndex.php" we get this message about a 1000 times, but on the good side: search is WORKING :-)
 
PHP Warning:  Invalid argument supplied for foreach() in .../CirrusSearch/includes/BuildDocument/PageTextBuilder.php on line 129
Comment 1 Chad H. 2014-08-27 22:10:42 UTC
CirrusSearch master generally depends on MW master and probably won't work all that well with the 1.23.x release branch. In this case, we're relying on Gerrit change #127238 from core.

I'll commit a fix in a moment that protect Cirrus master here a bit more and should make things a tad more compatible.
Comment 2 Gerrit Notification Bot 2014-08-27 22:11:38 UTC
Change 156712 had a related patch set uploaded by Chad:
Protect against older versions of HtmlFormatter

https://gerrit.wikimedia.org/r/156712
Comment 3 Gerrit Notification Bot 2014-08-27 23:56:47 UTC
Change 156712 merged by jenkins-bot:
Protect against older versions of HtmlFormatter

https://gerrit.wikimedia.org/r/156712
Comment 4 keyler 2014-08-28 09:35:24 UTC
Thank you Chad an Nik, as always!

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


Navigation
Links