Page MenuHomePhabricator

Catchable fatal error: Argument 6 passed to EducationProgram\EPArticle::__construct() must be an array, boolean given, called in ArticleStore.php on line 215 and defined at EPArticle.php on line 119
Closed, ResolvedPublicPRODUCTION ERROR

Description

[2014-04-03 19:32:56] Catchable fatal error: Argument 6 passed to EducationProgram\EPArticle::__construct() must be an array, boolean given, called in /usr/local/apache/common-local/php-1.23wmf20/extensions/EducationProgram/includes/ArticleStore.php on line 215 and defined at /usr/local/apache/common-local/php-1.23wmf20/extensions/EducationProgram/includes/rows/EPArticle.php on line 119
Server: mw1044
Method: POST
URL: http://en.wikipedia.org/w/index.php?title=Education_Program:CCT110:_The_Rhetoric_of_Digital_and_Interactive_Media_Environments&returnto=Education+Program%3AUniversity+of+Toronto+Mississauga%2FCCT110%3A+The+Rhetoric+of+Digital+and+Interactive+Media+Environments+%28W14%29
Backtrace:
#0 /usr/local/apache/common-local/php-1.23wmf20/extensions/EducationProgram/includes/rows/EPArticle.php(119): EducationProgram\EPArticle::__construct()
#1 /usr/local/apache/common-local/php-1.23wmf20/extensions/EducationProgram/includes/ArticleStore.php(215): EducationProgram\EPArticle->__construct(0, 0, 0, 0, NULL, false)
#2 /usr/local/apache/common-local/php-1.23wmf20/extensions/EducationProgram/includes/ArticleStore.php(195): EducationProgram\ArticleStore->newArticleFromRow(false)
#3 /usr/local/apache/common-local/php-1.23wmf20/extensions/EducationProgram/includes/actions/RemoveArticleAction.php(38): EducationProgram\ArticleStore->getArticle(3409)
#4 /usr/local/apache/common-local/php-1.23wmf20/includes/actions/FormlessAction.php(67): EducationProgram\RemoveArticleAction->onView()
#5 /usr/local/apache/common-local/php-1.23wmf20/includes/Wiki.php(428): FormlessAction->show()
#6 /usr/local/apache/common-local/php-1.23wmf20/includes/Wiki.php(292): MediaWiki->performAction(Object(EducationProgram\OrgPage), Object(Title))
#7 /usr/local/apache/common-local/php-1.23wmf20/includes/Wiki.php(588): MediaWiki->performRequest()
#8 /usr/local/apache/common-local/php-1.23wmf20/includes/Wiki.php(447): MediaWiki->main()
#9 /usr/local/apache/common-local/php-1.23wmf20/index.php(49): MediaWiki->run()
#10 /usr/local/apache/common-local/w/index.php(3): require('/usr/local/apac...')
#11 {main}

Version: unspecified
Severity: normal

Details

Reference
bz63499

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:11 AM
bzimport set Reference to bz63499.

Reedy, did you just catch this once, or is it occurring regularly?

Our working hypothesis is that someone manaully put in the wrong url by mistake, and that this is related to or the same as Bug 62872.

andrew.green.df wrote:

It looks like this is due to two attempts to remove the same article assignment. The second attempt, which I think triggered the error, was likely done from a stale version of the course page.

To reproduce the error:

  1. On a course page, assign an article to a student
  2. Open a refreshed version of the page in two browser tabs.
  3. In one of the tabs, remove the article assignment.
  4. In the other tab, without refreshing the page, try to remove the same article assignment again.

Assuming this is what happened, this bug should be something like: handle non-existent article assignment ids remove article (assignment) actions.

Note that on my local install, I got several additional related errors just before the error and stack trace mentioned above.

andrew.green.df wrote:

Full local stack trace when following procedure to reproduce the error.

Attached:

I've poked around to see if any other duplicated events from a non-refreshed separate tab create similar errors, but this seems to be the only one.

(In reply to Sage Ross from comment #1)

Reedy, did you just catch this once, or is it occurring regularly?

Our working hypothesis is that someone manaully put in the wrong url by
mistake, and that this is related to or the same as Bug 62872.

I think this might be the case. I only saw the one, but it was there at the point I was dealing with the content of the recent logs. Feel free to dupe against bug 62872

Per Andrew's comment, it's actually unrelated to that bug; it'll happen whenever a user tries to remove the same article from a student twice, by having two different copies of the page open. So, fairly rare and should be easy to fix.

Thanks Reedy.

Anyone with shell access who knows if this still an issue?

Krenair claimed this task.
Krenair subscribed.

Doesn't look like it:

krenair@fluorine:/a/mw-log$ grep EducationProgram exception.log
krenair@fluorine:/a/mw-log$ zgrep Educationprogram archive/exception.log-*.gz
krenair@fluorine:/a/mw-log$
demon triaged this task as Medium priority.Jul 9 2015, 4:40 PM
mmodell changed the subtype of this task from "Task" to "Production Error".Aug 28 2019, 11:12 PM