I ran into a use case where i needed to rebuild a uc_node_checkout governed node with AHAH. In the process when uc_node_checkout_form_alter is called it returns an HTTP ERROR 0. The culprit seems to be below line 290 when it checks for the product node association. First it checks for the nid in the array returned by uc_node_checkout_product_map() then it checks for it in the $_GET variable, and then it defaults to sending the user to a form to select one. This redirection is what seems to be returning the HTTP ERROR 0. I added a quick check for $_POST['ucnc_product_nid'] and it appears to be working, though I'm not sure if this will cause other issues someplace else. Here is the code:

  // Alter the node forms for UC Node Checkout governed node types.
  foreach (uc_node_checkout_product_map() as $type => $value) {
    if ($form_id == $type .'_node_form') {
      // Determine the product nid that will be added to the cart when this
      // node is created.  First look for a single product node association.
      if ($value['nid'] && !$value['view']) {
        $product_nid = $value['nid'];
      }
      elseif ($_GET['product_nid']) {
        // Otherwise look in the $_GET array.
        $product_nid = $_GET['product_nid'];
      }
      // ADDED by richygecko
      elseif ($_POST['ucnc_product_nid']) {
        // Try the $_POST array
        $product_nid = $_POST['ucnc_product_nid'];
      }
      // END ADDED
      elseif (empty($form['nid']['#value']) && !$form['#programmed']) {
        // If none was found, we must ask the user to specify one.
        drupal_goto('ucnc-select/'. $type);
      }

If this is correct, please let me know and I can supply a patch file.

Comments

aidanlis’s picture

aidanlis commented
Status: Active » Needs review

This seems reasonable ...

aidanlis’s picture

aidanlis commented
Assigned: Unassigned » aidanlis
aidanlis’s picture

aidanlis commented
Status: Needs review » Fixed

I've committed this, but I'm curious as to what module you were using to rebuild the node dynamically and why it was using ucnc_product_nid instead of product_nid.

Thanks,

richygecko’s picture

richygecko commented

Thanks aidanlis, It's a custom module. My customer has a product with a "Print Setup" attribute on it and options for "First time print" and "Re-print". The Re-print option only appears if they have a previous order with this particular product in it. To do this i check if there is a node for this uc_checkout node type and use hook_form_alter to remove the option if there is not.

The customer wanted to have the data from the previous ordered product checkout node pre-filled in the checkout form, and clear the form/re-fill it if the attribute is changed. To do this in the hook_form_alter i attached [#ahah] to the attribute and in the call back is where i call drupal_rebuild_form and the error occurs. I used ucnc_product_nid because I am updating with ahah the request is sent via POST and product_nid doesn't appear in $_POST. Here is the $_POST array from dvm():

array(21) {
  ["attributes"]=> array(2) {
    [8]=> string(2) "18"
    [25]=> string(3) "141"
  }
  ["field_pf_practice_name"]=> array(1) {
    [0]=> array(1) {
      ["value"]=> string(20) "PRACTICE TEST"
    }
  }
  ["field_pf_promo"]=>array(1) {
    [0]=> array(1) {
      ["value"]=> string(1046) "Some test text here"
    }
  }
  ["field_pf_logo"]=> array(1) {
    [0]=> array(2) {
      ["fid"]=> string(3) "637"
      ["list"]=> string(1) "1"
    }
  }
  ["menu"]=> array(3) {
    ["link_title"]=> string(0) ""
    ["parent"]=> string(15) "primary-links:0"
    ["weight"]=> string(1) "0"
  }
  ["nodewords"]=> array(6) {
    ["abstract"]=> array(1) {
      ["value"]=> string(0) ""
    }
    ["canonical"]=> array(1) {
      ["value"]=> string(0) ""
    }
    ["copyright"]=> array(1) {
      ["value"]=> string(0) ""
    }
    ["description"]=> array(1) {
      ["value"]=> string(0) ""
    }
    ["keywords"]=> array(1) {
      ["value"]=> string(0) ""
    }
    ["revisit-after"]=> array(1) {
      ["value"]=> string(1) "1"
    }
  }
  ["changed"]=>  string(0) ""
  ["form_build_id"]=>  string(37) "form-a9963a8f932f0e38a1174de4cda2ccd3"
  ["form_token"]=>  string(32) "1a9f384b14a2f41149837bb9d3254f20"
  ["form_id"]=>  string(30) "promo_flyers_details_node_form"
  ["default_qty"]=>  string(1) "1"
  ["ucnc_product_nid"]=>  string(2) "13"
  ["setup_node"]=>  string(3) "333"
  ["prev_details_node"]=>  string(3) "334"
  ["log"]=>  string(0) ""
  ["pathauto_perform_alias"]=>  string(1) "1"
  ["comment"]=>  string(1) "2"
  ["name"]=>  string(17) "mountaindesignlab"
  ["date"]=>  string(0) ""
  ["status"]=>  string(1) "1"
  ["promote"]=>  string(1) "1"
}

Status: Fixed » Closed (fixed)

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