Port of Content access for D6

will_in_wi - February 15, 2008 - 19:56
Project:Content Access
Version:5.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

Is there a Drupal 6 version in the works?

#1

haggan - February 21, 2008 - 15:52

I also wonder about this!

#2

j0hn-smith - February 22, 2008 - 09:51

Me too.

#3

forschi - February 22, 2008 - 09:55

A Version for Drupal 6 would be very nice to have.

#4

gellpak - February 25, 2008 - 15:29

I agree, or at least some good access restriction system

#5

acker - February 27, 2008 - 01:03

Yes, with ACL and Forum Access now working with v6 it would be good to be able to permission other content too.

#6

WorldFallz - February 28, 2008 - 17:21

from the node privacy by role page:

...This module is deprecated in favor of Content Access with ACL...

So it seems as if this module is the AC future for drupal-- an update on a D6 version would be greatly appreciated. And if there's anything we can do to help, just let us know....

#7

droogie - March 7, 2008 - 16:57

It would be nice if the author could let us know if there is in fact a Drupal 6 version coming. If not, we need to make other plans. Would hate to start developing something that is about to be released.

#8

miahawk - March 21, 2008 - 23:42

has anyone heard any news on this? I see work is still being done on 5.x, but no word on a 6.x version

#9

flickerfly - April 5, 2008 - 00:56

:-) I have a project coming up that would find this very useful, but the plan is D6 based. I'd also like to know if there is any plans to move forward on it in the near future or if something particular is in the way.

#10

Arkar - April 9, 2008 - 22:30
Title:Drupal 6 Version» Content access Drupal 6 Version

#11

japanitrat - April 14, 2008 - 18:17

yeah i wondered why no module is currently doin the job.
tell us what we can do!

#12

Scott06 - April 14, 2008 - 18:57

Oh yes, I would like to get a D6 Version of content_access too. I hope the developers will publish it in a few days.

Thank you very much for developing! :-)

#13

CosmicVoyager - April 17, 2008 - 19:23

As droogie mentioned in #7, it would be great if someone connected to this module could give us some feedback on when (or if) we can expect a D6 version of Content Access. Any info would help. Thanks.

#14

flickerfly - April 26, 2008 - 00:13
Title:Content access Drupal 6 Version» Port of Content access for D6

This naming scheme seems to be what is being used in other projects. Hopefully will make life easier in tracking the ports of multiple projects to D6. Sorry if its inappropriate for me to do this, just chastise me. :-)

#15

timetwister - May 2, 2008 - 09:26

yes a D6 port would be greatly appreciated, as there, as far as i know, no proper user restriction modules for D6...

#16

kenjiru - May 5, 2008 - 21:45

I'm also waiting for this module to be ported to D6!

#17

mvdschoot - May 14, 2008 - 13:26

First of all, I'm not quiet experienced with Drupal but during my evaluation it seems like a pretty good system. Currently investigating the usability of Drupal 6, but missing one major feature: per node ACL (e.g. display only menu item x to user 2, 3 and 4 and menu item y to user 3, 6 and 9).

Instead of waiting for the Content Access Developers (It seems quiet around there?) to port their module to Drupal 6, going to try it on my own. Volunteers are welcome!

Done so far:
- Module is getting recognized by Drupal 6
- After installation menu option 'Access Control' is available at Admin section.

So: it's still not working, and you're welcome to join!

AttachmentSize
content_access.tar_.gz12.78 KB

#18

Nick Wilson - May 18, 2008 - 08:41

