The code in changeset 211966 is only one piece of the puzzle. The code is mostly duplicated in BlogPage as getCommentsOfTheDay(). This also needs to be updated. See T98550.
Description
Details
Related Objects
Event Timeline
Change 214938 had a related patch set uploaded (by UltrasonicNXT):
Use Ext:Comments for comments of the day
This task has been "Unbreak now!" priority for more than two weeks.
A patch is available but has not been reviewed yet.
The definition of "Unbreak now" is "to be fixed immediately".
Please either review/merge the patch or correct the priority value of this task. Thank you!
@Aklapper - It has been reviewed. I'll quote the patch author's most recent comment - "Right, that's some of the comments fixed, but still work to do."
It is 'Unbreak now' because until that change is merged, the extension's main functionality is broken because the database schema has changed.
(Removing "Unbreak now" priority as this was set for more than half a year ago but reality has shown that this task has not such a level of urgency.)
Change 275197 had a related patch set uploaded (by Jack Phoenix):
[WIP/UNTESTED?] Version 4.2.0: comments of the day cleaned up & unconditionally enabled
Change 275197 abandoned by Jack Phoenix:
[WIP/UNTESTED?] Version 4.2.0: comments of the day cleaned up & unconditionally enabled
Reason:
Will submit again as a new changeset.
Change 275198 had a related patch set uploaded (by Jack Phoenix):
[WIP/UNTESTED?] Version 4.2.0: comments of the day cleaned up & unconditionally enabled
Change 275198 merged by Jack Phoenix:
Comments of the day cleaned up & unconditionally enabled
Change 331082 had a related patch set uploaded (by Jack Phoenix):
Finally make the "comments of the day" feature work again
Change 331082 merged by jenkins-bot:
Finally make the "comments of the day" feature work again