To see the permissions for a role is important to use the pace module and manage the access permissions.

According to the instruction of pacs module, in the tree view, click on a role button to see the permissions for that role.

So I created a few roles and set different permissions (view, or update, or manage) for different roles on a node. It works, I mean a user with the view permission can view the node, another user with the update permission can edit the node, ...

But clicking any role button in the tree view I can see nothing about the permissions information of that role. Is it a bug or something else wrong?

Comments

peterone’s picture

peterone commented
Assigned: astra » peterone
Status: Active » Closed (fixed)

Hi astra,

In the pacs tree view, the permission for the active role on each node is represented by the color of the node: gray for none, green for view, yellow for update and red for manage.

Peter

astra’s picture

astra commented

Maybe the permissions represented in different color is not supported by some of themes. I'm using the theme B7 and I can't see the colors you talked about. Is it possible to represent the permissions by text witch is suitable for any theme? Thanks!

sigsby’s picture

sigsby commented

I noticed the latest 4.7 release has a bug in including the css files if you put the module in a site folder (i.e. sites/default/modules/pacs). The module currently tries to load the css from modules/pacs/*.css. If this you put the module anywhere other than modules/pacs the css doesn't get loaded and you won't see the colors in the tree view.

Line 571:

  // load style sheets on demand
  if(strpos('.'.request_uri(),'/tree') > 0) {
    theme('add_style','modules/pacs/pacs.css');
  }
  if(strpos('.'.request_uri(),'/help/pacs') > 0) {
    theme('add_style','modules/pacs/help.css');
  }

Should be:

  // load style sheets on demand
  if(strpos('.'.request_uri(),'/tree') > 0) {
    theme('add_style',drupal_get_path("module","pacs").'/pacs.css');
  }
  if(strpos('.'.request_uri(),'/help/pacs') > 0) {
    theme('add_style',drupal_get_path("module","pacs").'/help.css');
  }
sigsby’s picture

sigsby commented

Just a comment as a follow-up, I think the UI for the tree tab needs to be reworked. It is not very usable as is and themes could eaily break it (as is shown by the original post). I would recommend using a standard form elements (checkboxes/radio buttons). Themes won't break it and it is more self explanatory.

I'm still getting a feel for this module, not sure I completely understand how it's working. I'm going on vacation here for two weeks, but when I get back, if it's something I decide to start using I would be more than happy to do some work towards this and submit some patches.