The code should be optimized in some points.

A line like

  $grant_update = $access['update'][$rid] > 0 ? TRUE : FALSE;

can be simply written

  $grant_update = ($access['update'][$rid] > 0)

The code between parenthesis will value to TRUE if $access['update'][$rid] is greater than zero, FALSE otherwise.

Comments

harry slaughter’s picture

Status: Active » Fixed

fixed in tonight's dev snapshot

Status: Fixed » Closed (fixed)

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