PHP Warning: count(): Parameter must be an array or an object that implements Countable
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Assigned To

Authored By

	zeljkofilipin
	Mar 20 2019, 8:05 PM

Description

Error

Request ID: AWmcomha8aQffZ3H9FZs

message

PHP Warning: count(): Parameter must be an array or an object that implements Countable

trace

#0 /srv/mediawiki/php-1.33.0-wmf.22/includes/parser/BlockLevelPass.php(202): MWExceptionHandler::handleError(integer, string, string, integer, array)
#1 /srv/mediawiki/php-1.33.0-wmf.22/includes/parser/BlockLevelPass.php(52): BlockLevelPass->execute()
#2 /srv/mediawiki/php-1.33.0-wmf.22/includes/parser/Parser.php(2555): BlockLevelPass::doBlockLevels(string, boolean)
#3 /srv/mediawiki/php-1.33.0-wmf.22/includes/parser/Parser.php(1415): Parser->doBlockLevels(string, boolean)
#4 /srv/mediawiki/php-1.33.0-wmf.22/includes/parser/Parser.php(485): Parser->internalParseHalfParsed(string, boolean, boolean)
#5 /srv/mediawiki/php-1.33.0-wmf.22/includes/cache/MessageCache.php(1260): Parser->parse(string, Title, ParserOptions, boolean)
#6 /srv/mediawiki/php-1.33.0-wmf.22/includes/Message.php(1265): MessageCache->parse(string, Title, boolean, boolean, LanguageEn)
#7 /srv/mediawiki/php-1.33.0-wmf.22/includes/Message.php(890): Message->parseText(string)
#8 /srv/mediawiki/php-1.33.0-wmf.22/includes/Message.php(943): Message->toString(string)
#9 /srv/mediawiki/php-1.33.0-wmf.22/includes/EditPage.php(3614): Message->parse()
#10 /srv/mediawiki/php-1.33.0-wmf.22/includes/EditPage.php(2980): EditPage->showEditTools()
#11 /srv/mediawiki/php-1.33.0-wmf.22/includes/EditPage.php(719): EditPage->showEditForm()
#12 /srv/mediawiki/php-1.33.0-wmf.22/includes/actions/EditAction.php(60): EditPage->edit()
#13 /srv/mediawiki/php-1.33.0-wmf.22/includes/MediaWiki.php(501): EditAction->show()
#14 /srv/mediawiki/php-1.33.0-wmf.22/includes/MediaWiki.php(294): MediaWiki->performAction(Article, Title)
#15 /srv/mediawiki/php-1.33.0-wmf.22/includes/MediaWiki.php(867): MediaWiki->performRequest()
#16 /srv/mediawiki/php-1.33.0-wmf.22/includes/MediaWiki.php(517): MediaWiki->main()
#17 /srv/mediawiki/php-1.33.0-wmf.22/index.php(42): MediaWiki->run()
#18 /srv/mediawiki/w/index.php(3): require(string)
#19 {main}

Impact

Notes

Details

Subject	Repo	Branch	Lines +/-
Avoid counting input lines twice in BlockLevelPass::execute()	mediawiki/core	wmf/1.33.0-wmf.22	+6 -6
BlockLevelPass: further fixes for T218817	mediawiki/core	wmf/1.33.0-wmf.22	+5 -2
parser: Count occurrences of newlines	mediawiki/core	wmf/1.33.0-wmf.22	+1 -1
parser: Count occurrences of newlines	mediawiki/core	master	+1 -1
Avoid counting input lines twice in BlockLevelPass::execute()	mediawiki/core	master	+6 -6
BlockLevelPass: further fixes for T218817	mediawiki/core	master	+5 -2
BlockLevelPass: further fixes for T218817	mediawiki/core	master	+5 -2
parser: Check that $textLines is an array that can be iterated	mediawiki/core	master	+2 -1

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved	Release	zeljkofilipin	T206676 1.33.0-wmf.22 deployment blockers
		Resolved	PRODUCTION ERROR	Arlolra	T218817 PHP Warning: count(): Parameter must be an array or an object that implements Countable

Event Timeline

zeljkofilipin created this task.Mar 20 2019, 8:05 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 20 2019, 8:05 PM

zeljkofilipin triaged this task as Unbreak Now! priority.Mar 20 2019, 8:05 PM

zeljkofilipin added a parent task: T206676: 1.33.0-wmf.22 deployment blockers.

Restricted Application added subscribers: Liuxinyu970226, TerraCodes. · View Herald TranscriptMar 20 2019, 8:05 PM

zeljkofilipin updated the task description. (Show Details)Mar 20 2019, 8:09 PM

zeljkofilipin added a project: MediaWiki-Parser.

Possibly caused by 480890.

cc @Arlolra @cscott @Krinkle

zeljkofilipin added projects: Parsing-Team--ARCHIVED, Platform Engineering.Mar 20 2019, 8:15 PM

@zeljkofilipin Looking, thanks. Maybe it should be an iterator_count ...

Change 497862 had a related patch set uploaded (by D3r1ck01; owner: Derick Alangi):
[mediawiki/core@master] parser: Check that $textLines is an array that can be iterated

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

gerritbot added a project: Patch-For-Review.Mar 20 2019, 8:29 PM