Waiting as well. Considering a downgrade to D5 :(

#19

mvdschoot - May 20, 2008 - 11:54

Updated content_access for Drupal 6. As mentioned: still not working!

To do:
- figure out howto disable cache for content_access menu
- after 'submit' has been clicked, values has to get parsed & stored.

AttachmentSize
content_access.tar_.gz12.67 KB

#20

coltrane - June 2, 2008 - 21:02

mvdschoot, can you post your work as a patch against the branch DRUPAL-5 of content_access? See http://drupal.org/patch

#21

flickerfly - June 5, 2008 - 20:24

watching eagerly

#22

BENNYSOFT - June 8, 2008 - 20:23

Subscribing

#23

cwdrup - June 9, 2008 - 07:35

Subscribing

#24

BENNYSOFT - June 9, 2008 - 11:19

It would be nice if the author could let us know if there is in fact a Drupal 6 version coming.

I've asked the developer whether he continues the development of this module.

#25

fago - June 10, 2008 - 12:56

It's continued, but I'm currently busy with other things/ports. So I probably won't port it before July.

If someone is quicker, just do it and I'll commit it. Porting shouldn't be a big deal here.

#26

fago - June 10, 2008 - 12:58
Status:active» patch (code needs work)

ah, and yes please use patches against 5.x-dev. When there is a working patch I'd be happy to commit it.

#27

BenK - June 10, 2008 - 14:01

Subscribing as well... looking forward to D6 version!

#28

boulwarek - June 11, 2008 - 09:30

subscribing

#29

xaweryz - June 13, 2008 - 08:40

Conntet access patch for content_access-5.x-1.x-dev

Nightly development snapshot from CVS branch: DRUPAL-5
Download: content_access-5.x-1.x-dev.tar.gz
Size: 12.57 KB
md5_file hash: 667adbdfd0f82bc6161e534b3698c7d9
First released: May 17, 2007 - 20:40
Last updated: June 11, 2008 - 00:02

AttachmentSize
content_access.patch6.37 KB

#30

xaweryz - June 13, 2008 - 08:42
Status:patch (code needs work)» patch (code needs review)

#31

xaweryz - June 13, 2008 - 08:59

Errata patch
how to set more than one user rights in hook_menu ??

AttachmentSize
content_access.patch6.45 KB

#32

modctek - June 16, 2008 - 23:44

Tagging this for subscribing

#33

mark_r - June 19, 2008 - 15:18

tagging.

question: why a patch for content_access-5.x-1.x-dev if we talk about having a dp6 version?

#34

mark_r - June 19, 2008 - 15:18

tagging.

question: why a patch for content_access-5.x-1.x-dev if we talk about having a dp6 version?

#35

coltrane - June 19, 2008 - 16:35

@mark_r, it has to do with the way projects are hosted on Drupal.org and tied in with the CVS repository. There isn't a DRUPAL-6--x branch of content_access so patches need to apply to another branch. fago removed the code from the HEAD branch of the cvs code so only the DRUPAL-5 branch is available. Patches need to be against the latest code of this branch which is why this issue is marked 5.x-1.x-dev.

#36

ddanier - June 20, 2008 - 08:29

Wouldn't it be possible to add xaweryz to the list on content_access developers, so he could work on the CVS-code directly (in its own D6 branch of course)? This way it should be easy to push out some dev-download for D6, as this patch works pretty well...didn't have any problems with it so far.

Just curious....

#37

fago - June 20, 2008 - 12:28
Status:patch (code needs review)» patch (code needs work)

>how to set more than one user rights in hook_menu ??
you need to write your own access callback

@ddanier: the patch misses proper access checks. Furthermore there also some tabs in there - please only use spaces for indention (drupal code style!). When this is fixed, I'm happy to open a 6.x branch.

#38

ddanier - June 20, 2008 - 16:42

> Furthermore there also some tabs in there - please only
> use spaces for indention (drupal code style!).

Thats nothing I would really worry about as my editor is able to convert tabs to spaces (I use it the other way round). I think many other editors have similar features. So the tabs-issue should not be the show-stopper...but missing access checks are. ;-)

#39

lhorthy - June 24, 2008 - 15:45

subscribing

#40

xaweryz - June 24, 2008 - 22:48

Ok. conntent_access patch with own user access callback :-)

#41

xaweryz - June 24, 2008 - 22:50
Status:patch (code needs work)» patch (code needs review)
AttachmentSize
content_access.patch7.07 KB

#42

xaweryz - June 25, 2008 - 16:45

Only change function name
from content_user_access
to content_access_user_access

AttachmentSize
content_access_4.patch7.09 KB

#43

xaweryz - June 26, 2008 - 14:31
Assigned to:Anonymous» xaweryz

#44

flickerfly - June 28, 2008 - 03:07
Status:patch (code needs review)» patch (code needs work)

I have 6 content types on my testing site. The patch applied cleanly.

I installed the module and went to permissions to edit the Content Access. In this area I had two sections added "content_access module" and "content_permissions module". The first has "grant content access" (duplicate of existing permission?) and the second has "grant own content access". The content_permissions module only listed 2 items under it. both had to do with granting contest access and granting own content access of the same content type. The other 5 content types were not present.

I don't think this patch worked for me, unless I don't know what I'm looking for. For this reason, I'm marking it needs more work.

Anything specific about it you want me to look into?

#45

xaweryz - June 28, 2008 - 10:36
Status:patch (code needs work)» patch (code needs review)

1. I dont't know what is content_permissions module ??
2. in Permissions path=> (admin/user/permissions) there are only two permissions .. And is it ok!!
3. Please check path (admin/content/node-type/page) or edit other node-type page.
There is "Access control" page from conntent access module. Not in Permissions!!!

And i change status to patch (code needs reviwew)

#46

xaweryz - June 28, 2008 - 10:36

