Page MenuHomePhabricator

Error: Class 'NewsletterDiffEngine' not found
Closed, ResolvedPublicPRODUCTION ERROR

Description

Error
normalized_message
[{reqId}] {exception_url}   Error: Class 'NewsletterDiffEngine' not found
exception.trace
from /srv/mediawiki/php-1.40.0-wmf.1/includes/content/ContentHandler.php(606)
#0 /srv/mediawiki/php-1.40.0-wmf.1/includes/page/Article.php(896): ContentHandler->createDifferenceEngine(RequestContext, integer, string, integer, boolean, boolean)
#1 /srv/mediawiki/php-1.40.0-wmf.1/includes/page/Article.php(465): Article->showDiffPage()
#2 /srv/mediawiki/php-1.40.0-wmf.1/includes/actions/ViewAction.php(78): Article->view()
#3 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(542): ViewAction->show()
#4 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(322): MediaWiki->performAction(Article, Title)
#5 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(904): MediaWiki->performRequest()
#6 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(562): MediaWiki->main()
#7 /srv/mediawiki/php-1.40.0-wmf.1/index.php(50): MediaWiki->run()
#8 /srv/mediawiki/php-1.40.0-wmf.1/index.php(46): wfIndexMain()
#9 /srv/mediawiki/w/index.php(3): require(string)
#10 {main}
Impact
Notes

Event Timeline

Umherirrender subscribed.

The error message is without namespace, the extension was namespaced in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Newsletter/+/778363 without aliases, but there is a usage in NewsletterContentHandler::getDiffEngineClass

Change 832708 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Newsletter@master] Use namespaced class in NewsletterContentHandler::getDiffEngineClass

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

Change 832708 merged by jenkins-bot:

[mediawiki/extensions/Newsletter@master] Use namespaced class in NewsletterContentHandler::getDiffEngineClass

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

Change 832682 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Newsletter@REL1_39] Use namespaced class in NewsletterContentHandler::getDiffEngineClass

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

Not sure if this needs backporting. It seems the use case to diff a newsletter page is very rare, because the error exists since 1.39-wmf.20 and is visible on, for example, https://www.mediawiki.org/w/index.php?title=Newsletter:The_Community_Tech_Newsletter&diff=2641518&oldid=2622492

Possible easier to just wait a week and let the train fix it

Change 832682 merged by jenkins-bot:

[mediawiki/extensions/Newsletter@REL1_39] Use namespaced class in NewsletterContentHandler::getDiffEngineClass

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

Should be fine to let it ride the train.