Spin-off from #935062: Change role id to machine name
No code in Drupal should use those magic numbers.
This should be backported.
Looks good. In a cursory grep for 'roles\[' I didn't catch any others.
Committed to 8.x. Moving to 7.x.
Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/bbe4b34
- * @param integer $rid Role id....+ * @param string $rid+ * The role ID to search for.
A little odd we're saying that the role ID is supposed to be a string, although in practice I suppose the calling code does tend to pass (numeric) strings to it rather than integers.
Automatically closed -- issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.