1. I dont't know what is content_permissions module ??
2. in Permissions path=> (admin/user/permissions) there are only two permissions .. And is it ok!!
3. Please check path (admin/content/node-type/page) or edit other node-type page.
There is "Access control" page from conntent access module. Not in Permissions!!!

And i change status to patch (code needs reviwew)

#47

flickerfly - June 28, 2008 - 12:58

On #1: Oh, I see, content permissions is a CCK thing. Nothing to do with this. Sorry for the confusion.

On #2: Yes, I see both those permissions. They appear fine.

On #3: :-) That's what I was looking for before. Thank you for the directions. It looks good, but I have a few questions.

Why doesn't the author have view, edit and delete permissions by default? In my view they are all unchecked.

The "Give Node Grant Priority" description leads me to believe that this is a module wide configuration. If that is so, shouldn't it be on a separate configuration screen specific to the module?

When giving per node control, it would seem to be more consistent with the typical drupal interface to have the controls in one of the lists at the bottom like publishing options, input format and the like. I'm thinking about this module for users to handle the security of their own blog posts and things like that so I'm interested in this being as easy as possible for them to catch on to.

I have not used the 5.x version of this module so I expect that my questions have to do with how this module is implemented in a more global sense so probably shouldn't slow down the release of this module.

#48

flickerfly - June 28, 2008 - 18:55

In the forum content type, under access control, I'm getting this error message:

    * warning: Illegal offset type in isset or empty in /home/user/public_html/includes/bootstrap.inc on line 812.
    * warning: Illegal offset type in /home/user/public_html/includes/bootstrap.inc on line 813.
    * warning: Illegal offset type in /home/user/public_html/includes/bootstrap.inc on line 817.

#49

xaweryz - June 28, 2008 - 19:51

On clean install drupal 6 plus conntent_access
i don't have any warnings.

    * warning: Illegal offset type in isset or empty in /home/user/public_html/includes/bootstrap.inc on line 812.
    * warning: Illegal offset type in /home/user/public_html/includes/bootstrap.inc on line 813.
    * warning: Illegal offset type in /home/user/public_html/includes/bootstrap.inc on line 817.

this related to bootstrap.inc

#50

fago - June 30, 2008 - 09:03
Status:patch (code needs review)» active

ok, I had a short look at it. Thanks for this work!

I've overhauled and cleaned some code, e.g. the menu code and committed in the new d6 branch.
--> http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/content_acc...

I've also fixed several php notices. Please develop with php notices enabled - use a development drupal 6 release as they are disabled in stable release afaik.

What needs to be done:
* test test test
* look at the ACL integration
* don't use the 'all' realm anymore: http://drupal.org/node/239139

So please file further patches again DRUPAL-6--1.
I don't close this issue as I don't consider the port as ready now. Let's have at least the basic functionality tested and verified working before doing a development release.

#51

xaweryz - June 30, 2008 - 17:50

One small patch :-)

AttachmentSize
content_access_5.patch700 bytes

#52

Nick Wilson - July 1, 2008 - 07:04

Thanks for this work xaweryz and fago.

#53

fago - July 1, 2008 - 07:47

xaweryz, one shouldn't use t() for the title any more, as it's now the default title callback, see http://drupal.org/node/140311

#54

mabublue - July 3, 2008 - 19:24

subscribing

#55

yas - July 3, 2008 - 23:17

subscribing

#56

Roxpace - July 16, 2008 - 03:24

