Since I was messing around with Global Redirect already, I thought I'd clean it up a wee bit. So here is the result.

I felt like changing stuff such as if (variable_get('site_offline', 0) == 1) return FALSE; into

if (variable_get('site_offline', 0) == 1) {
  return FALSE;
}

as well, but realised I didn't know whether these one liners were preserved for keeping the processing time at a minimum, or... ? (If so, there's a place where a "multi-liner" could be changed into a "one-liner" - consistency please!) Pray let me know so I can alter the patch accordingly. Thank you. :)

CommentFileSizeAuthor
gr_small_cleanup.patch2.72 KBFreso

Comments

nicholasthompson’s picture

Version: 6.x-1.x-dev » 5.x-1.x-dev
Status: Needs review » Fixed

Applied to D5... Will sync all this to D6 when D5 is working better.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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