Page MenuHomePhabricator

Edit notice saying that changes must be reviewed appears even when the user has autoreview rights
Open, NormalPublic8 Story Points


Flaggedrevs edit notice saying that you don't have reviewer rights appears even when you do have them. This is quite annoying since it's rather long.

Reported at

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:22 AM
bzimport set Reference to bz58057.
bzimport added a subscriber: Unknown Object (MLST).
matmarex created this task.Dec 5 2013, 10:01 PM

Does it actually say that you don't have reviewer rights? Or is it the "Your changes will be displayed to readers once an authorised user accepts them. (help)" one that implies it?

If so, this is correctly shown when there are any unapproved revisions on a page - someone (possibly including the current user) will need to manually approve any new changes even if the current user would normally be able to autoreview (obviously autoreview does not occur if there are outstanding unreviewed changes).

Seems invalid.

Not really.

The notice should not be shown for users whose edits do not need to be reviewed – that is the behavior of the wikitext editor.

So what exactly is happening differently to the wikitext editor here then?

As far as I can tell FR is behaving properly by showing this notice when there are changes waiting for review on the current page (even if the user could review their own change).

The notice appears, always, regardless of everything.

It should be only shown when the user can't autoreview the changes they're about to make, just like in the wikitext editor.

Can't replicate this issue.

  • Bug 59153 has been marked as a duplicate of this bug. ***
  • Bug 57835 has been marked as a duplicate of this bug. ***

Marking as WFM given lack of replication steps.

The problem definitely occurs, still occurs, but also occurs in the source editor. It is just that the problem is more pronounced in the Visual Editor.

The message might need to be split into two - one for people with the rights to approve changes, and another for people whose edits will be queued for review.
Hopefully the latter message is shorter.

Alternatively there is another message (on de.wp) "Notice: Some of the pending changes affect the area of the page you are editing." , which could be split into two, so that editors with review rights see "Notice: There are pending changes which will be automatically sighted if your changes are saved."

Ah, right. Marking as such.

Elitre added a subscriber: Elitre.Jan 6 2015, 12:52 PM
Jdforrester-WMF lowered the priority of this task from High to Normal.Jan 15 2015, 12:12 AM
Jdforrester-WMF set Security to None.

Noting that this would fix the editnotice part of T75931.

WTM added a subscriber: WTM.Aug 1 2015, 1:16 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 1 2015, 1:16 PM
Cenarium claimed this task.Nov 28 2015, 6:24 PM

Change 255811 had a related patch set uploaded (by Cenarium):
Don't display review notice if edit requires no review

Catrope removed a subscriber: Catrope.Jan 13 2016, 9:28 PM
Jdforrester-WMF set the point value for this task to 8.May 10 2016, 1:24 AM
Krinkle removed a subscriber: Krinkle.May 16 2016, 1:14 PM