Page MenuHomePhabricator

Redirects to Education Program: pages results in fatal error
Closed, ResolvedPublic

Description

https://en.wikipedia.org/wiki/Wikipedia:Ambassadors/Courses/Theater_History_from_1642_%28Amy_E_Hughes%29/Timeline?redirect=no is a redirect to https://en.wikipedia.org/wiki/Education_Program:CUNY,_Brooklyn_College/Theater_History_from_1642_%28Spring_2013%29 which exists. However if you try to use the redirect directly (https://en.wikipedia.org/wiki/Wikipedia:Ambassadors/Courses/Theater_History_from_1642_%28Amy_E_Hughes%29/Timeline), the "Wikimedia Foundation error" screen comes up with:

PHP fatal error in /usr/local/apache/common-local/php-1.21wmf8/includes/Wiki.php line 387:
Call to undefined method CoursePage::loadPageData()


Version: unspecified
Severity: critical

Details

Reference
bz44490

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 1:21 AM
bzimport set Reference to bz44490.

This will be fixed when switching to ContentHandler.

If this is very critical I cane come up with a way less nice fix that will nevertheless get rid of the error for now.

We could probably request an edit filter to prevent creation of redirects to course pages, for now. But a less nice quick fix that still permits the use of redirects like this would be better.

Why no redirects to course pages??! I could look into this, though would need a separate bug (feature request) for this.

That's the bug: redirects to course pages cause a fatal error. It'd be nice to be able to redirect to course pages without that error. Is that what the patch does?

My patch solves the issue. Although switching to content handler would do so more elegantly and prevent similar issues which we perhaps have not uncovered yet.

Are you happy with redirects working now, or should redirects to education pages be disallowed altogether.

Sounds good. Redirects working is better than not working.

(In reply to comment #4)

https://gerrit.wikimedia.org/r/#/c/46778/

Merged by Chris Steipp on the 30th.