Hello,

Thank you for your work on this module. I have a hopefully quick question, how would I go about making the checkbox for the node panel required on the checkout page?

I have a need for users to agree to Terms of Service for only 1 product node, not all. This module provides that nicely. The only problem is the checkbox isn't required. I need that to be required so they have to click the checkbox to proceed w/ checkout.

Any way you can help me would be greatly appreciated. I found the section of code I'm talking about at line 100 of uc_node_panes.module:

if ($row['text'] != '') {
$contents[$id]['agree_terms'] = array(
'#type' => 'checkbox',
'#title' => $row['text'],
'#weight' => 1,
'#required' => true,
'#default_value' => false,
);
}

It looks like the checkbox is set to required but it doesn't turn out that way on the check out page. Very confused here.

Comments

nicholas.alipaz’s picture

nicholas.alipaz commented

I believe it is a bug in core. You should search around for the module that corrects that issue and let me know if it fixes it for you. I would link you but don't remember the name. sorry.

nicholas.alipaz’s picture

nicholas.alipaz commented
Status: Active » Closed (won't fix)

Since this is a bug in core and it is documented somewhere I will have to close it out as "won't fix."

hixster’s picture

hixster commented

I am looking for a solution for this too - it's doesn't seem to make sense to have a checkbox for acknowledging terms and conditions if required is not enforced.

@nicholas could be a little bit more specific about which bug in core causes the issue so we can fix ?

As discussed in this issue http://drupal.org/node/711382 - the http://drupal.org/project/uc_termsofservice module does allow checkboxes to be required. So it should be possible to have this functionality in this module, no?

Please can we reopen this issue? This would be awesome to have this functionality as I don't think there are any other modules that can allow ubercart to provide TOC's per product.

nicholas.alipaz’s picture

nicholas.alipaz commented

Use this and you should be fine:
Checkbox Validate
It is a core bug and I am not going to make this module work around it. Core should fix it.

hixster’s picture

hixster commented

Nicholas thanks for posting that link to Checkbox validate which fixes the required checkbox issue - many thanks