It seems like logs of people are getting these same errors.

Does Node Checkout work anymore? I just can't get it to work in combination with the attributes module.

Line 390 seems to be the culprit. I am running Ubercart 7.x-3.4 With UC 7.2.dev. I have tried every patch available, different versions - no matter what - I always get the below. The attribute pricing is not transferred to the checkout after creating a node, and it just shows the base pricing.

Warning: Missing argument 2 for _uc_attribute_alter_form(), called in /home/public_html/sites/all/modules/uc_node_checkout/uc_node_checkout.module on line 390 and defined in _uc_attribute_alter_form() (line 1273 of /home/public_html/sites/all/modules/ubercart/uc_attribute/uc_attribute.module).
Warning: Missing argument 3 for _uc_attribute_alter_form(), called in /home/public_html/sites/all/modules/uc_node_checkout/uc_node_checkout.module on line 390 and defined in _uc_attribute_alter_form() (line 1273 of /home/public_html/sites/all/modules/ubercart/uc_attribute/uc_attribute.module).

Comments

mr.andrey’s picture

mr.andrey commented

subscribing...

todos27’s picture

todos27 commented

Hello guys, did you find a solution to this problem?

I'm getting the same errors.

Cheers,

stadulis66’s picture

stadulis66 commented

Yes this issue has been resolved.

Node Checkout 7.x-2.x-dev

With this modification:

http://drupal.org/node/1444596 (see bottom).

And here is my fix:

Change:
//it's for Drupal 6 (i think)

$attributes = _uc_attribute_alter_form(node_load($product_nid));

To:
//it's for Drupal 7

$attributes = _uc_attribute_alter_form($product_nid, node_load($product_nid), FALSE);

However, to add attributes when you check out, i have to make:

Change:

// Pass these values to modules for alteration.
$extra = module_invoke_all('add_to_cart_data', $values);

To:

// Pass these values to modules for alteration.
    $extra = module_invoke_all('uc_add_to_cart_data', $values);

Besides, you have to enable settings on this module for the product (Video Promotion).

http://www.digitalworkhorse.com

spyderpie’s picture

spyderpie commented

I am also getting these errors but it is not clear to me how to integrate this and the other referenced fix into a solution. Is it possible the fixes will be rolled into the module soon?

yaworsk’s picture

yaworsk commented

there hasn't been much movement on updates in over a year - the last commit was 2012-Jan-27. I was interested in working on the module but never got a response back (a little disappointing). I'd suggest checking out the Pay Per Node module depending on the functionality you need. It is actively maintained https://drupal.org/project/paypernode

spyderpie’s picture

spyderpie commented

I just tried that module and it works beautifully except .. I like that in node checkout, the user can create their node prior to paying for it .. its more tangible .. Pay per node leaves the user with no clue where what they just paid for is .. I know I can create rules and such .. i just liked the flow better of this module ... maybe offer to help on pay per node to integrate the two features ....

I wish I were a programmer ..... sigh

peace and thanks for the advice ;)

Julie

allegragf’s picture

allegragf commented

If you replace the line 390 with this code the in the "uc_node_checkout.module" error should go away

$thisNone = node_load($product_nid);
$attributes = _uc_attribute_alter_form($form['#form_id'], $thisNone, FALSE);
yaworsk’s picture

yaworsk commented

@spyderpie, yup, i agree, it is nice to have the pre-purchase node creation.

@allegragf, appreciate the suggestion but what is the point of hacking up the module if it isn't going to be maintained? kind of unreliable to use it on a production site, especially one related to commerce and the exchange of money.

allegragf’s picture

allegragf commented

Drupal is a open source project, that means code is being contributed by many users. This is not a hack, it is a patch, because _uc_attribute_alter_form() use to only require 1 argument, now it requires 3. This code fixes the problem. From this post it seem I am suppose to post this patch on this thread https://drupal.org/patch/submit

The uc_node_checkout module has nothing to do with the exchange of money.

yaworsk’s picture

yaworsk commented

I don't want to detract from their thread any more than I already have so I won't post anything here after this but what I meant was, why patch / use a module that isn't going to be maintained. That patch isn't going to be committed to a dev or stable version, nothing has been committed in over a year. As a result, there could be security issues here. Because of that, I would never recommend using this module on a production site.

And this module is meant to integrate with Ubercart (the uc in uc_node_checkout), an ecommerce module. It binds a node type to a product which can be purchased, so yes, it would be used in relation to purchase and the exchange of money.