Last modified: 2014-09-23 18:12:45 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 T73081, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71081 - MassMessage LQT support broken
MassMessage LQT support broken
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
MassMessage (Other open bugs)
unspecified
All All
: Unprioritized major (vote)
: ---
Assigned To: Kunal Mehta (Legoktm)
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-20 00:19 UTC by Kunal Mehta (Legoktm)
Modified: 2014-09-23 18:12 UTC (History)
5 users (show)

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


Attachments

Description Kunal Mehta (Legoktm) 2014-09-20 00:19:06 UTC
From bug comment 7:

> It is not working anymore:
> https://pt.wikibooks.org/wiki/Special:Log/massmessage?uselang=en
> 
> When I tried to send a message to
> https://pt.wikibooks.org/wiki/Wikilivros_Discuss%C3%A3o:Caixa_de_areia
> by creating this list
> https://pt.wikibooks.org/w/index.php?title=Wikilivros:
> Caixa_de_areia&oldid=273032
> it showed
> "Send mass message
> Your message delivery to 1 page has been queued.
> Send another message or view how many messages are queued."
> but the message was not sent. There are now 6 queued mass messages on
> https://pt.wikibooks.org/wiki/Special:Statistics?uselang=en
Comment 1 Kunal Mehta (Legoktm) 2014-09-20 00:20:04 UTC
(In reply to Kunal Mehta (Legoktm) from comment #0)
> From bug comment 7:

bug 52729 comment 7*
Comment 2 Kunal Mehta (Legoktm) 2014-09-20 00:57:37 UTC
I'm debugging this locally right now, it appears to be getting stuck at Thread::validateSubject() (via the LQT API) and my VM starts maxing out the CPU it has...
Comment 3 Kunal Mehta (Legoktm) 2014-09-20 01:02:49 UTC
Aaaand as I suspected it looks like Thread::validateSubject() for whatever reason is going into an infinite loop.
Comment 4 Kunal Mehta (Legoktm) 2014-09-20 01:06:52 UTC
error=MWException: Empty $mTitle in OutputPage::parse
Comment 5 Kunal Mehta (Legoktm) 2014-09-20 01:17:57 UTC
Threads::stripWikitext() calls $wgOut->parseInline() which dies if a Title isn't set.
Comment 6 Gerrit Notification Bot 2014-09-20 01:22:31 UTC
Change 161638 had a related patch set uploaded by Legoktm:
ApiThreadAction: Make sure a Title is set

https://gerrit.wikimedia.org/r/161638
Comment 7 Nemo 2014-09-20 08:45:46 UTC
(In reply to Gerrit Notification Bot from comment #6)
> Change 161638 had a related patch set uploaded by Legoktm:
> ApiThreadAction: Make sure a Title is set
> 
> https://gerrit.wikimedia.org/r/161638

Bug 55739?
Comment 8 Gerrit Notification Bot 2014-09-22 18:14:06 UTC
Change 161638 merged by jenkins-bot:
ApiThreadAction: Make sure a Title is set

https://gerrit.wikimedia.org/r/161638
Comment 9 Kunal Mehta (Legoktm) 2014-09-22 18:17:50 UTC
(In reply to Nemo from comment #7)

> Bug 55739?

No, don't think it's related.
Comment 10 Gerrit Notification Bot 2014-09-22 18:18:38 UTC
Change 162003 had a related patch set uploaded by Legoktm:
ApiThreadAction: Make sure a Title is set

https://gerrit.wikimedia.org/r/162003
Comment 11 Gerrit Notification Bot 2014-09-22 18:19:05 UTC
Change 162004 had a related patch set uploaded by Legoktm:
ApiThreadAction: Make sure a Title is set

https://gerrit.wikimedia.org/r/162004
Comment 12 Gerrit Notification Bot 2014-09-22 23:11:21 UTC
Change 162004 merged by jenkins-bot:
ApiThreadAction: Make sure a Title is set

https://gerrit.wikimedia.org/r/162004
Comment 13 Gerrit Notification Bot 2014-09-22 23:36:49 UTC
Change 162003 merged by EBernhardson:
ApiThreadAction: Make sure a Title is set

https://gerrit.wikimedia.org/r/162003
Comment 14 Kunal Mehta (Legoktm) 2014-09-23 18:12:45 UTC
Backported to WMF sites yesterday, confirmed working again: <https://www.mediawiki.org/wiki/Thread:User_talk:Legoktm/LQT/Testing_lqt>.

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


Navigation
Links