This is actually in alpha6.

In hosting_client.module:

function hosting_client_menu_access($perm, $node) {
  if ($node->type == 'client') {
    return user_access($perm, $node);
  } else {
    return false;
  }
}

Should just be return user_access($perm); I think. Otherwise should do a user_load($node->uid) and pass the resulting user object into user access not the node.

Right now this is triggering a bunch of errors for me on the client pages if the UID of the client isn't 1.



    * warning: array_fill() [function.array-fill]: Number of elements must be positive in /home/aegir/hostmaster-0.4-alpha6/includes/database.inc on line 253.
    * warning: implode() [function.implode]: Bad arguments. in /home/aegir/hostmaster-0.4-alpha6/includes/database.inc on line 253.
    * warning: array_keys() [function.array-keys]: The first argument should be an array in /home/aegir/hostmaster-0.4-alpha6/modules/user/user.module on line 502.
    * user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1 query: SELECT p.perm FROM role r INNER JOIN permission p ON p.rid = r.rid WHERE r.rid IN () in /home/aegir/hostmaster-0.4-alpha6/modules/user/user.module on line 502.

Comments

hadsie’s picture

Project: Hostmaster (Aegir) » Hosting

I think this should be in the hosting queue not hostmaster...

adrian’s picture

Status: Active » Fixed

I commited this.
thanks

Status: Fixed » Closed (fixed)

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