I had a look at the patch for D7 that theoretically merges this module into core, however it doesn't provide all the functionality that this module does, so I think a D7 port might be useful.

Also, I've written a drush module based on this API, should I upload it as a patch?

CommentFileSizeAuthor
#3 permissions_api.drush_.txt5.15 KBlambic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ebeyrent’s picture

ebeyrent CreditAttribution: ebeyrent commented
Assigned: Unassigned » ebeyrent

I reviewed the D7 patch, and came to the same conclusion as you; not all of the functionality of the Permissions API module was included in core. I could certainly do the port; however, I feel that would duplicate a lot of code unnecessarily. I feel that the best course of action is to submit patches to D7 core to include any missing functionality - it helps out everyone, and eliminates duplicitous code. Who wants to lead the charge? :)

Regarding your Drush module, by all means upload it here. If it's a standalone module, you should uploaded it as a .tar.gz file. Otherwise, a patch is fine.

Thanks for your feedback!

lambic’s picture

lambic CreditAttribution: lambic commented

I agree that it would be better in core, however D7 is in code freeze now, so I don't know if a patch would be accepted at this point.

I'll upload the drush module later. I wrote it as a stand-alone module thinking I could make it work with D6 and D7 with lots of function_exists or major_version checking but for now it's D6 only so it could be packaged with permissions_api (as it depends on it).

lambic’s picture

lambic CreditAttribution: lambic commented
FileSize
permissions_api.drush_.txt5.15 KB

Here's the drush module.

ebeyrent’s picture

ebeyrent CreditAttribution: ebeyrent commented
Status: Active » Closed (won't fix)

Closing this issue as a "won't fix". I also created http://drupal.org/node/802248 to deal with the patch you provided regarding Drush support.