This thread is for creating a D6 port of Project Permissions (per request of dww and scor). I had the project marked as abandoned a while ago, so I opened up an issue in the webmasters queue to have it reopened and assigned to me.

Comments

scor’s picture

scor commented

Are you planning a straight port to D6, or do we want to refactor some code from project_permissions D5?

Note, you might want to create a new branch for D6 so we have it in the Version: drop down and can file issues against the appropriate version of the module ;)

wonder95’s picture

wonder95 commented
Project: Project Permissions » Project Access
Version: 5.x-1.x-dev » 6.x-1.x-dev

Reassigning to Project Access issue queue since that is the namespace we will be using going forward.

wonder95’s picture

wonder95 commented

OK @dww and @scor (and anyone else that wants to opine), now that I have the basics converted to D6, I have a couple questions.

  1. Do we still need to cover both role and user based permissions? I'm assuming so. In the case that both are enabled, I would also assume that user level permissions should take priority over role level permissions, since they would be more granular. Correct?
  2. Currently there is a form on the project page (node/%nid/edit/access) that specifies both role and user level permissions, and also a separate tab on the user profile (user/%uid/project) that lists the projects the user has access to and provides checkboxes to modify permissions. Do you think we should keep both pages, since the user permissions are a duplicate of what is on the project node tab?
wonder95’s picture

wonder95 commented

OK, I've got this so that it's writing records to node_access correctly, after I fixed it. However, my project_access_node_grants isn't even being touched for some reason whenever I view, edit, or save a project node. @dww and @scor, if you can take a look at it, I would appreciate. I'll look at it later tonight. HEAD is current.

wonder95’s picture

wonder95 commented

@scor and @dww, I have a site working where you can test this out. You can access it at http://project.smgaweb.com. I don't have email addresses for you, so please just create an account and I'll set you up as an admin so you can play with it.

If anyone else wants to help test this out, just create an account (admin approval is required) and I'll set you up to help test.

Thanks.

scor’s picture

scor commented

I create a new issue #785546: What values to return in hook_node_grants?. I suppose this issue could not be closed since we technically have a D6 port now, and new issues should be created when bugs are found or features are requested.

scor’s picture

scor commented

Currently there is a form on the project page (node/%nid/edit/access) that specifies both role and user level permissions, and also a separate tab on the user profile (user/%uid/project) that lists the projects the user has access to and provides checkboxes to modify permissions. Do you think we should keep both pages, since the user permissions are a duplicate of what is on the project node tab?

I can't see the project tab on the user profile page, was that removed?

wonder95’s picture

wonder95 commented

No, it is currently not enabled on the site. The setting to enable user access is at admin/project/access. @dww and I had talked about it at DCSF, and decided that this part isn't needed, since we already add specific users on the project node itself. I just haven't taken it out of the code yet.

wonder95’s picture

wonder95 commented
Status: Active » Closed (fixed)

Closing this issue since the a D6 port has been created. Open new issues for any issues that come up in testing.