There's a handful of notices in the code which I'm fixing and changing the menu titles to be "Sentence case" which is the standard and removing trailing whitespace from any lines. I also added a more robust check that the users have actually configured the site.

The one I'm least certain about is the change in security_review.checks.inc in security_review_check_comments.

CommentFileSizeAuthor
#1 652244_small_fixups.patch14.22 KBgreggles

Comments

greggles’s picture

Status: Active » Needs review
StatusFileSize
new14.22 KB
coltrane’s picture

Status: Needs review » Fixed

Looks good, I was able to get a variable_get working for the first-time check. Committed patch along with some minor mods.

Status: Fixed » Closed (fixed)

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