Hello, would it be possible to port it to Drupal 6 or tell me how could I make it? Thank you!

Comments

Renee S’s picture

subscribe

jody lynn’s picture

Title: Port to Drupal 6? » Port to Drupal 6
Version: 5.x-1.x-dev » 5.x-1.0
Category: support » task
Status: Active » Needs review
StatusFileSize
new21.49 KB

Here's a patch for a Drupal 6 version.
It needs testing.

lonehorseend’s picture

Just 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.

jody lynn’s picture

StatusFileSize
new21.45 KB

Updated patch.

@lonehorseend, Ubercart no longer relies on workflow_ng/rules but uses its own conditional actions module.

technivant’s picture

Thanks 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!

jody lynn’s picture

StatusFileSize
new21.41 KB

You 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.

rszrama’s picture

Version: 5.x-1.0 » 6.x-1.x-dev
Status: Needs review » Fixed

Thanks 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.

Status: Fixed » Closed (fixed)

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

marty.true’s picture

Is 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.