This is great, but what makes Content Access better than Node Privacy ByRole (http://drupal.org/project/node_privacy_byrole) ? Just curious if this upgrade is not so prioritized, why not cooperate with others when there is already a finished solution like the other module ? Which is already at version 1.3.

#57

WorldFallz - July 16, 2008 - 13:37

It's not really a matter of "better". They're different. Content Access allows for more flexibility than node_privacy_byrole. If the functionality you need is covered by node_privacy_byrole, then Content Access would probably be overkill.

#58

designanddraft - July 17, 2008 - 00:39
Component:Code» User interface
Category:task» feature request
Assigned to:xaweryz» Anonymous

drupal 6.x, v6 transparent admin
I am looking for transparent administration, open, visible and sometimes editable admin settings with security. based on permissions, locked individually (as a node?), password protected, view and/or edit choice for admin inserted at the top or bottom of all admin/settings pages. even better, password or radio button next to every settings choice as viewable and/or editable similar to form field protect. a prominent notification to the user, “you can view this page of administration settings, if they require edit, please email administration with your request link”. Possibly even attached comment or site note section, ability to insert instruction or voting widget.
I gather this is the approach taken by adding access control to the core and maybe it is similar to drupal core building. I have been looking at everything I can find and I’m not a programmer yet, so I need to pass what I’ve found on to someone else here. If someone can put these pieces together or have experience with something similar with any version 4,5,6,7. i'm listing this at all of these locations aswell. Rsvp, elisa

http://drupal.org/node/109157 access control
http://drupal.org/node/131101#comment-735943 Add disabled attribute to protected fields
http://drupal.org/node/245900 How to Protect Nodes From Editing on Demand
http://drupal.org/node/31143#comment-796152 Node Privacy by Role not working
http://drupal.org/project/nodeaccess nodeaccess
http://drupal.org/node/126129#comment-703933 password protect a page
http://drupal.org/node/222263#comment-902378 Port of Content access for D6
http://drupal.org/node/29991#comment-51352 Protected content
http://drupal.org/node/13266 Show block to certain users or roles only- Drupal 4.6
http://drupal.org/project/simple_access simple access
http://drupal.org/node/118404#comment-735895 View, but not edit, field at node/xxx/edit

#59

BENNYSOFT - July 17, 2008 - 12:59
Component:User interface» Code
Category:feature request» task

I’m not a programmer yet

Then do not change the settings of an existing issue. Please open a new issue.

#60

thomas23@drupal.org - July 24, 2008 - 23:57

subscribing

#61

TheresaM - July 27, 2008 - 00:36

subscribing

#62

rismondo - July 27, 2008 - 19:00

subscribing

#63

diodata - July 28, 2008 - 20:29

subscribing

#64

stefan_seefeld - August 2, 2008 - 23:05

The above comments suggest the D6 port is under way. That's great ! What is the reason there is no D6 development snapshot (yet) ? Having that may help others experiment and test.
Thanks !

#65

pacemkr - August 3, 2008 - 20:07

Subscribing

#66

ggamba - August 4, 2008 - 10:26

subscribing

#67

emptyvoid - August 12, 2008 - 22:08

So I ported content access over to Drupal 6.3 but decided to call it nodeaccess (much to my surprise) there is already a module called that in the repository. So I reviewed the code for nodeaccess, yuck. Regardless neither of the existing modules provided the level of sophistication I needed for my clients. And I found the interface ergonomics of content access are far superior to the node access module. I also didn't like the ACL module so I rewrote my code to exclude it.

So either my code can be useful for the team or not. I have no interest in going through the code review, posting via cvs stuff. so I am willing to post the code here is someone else would like to manage it.

#68

cwdrup - August 13, 2008 - 09:16

Hi Emptyvoid,

Could you provide it as a dev for testing ?

Thank you

#69

shrop - August 17, 2008 - 12:49

Subscribing

#70

emptyvoid - August 22, 2008 - 17:28

Unfortunately I don't have a location to host the build.. I can provide the source as it is and everyone is welcome to review, fix, or otherwise bitch me out for my poor coding ;P.

Update!! - Employer requested I remove source until formalized policies are defined for distributing company/client source code.

#71

BENNYSOFT - August 19, 2008 - 11:32

This isn't the right place for a new or modified module. This is an issue for Content Access.

Please open a new issue, assign the issue to yourself and post your code for review.
Just my two cents.

#72

cwdrup - August 19, 2008 - 11:56

Hi Bennysoft,

It seems to be about content access port. Isn't it ?

#73

fago - August 19, 2008 - 14:55

It's not.

@everyone waiting for the port: I'll have a look at it on Thursday!

#74

dirtabulous - August 25, 2008 - 19:30

That's great! Been using the cvs version for quite some time. Seems to be working well... at least we haven't had any problems with it.

#75

jalama - August 28, 2008 - 15:05

subscribe

#76

fago - September 4, 2008 - 10:14
Status:active» fixed

I've just created the first dev release. It basically works, but needs some polishing. I have to look at the ACL integration it seems to somehow fail. Furthermore workflow-ng integration still needs to be ported to rules.

#77

fago - September 4, 2008 - 10:14

Please open separate 6.x issues for 6.x related stuff.

#78

werbeklaus - September 4, 2008 - 13:16

I am sorry, but am I too stupid, or is there no 6.x(-dev) release yet? At least I didn't find it ...

#79

WorldFallz - September 4, 2008 - 13:46

There is now-- it takes 12 hours for the packaging script to do it's business.

#80

balmas - September 12, 2008 - 16:50

Hi,

I've downloaded the 6.x version and it seems to be working fine. Can you provide details on any specific issues to be concerned about for use in a production site? I'm using it to restrict content by content-type, and don't need any particular workflow related functionality.

Thanks.

#81

jparets - September 17, 2008 - 08:15

#82

Anonymous (not verified) - October 1, 2008 - 08:22
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.