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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
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
CreditAttribution: dww 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
CreditAttribution: Dave Reid 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.