Adding additonal confirmation box 2 chkout

[A-1 Electronics]

Now I have found additonal files that are attached. They are english.php and header_php.php. If I make changes to these files as well will this accomplish what I am looking for? I can not locate any additional files at this time. But I will keep looking.

[A-1 Electronics]

Went back and changed the wording for each of the conditions. No I do need to add a line between each one. And I made changes to english.php and header_php.php. It is still letting me go on without having those two boxes checked. Still searching though.

[A-1 Electronics]

I meant Now I do need to add a line between each one. And I still can not figure out how to make it manditory that all the boxes must be checked.

[kobra]

Looks like you have made some progress!!!!

How about a url to this so we can see where you are and possibly view your generated source code

[A-1 Electronics]

Warning - This site contains adult content.

If you would prefer, I could copy the source code for you. Otherwize, it is adultks.com.
So I am down to needing the last two things: Making sure that the boxes are checked and moving each box to it's own line.

[kobra]

Without my stepping through each step of this i used the developers tool kit and searched for "NOT_ACCEPTED"

the following is from source of one of my sites:

<fieldset>
<legend><span class="termsconditions">Terms and Conditions</span></legend>
<div><span class="termsdescription">Please acknowledge the terms and conditions bound to this order by ticking the following box. The terms and conditions can be read <a href="http://www.mysite.com/index.php?main_page=conditions"><span class="pseudolink">here</span></a>.</div>
<input type="checkbox" name="conditions" value="1" id="conditions" /><label class="checkboxLabel" for="conditions"><span class="termsiagree">I have read and agreed to the terms and conditions bound to this order.</span></label>
</fieldset>

and the code for validation is in /includes/modules/pages/checkout_confirmation/header_php.php and is as follows:

if (DISPLAY_CONDITIONS_ON_CHECKOUT == 'true') {
  if (!isset($_POST['conditions']) || ($_POST['conditions'] != '1')) {
    $messageStack->add_session('checkout_payment', ERROR_CONDITIONS_NOT_ACCEPTED, 'error');  }

Now you need to replicate at least this for each of your new check boxes and of course change the if to the new box defines.
The initial "if" refers to a condition set in the admin and this is another whole bag of code and if all other things are covered you might just omit these for your added boxes

if (DISPLAY_RETURNS_ON_CHECKOUT == 'true') { and
if (DISPLAY_CCBILLING_ON_CHECKOUT == 'true') {

I assumed that you have defined a class for each of the added boxes but your source does not indicate that you have and this should be in the tpl_ file for this and you can also add a clear both between statements there.
This should get you closer

[A-1 Electronics]

New code for tpl_checkout_payment_default.php

<?php
if (DISPLAY_CONDITIONS_ON_CHECKOUT == 'true') {
?>
<fieldset>
<legend><?php echo TABLE_HEADING_CONDITIONS; ?></legend>
<div><?php echo TEXT_CONDITIONS_DESCRIPTION;?></div>
<div><?php echo zen_draw_checkbox_field('conditions', '1', false, 'id="conditions"');?>
<label class="checkboxLabel" for="conditions"><?php echo TEXT_CONDITIONS_CONFIRM; ?></label>
<div><?php echo zen_draw_checkbox_field('returns', '1', false, 'id="returns"');?>
<label class="checkboxLabel" for="returns"><?php echo TEXT_RETURNS_CONFIRM; ?></label>
<div><?php echo zen_draw_checkbox_field('ccbilling', '1', false, 'id="ccbilling"');?>
<label class="checkboxLabel" for="conditions"><?php echo TEXT_CCBILLING_CONFIRM; ?></label>
</fieldset>

Well, I figured out the spacing. However, I am still unable to force the checkbox. I know you can make this selection in the admin panel to turn conditions on and off. I hope I don't need to make a change in the admin panel.

[Ajeh]

If you peek in the:
/includes/modules/pages/checkout_confirmation/header_php.php

You will notice this test that addes to the message_stack:

if (DISPLAY_CONDITIONS_ON_CHECKOUT == 'true') {
  if (!isset($_POST['conditions']) || ($_POST['conditions'] != '1')) {
    $messageStack->add_session('checkout_payment', ERROR_CONDITIONS_NOT_ACCEPTED, 'error');
  }
} 

Further along, you will notice this redirect when the message_stack is > 0:

if ($messageStack->size('checkout_payment') > 0) {
  zen_redirect(zen_href_link(FILENAME_CHECKOUT_PAYMENT, '', 'SSL'));
} 

[A-1 Electronics]

I do see that. But I don't understand. Maybe I have made an error.

This is the code and the mod I made in header_php.php:

if (DISPLAY_CONDITIONS_ON_CHECKOUT == 'true') {
if (!isset($_POST['conditions']) || ($_POST['conditions'] != '1')) {
$messageStack->add_session('checkout_payment', ERROR_CONDITIONS_NOT_ACCEPTED, 'error');
}
}
if (DISPLAY_RETURNS_ON_CHECKOUT == 'true') {
if (!isset($_POST['returns']) || ($_POST['returns'] != '1')) {
$messageStack->add_session('checkout_payment', ERROR_RETURNS_NOT_ACCEPTED, 'error');
}
}
if (DISPLAY_CCBILLING_ON_CHECKOUT =='true') {
if (!isset($_POST['ccbilling']) || ($_POST['ccbilling'] != '1')) {
$messageStack->add_session('checkout_payment', ERROR_CCBILLING_NOT_ACCEPTED, 'error');
}
}

I know I am missing something very simple.

[A-1 Electronics]

if ($messageStack->size('checkout_payment') > 0) {
zen_redirect(zen_href_link(FILENAME_CHECKOUT_PAYMENT, '', 'SSL'));


Do I need to change > 0 to >2?


Nope. That doesn't work.

How about if I change the second and third instance of checkout_payment to checkout_payment_2 and checkout_payment_3.