Change 497863 had a related patch set uploaded (by Arlolra; owner: Arlolra):
[mediawiki/core@master] Count occurrences of newlines

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

Change 497862 abandoned by D3r1ck01:
parser: Check that $textLines is an array that can be iterated

Reason:
superseded by https://gerrit.wikimedia.org/r/#/c/497863/

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

Change 497865 had a related patch set uploaded (by C. Scott Ananian; owner: C. Scott Ananian):
[mediawiki/core@master] Don't count the input lines

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

xSavitar awarded a token.Mar 20 2019, 8:41 PM

Change 497863 merged by jenkins-bot:
[mediawiki/core@master] parser: Count occurrences of newlines

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

Hold up, there's another bug here with ExplodeIterator we're discussing.

ReleaseTaggerBot added a project: MW-1.33-notes (1.33.0-wmf.23; 2019-03-26).Mar 20 2019, 9:01 PM

ExplodeIterator::key() returns $this->curPos, *not* the line number. So with the 497863 patch it fixes the exceptions (and so should be fine to backport to wmf.22) but for wikitexts over a thousand lines it will effectively "~~never~~ always be on the last line" and so you'll get an extra trailing newline.

~~I think that's harmless for wmf.22,~~ but we'll probably want to amend the patch; either 497865 or else just moving $i=0 outside the loop and $i++ inside the loop so we're always counting lines (EDIT: that's 497871).

EDIT: oops, got the sense of the test wrong, since there's a less-than comparison, not a simple equals. So this is more significant that I'd thought.

Change 497871 had a related patch set uploaded (by Arlolra; owner: Arlolra):
[mediawiki/core@master] parser: More accounting for ExplodeIterator

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

zeljkofilipin added a project: User-zeljkofilipin.Mar 20 2019, 9:09 PM

zeljkofilipin moved this task from Backlog 🪒 to Project ♟ on the User-zeljkofilipin board.

Change 497879 had a related patch set uploaded (by C. Scott Ananian; owner: Arlolra):
[mediawiki/core@master] BlockLevelPass: further fixes for T218817

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

Change 497871 had a related patch set uploaded (by Arlolra; owner: Arlolra):
[mediawiki/core@master] BlockLevelPass: further fixes for T218817

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

Change 497879 abandoned by Arlolra:
BlockLevelPass: further fixes for T218817

Reason:
As PS4 on https://gerrit.wikimedia.org/r/c/mediawiki/core/ /497871

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

Change 497871 merged by jenkins-bot:
[mediawiki/core@master] BlockLevelPass: further fixes for T218817

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

Change 497865 merged by jenkins-bot:
[mediawiki/core@master] Avoid counting input lines twice in BlockLevelPass::execute()

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

Which of the patches should get back-ported?

The final result, I41a45266d266195aa6002d3854e018cacf052ca6, is the one we want but since they all stack on top of each other, you might have to do all three. It's fine to just rebase the last one though.

Change 497939 had a related patch set uploaded (by Jforrester; owner: Arlolra):
[mediawiki/core@wmf/1.33.0-wmf.22] parser: Count occurrences of newlines

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

Change 497940 had a related patch set uploaded (by Jforrester; owner: Arlolra):
[mediawiki/core@wmf/1.33.0-wmf.22] BlockLevelPass: further fixes for T218817

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

Change 497941 had a related patch set uploaded (by Jforrester; owner: C. Scott Ananian):
[mediawiki/core@wmf/1.33.0-wmf.22] Avoid counting input lines twice in BlockLevelPass::execute()

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

Arlolra mentioned this in T218834: Write a test that exercises the ExplodeIterator in BlockLevelPass.Mar 20 2019, 11:02 PM

Change 497939 merged by jenkins-bot:
[mediawiki/core@wmf/1.33.0-wmf.22] parser: Count occurrences of newlines

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

ReleaseTaggerBot edited projects, added MW-1.33-notes (1.33.0-wmf.22; 2019-03-19); removed MW-1.33-notes (1.33.0-wmf.23; 2019-03-26).Mar 21 2019, 12:01 AM

Change 497940 merged by jenkins-bot:
[mediawiki/core@wmf/1.33.0-wmf.22] BlockLevelPass: further fixes for T218817

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

Change 497941 merged by jenkins-bot:
[mediawiki/core@wmf/1.33.0-wmf.22] Avoid counting input lines twice in BlockLevelPass::execute()

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

It would be a good idea to add some parser tests which exercise both modes of StringUtils::explode. If this hadn't thrown an exception for being not Countable, we probably wouldn't have noticed that the array keys shift on large articles until it made production.

Probably fixed?

It would be a good idea to add some parser tests which exercise both modes of StringUtils::explode

I've already filed T218834 for that.

Probably fixed?

I think so.

Krinkle moved this task from Untriaged to Resolved on the Wikimedia-production-error board.Mar 21 2019, 8:08 PM

Liuxinyu970226 unsubscribed.Mar 26 2019, 12:14 AM

• mmodell changed the subtype of this task from "Task" to "Production Error".Aug 28 2019, 11:07 PM

PHP Warning: count(): Parameter must be an array or an object that implements CountableClosed, ResolvedPublicPRODUCTION ERRORActions

Description

Error

Impact

Notes

Details

Related ObjectsSearch...

Event Timeline

PHP Warning: count(): Parameter must be an array or an object that implements Countable
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Related Objects
Search...