Closed (fixed)
Project:
UC Node Access
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
2 Oct 2008 at 22:19 UTC
Updated:
23 Mar 2010 at 14:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
Renee S commentedsubscribe
Comment #2
jody lynnHere's a patch for a Drupal 6 version.
It needs testing.
Comment #3
lonehorseend commentedJust looked at this patch and workflow-ng has been renamed rules for Drupal 6 according to this thread: http://drupal.org/node/267755. You can find the module here: http://drupal.org/project/rules.
Comment #4
jody lynnUpdated patch.
@lonehorseend, Ubercart no longer relies on workflow_ng/rules but uses its own conditional actions module.
Comment #5
technivant commentedThanks for the patch. Unfortunately, it doesn't seem to work for me as it doesn't assign any values to $access. If I hardcode the $access['access_nid'] value into function uc_node_access_grant_user_access it assigns ACL permissions correctly.
I've narrowed the problem (I believe) to the sql in:
function uc_node_access_feature_load($pfid) {
$access = db_fetch_array(db_query("SELECT nap.*, pf.nid FROM {uc_node_access_products} AS nap LEFT JOIN {uc_product_features} AS pf ON nap.pfid = pf.pfid WHERE nap.pfid = %d", $pfid));
It also seems to return an empty array for me. Is there anything obviously wrong here?
Thanks!
Comment #6
jody lynnYou were right technivant - the new node access features were not saving correctly when saving the 'features' tab.
Here is a fixed patch, working for me.
Comment #7
rszrama commentedThanks so much for the help. I've combined this patch w/ my own upgrade that was in progress and come up w/ a dev release you should be able to test. A 6.x-1.0 release will be packaged up ASAP.
Comment #9
marty.true commentedIs there any chance of a 6.x-1.0 release? You mentioned it would be packaged ASAP over a year ago and still haven't seen it :( This is very much needed. Please advise where this stands.