1459336 - feed daemon skips setting r+ for accepted revision if the same user already has a flag set even if flag is status of ?

Status: RESOLVED FIXED

(Conduit :: General, defect)

Description

[David Lawrence [:dkl]]

The feed daemon loops over current flags and if the setter of the current flag is the same as the new flag being set, it skips adding that flag. Even if the current flags status is '?'. Originally it was designed this way thinking that only Phabricator would be setting review flags for those phabricator revision related patches (those that redirect to phabricator revisions). 

The change will only skip if the review flag is set to '+'. Otherwise it will add another review+ flag separately.

This may have the side effect of having more than one review flag set for an attachment. One with the status '?' set manually by a user, and one with status '+' set by phab-bot.

dkl

Comment 1

[David Lawrence [:dkl]]

Attachment: github pr #568