node_access hasn't really been worked on for D7 yet, and it can be improved quite a lot.

greggles points out that the private module is a better example, so we may want to import features from that.

For now, I'm setting the .info file's compatibility to 6.x so that people don't mistakenly think this is ready. I'm also removing the mistaken load schema from hook_install().

CommentFileSizeAuthor
#4 examples.node_access_tests.patch19.41 KBrfay
#3 examples.node_access_d7_tests.patch8.93 KBrfay
#1 examples.d7_node_access_update.patch11.71 KBrfay

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Active » Needs review
StatusFileSize
new examples.d7_node_access_update.patch11.71 KB

Here's the D7 version.

Related: #791144: node_access_example.module example for D6 is for D7. The D6 branch has a damaged, incomplete D6 version.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Fixed

Committed: http://drupal.org/cvs?commit=376298.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Fixed » Needs review
StatusFileSize
new examples.node_access_d7_tests.patch8.93 KB

Now for tests

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
StatusFileSize
new examples.node_access_tests.patch19.41 KB

More tests, and now demonstrates node listing and search.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Fixed

Committed to HEAD: http://drupal.org/cvs?commit=376972

Status: Fixed » Closed (fixed)

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