Hi ray007, hi Takafumi,

I have a initial version of this module ported to Drupal 6.
I offer my collaboration to be a co-maintainer of this version of menu_stp.

Thanks.

CommentFileSizeAuthor
#15 menu_stp-329079-15.patch1.59 KBellen.davis
#14 menu_stp-ellen.patch9.79 KBellen.davis
#8 menu_ap.tar_.gz3.23 KBAnonymous (not verified)
#7 menu_ap.tar.gz3.2 KBAnonymous (not verified)
#3 menu_stp.zip12.44 KBazuledu

Comments

bnice5000’s picture

Has there been any discussion as to posting the dev version of this for drupal 6? Is it still being maintained?

joe-b’s picture

Would be great if you could post something here so others can start working with it also, Azuledu.

azuledu’s picture

StatusFileSize
new12.44 KB

Here is the initial Drupal 6 version of this module.

Right now, it only works with menu items, not with complete menus.

I've applied for a CVS account to upload and maintain it.

slosa’s picture

subscribing

ray007’s picture

@azuledu: haven't checked your code, but since work has currently led me away from drupal, I'll be happy to hand over maintainership of the module to you.

best regards,
Ray

tlogan’s picture

@azuledu
Thanks for taking this over. I am currently setting up a site that needs this functionality so the timing of this couldn't be better. I'd love to see this work for the menu as well as the menu items. Does that look like something that will be easily do-able or is that a major undertaking that will not be done for quite some time?

Anonymous’s picture

StatusFileSize
new3.2 KB

Not sure what the status is of making this happen or if azuledu is still interested. Either way, I needed the functionality so I worked on this today. Attached is a port that I'm calling menu_ap or Menu Advanced Permissions. Feedback is highly desired.

If azuledu is no longer interested in maintaining I'd be happy to take over or even co-maintain.

My goal is to get something in a stable format that I can hopefully turn around as a D7 patch to try and get into core.

Anonymous’s picture

StatusFileSize
new3.23 KB

D'oh!

The install file was borked on that last one .. this one should actually install properly.

casey’s picture

I am also interested in becoming a co-maintainer.

bgilday’s picture

subscribing

Anonymous’s picture

Working on an update to the menu_ap code that I previously posted and discovered a critical bug that likely impacts the menu_stp code posted earlier by azeledu.

http://drupal.org/node/326210

mrfelton’s picture

sub

jacerider’s picture

subscribe

ellen.davis’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new9.79 KB

I have made some changes to menu_stp.module. Attached is a patch to be applied to menu_stp-6.x-1.x-dev. The module is working on my test site under drupal 6.10.

I don't normally work on updating modules, so the code needs some review and testing.

ellen.davis’s picture

StatusFileSize
new1.59 KB

At looking at this again, it looks like most of my changes in the patch at comment 14 were already implemented in the zip file in comment 3 by azuledu. For some reason, I thought menu_stp-6.x-1.x-dev was newer than the zip file. So ignore my patch in 14 and go with the zip in 3.

I do have one patch for menu_stp from menu_stp.zip in comment 3. When using menu_st from comment 3, I got the below error when trying to create a page that had special permissions set. The patch fixes this problem.

Fatal error: Maximum execution time of 30 seconds exceeded in .../drupal-6.10/includes/database.mysql.inc on line 108

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 40924729 bytes) in .../drupal-6.10/includes/database.mysql.inc on line 321

egfrith’s picture

From the CVS commits in January, it looks like azeludu is now a co-maintainer. If azeludu and ray007 agree for azeludu to be the module maintainer, I think the procedure is to move this issue to the webmasters' queue. See (http://drupal.org/node/251466). If azeludu and ray007 are happy being co-maintainers, this issue should be closed.

egfrith’s picture

Ooops: I hadn't seen this issue in the webmasters queue #340771: Menu Subtree Permissions module appears to be abandoned.

Andreas Wolf’s picture

sub

dmsmidt’s picture

Good to hear new people are willing to take this module to the next level. Ray007 doesn't seem to have done anything since August 24, 2007.
I've been searching for a module like this for ages. (see http://drupal.org/node/328272)

I really need this in a production site, any chance for a stable version for d6 soon? Current dev doesn't work at all (prints arrays on every page and doesn't add any extra fieldset for permissions..)

casey’s picture

I tracked down his mail-address and have sent him an e-mail last week; no response however.

casey’s picture

Project: Menu Subtree Permissions » Drupal.org site moderators
Version: 6.x-1.x-dev »
Component: Code » Project ownership

http://drupal.org/node/251466

If there is no response or if the maintainer responds and gives the ok, move the issue to the webmasters queue to bring it to the attention of someone with access to make the change. Please include a link back to the project page to make it easier for the admin.

Project: http://drupal.org/project/menu_stp

casey’s picture

Title: Menu_stp 6.x-dev. Co-maintainer » Menu_stp appears to be abandoned
casey’s picture

Project: Drupal.org site moderators » Menu Subtree Permissions
Version: » 6.x-1.x-dev
Component: Project ownership » Code
Status: Needs review » Fixed

Ow azuledu seems to be maintainer already.

casey’s picture

Title: Menu_stp appears to be abandoned » Menu_stp 6.x-dev. Co-maintainer
rwohleb’s picture

The above patches, plus additional refactoring, have been rolled into a new patch at http://drupal.org/node/430564. I did this to keep a clean issue for a D6 patch.

rwohleb’s picture

Status: Fixed » Closed (fixed)

I was able to reach the old maintainer and he agreed to hand over the reins. I've just committed my D6 working copy, so feel free to grab it from CVS, or wait for the dev package to update. If people are happy with it, we can roll a beta release.