Page MenuHomePhabricator

RevisionSlider fails to load on MainPage diff
Closed, ResolvedPublic2 Story Points


Displays: ⧼revisionslider-loading-failed⧽


TypeError: Cannot read property 'trim' of undefined
    at Revision.hasEmptyComment (
    at RevisionListView.makeCommentLine (
    at RevisionListView.makeTooltip (
    at RevisionListView.render (
    at SliderView.render (
    at Object.success (
    at fire (
    at Object.fireWith [as resolveWith] (
    at done (
    at XMLHttpRequest.callback (

API response can be seen at:

Event Timeline

Restricted Application added a project: TCB-Team. · View Herald TranscriptJul 14 2016, 6:36 PM
Restricted Application added subscribers: Zppix, Aklapper. · View Herald Transcript
Addshore updated the task description. (Show Details)Jul 14 2016, 6:42 PM
Addshore added a comment.EditedJul 14 2016, 6:50 PM

comment , parsedComment, user are all undefined for revid = 23793

This revision appears in the API as:

    "revid": 23793,
    "parentid": 23702,
    "minor": false,
    "userhidden": true,
    "timestamp": "2007-05-18T15:25:25Z",
    "size": 49,
    "commenthidden": true,
    "suppressed": true

A quick fix would be to either allow the comment to be undefined, or ensure it is only ever an empty string in the case the comment is hidden. (The same consideration needs to be made for all fields that can be suppressed / hidden)

A further development would be to show in the revision slider when a comment is hidden in some special way.

Addshore moved this task from Unsorted 💣 to Next on the User-Addshore board.

Change 299041 had a related patch set uploaded (by Addshore):
Account for suppressed comments and users

Addshore triaged this task as High priority.
Addshore moved this task from Proposed to Review on the TCB-Team-Sprint-2016-07-14 board.
Addshore moved this task from Next to Active 🚁 on the User-Addshore board.
Addshore moved this task from Incoming to Revision Slider on the TCB-Team board.
Addshore moved this task from Incoming to Doing on the Revision-Slider board.
Tobi_WMDE_SW set the point value for this task to 2.Jul 15 2016, 7:00 AM

Change 299041 merged by WMDE-leszek:
Account for suppressed comments and users

I'll close this ticket once the fix is deployed on

Addshore closed this task as Resolved.Jul 20 2016, 4:25 PM
Addshore moved this task from Doing to Done on the Revision-Slider board.Aug 3 2016, 5:13 PM