Looking at update_advanced_update_status_alter() was just kind of loopy and it was hard to see exactly what was going on.

CommentFileSizeAuthor
#1 589404-D6.patch2.64 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 589404-D6.patch2.64 KB

Moved the ignore detection into update_advanced_is_project_ignored() which returns the admin note or TRUE if the project is ignored (since a string equals TRUE), or FALSE if the project should not be ignored. Look good?

dww’s picture

dww
we/he/they
commented
Status: Needs review » Reviewed & tested by the community

Sure, that makes it a lot more clear. Tested and working fine. Please commit at will. ;) Thanks!

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks dww. Committed to CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.