29

On top of the bug in review queues greying out audit items (happening for the last month or so), now the close vote review queue is suddenly telling me "this is an audit" on review items that are definitely not audits. I tried to leave a comment on this review item but was greeted with this error about audits:

enter image description here

This was not the first non-audit item I saw that error on today.

As you can see, going back to the review item doesn't show anything about it being an audit, and there's no indication by the score/dearth of answers that it would qualify as an audit anyway:

enter image description here

This issue was encountered in the close vote review queue, but given the previously-mentioned bug affects all the queues, I'm guessing this one does, too.

Can devs please fix these growing review queue bugs?

2
  • 1
    Or just remove the audit system. Whatever is easiest.
    – Gimby
    Commented Apr 23 at 12:16
  • @Gimby I can't upvote comments or leave comments on review items at all so definitely more to do than removing audits :-)
    – TylerH
    Commented Apr 23 at 14:29

1 Answer 1

9

This is fixed now, along with another bug where audits appear greyed out.

Kristina's answer there has a great description of the underlying cause. Basically a new feature was re-initializing an object, which set the comment functionality to "passive" i.e. read-only. This meant comments.extendCommentUi() did not exist, which caused the exception reported on that post.

For this bug specifically, that also meant comments.init() did not reset any "active" functionality, such as adding comments. When we see an audit, we overwrite some functions to show the "This is an audit" error. Previously those were reset with init() when the next task was loaded, but because of this bug, the overwrites stayed in place, so you would always see "This is an audit" on all tasks until the page was refreshed.

Very tricky one to track down! Made much easier with the details shared in these posts, so thank you!

2
  • 7
    "When we see an audit, we overwrite some functions to show the "This is an audit" error" That reads like some lovely spaghetti-code to maintain o.O
    – Cerbrus
    Commented Jun 27 at 14:14
  • 4
    @Cerbrus No comment! :D
    – kristinalustig StaffMod
    Commented Jun 28 at 13:14

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .