Ubercart

This is G o o g l e's cache of http://www.ubercart.org/forum/support/1649/image_buttons_instead_submits... as retrieved on Aug 2, 2007 18:32:18 GMT.
G o o g l e's cache is the snapshot that we took of the page as we crawled the web.
The page may have changed since that time. Click here for the current page without highlighting.
This cached page may reference images which are no longer available. Click here for the cached text only.
To link to or bookmark this page, use the following url: http://www.google.com/search?q=cache:LuLNjJTN99AJ:www.ubercart.org/forum...

Google is neither affiliated with the authors of this page nor responsible for its content.
These search terms have been highlighted: 08 02 2007

--------------------------------------------------------------------------------

Ubercart
Home
Documentation
Forums
Issues
Livetest
Home » Forums » Ubercart » Support
Image buttons instead of submits for 'Add to Cart' on product pages
Submitted by monkeybeach on Thu, 08/02/2007 - 06:02
monkeybeach
Posts: 10
Joined: 07/04/2007

Thought I'd sussed this then realised the hidden inputs were missing and the button wouldn't add to the cart! I'd be really appreciative if anyone can suggest how to get this to work and keep the code in template.php to avoid messing with module files!

In my template.php:

function phptemplate_uc_product_add_to_cart_form($node){

$form = array();
$form['#base'] = 'uc_product_add_to_cart_form';
$form['nid'] = array('#type' => 'value', '#value' => $node->nid);
foreach ($node->products as $i => $product){
$form['products'][$i] = array(/* '#type' => 'fieldset', */'#title' => $product->title);
$form['products'][$i]['nid'] = array('#type' => 'hidden', '#value' => $product->nid);
$form['products'][$i]['qty'] = array('#type' => 'hidden', '#value' => $product->qty);
}
$form['submit']['#theme'] = 'button';
$form['submit']['#button_type'] = 'image';
$form['submit']['#attributes'] = array(
'class' => 'product-submit',
'src' => '/assets/images/buttons/add-to-cart.gif',
'alt' => t('Cancel'),
);
return drupal_render($form);

}

(This relies on another function in template.php which allows image submits in Drupal. I won't include it here as I think its irrelevant - I know that code works as the image button works)

Adding the code to supposedly insert the hidden inputs so the form knows what product is being added (shown below)...

foreach ($node->products as $i => $product){
$form['products'][$i] = array(/* '#type' => 'fieldset', */'#title' => $product->title);
$form['products'][$i]['nid'] = array('#type' => 'hidden', '#value' => $product->nid);
$form['products'][$i]['qty'] = array('#type' => 'hidden', '#value' => $product->qty);
}

...causes this error in the page:

warning: Invalid argument supplied for foreach() in /homepages/28/d195610550/htdocs/rin/themes/rin/template.php on line 276.

I think this is something to do with $product not being available. Can anyone suggest what precisely is happening and a way to fix it?

Original code in uc_product_kit_module file for your convenience:

function uc_product_kit_add_to_cart_form($node){
$form = array();
//$form['#form_id'] = array('#type' => 'value', '#value' => 'uc_product_add_to_cart_form');
$form['nid'] = array('#type' => 'value', '#value' => $node->nid, );
$form['products'] = array('#tree' => true);
foreach ($node->products as $i => $product){
$form['products'][$i] = array(/* '#type' => 'fieldset', */'#title' => $product->title);
$form['products'][$i]['nid'] = array('#type' => 'hidden', '#value' => $product->nid);
$form['products'][$i]['qty'] = array('#type' => 'hidden', '#value' => $product->qty);
}
$form['submit'] = array('#type' => 'submit', '#value' => t('Add to cart.'), );
return $form;
}

EDIT
=====================================================

For those who want it, the function to add to template.php to allow image type inputs is below - its not my work, its from somewhere on drupal.org but I can't relocate it right now:

/**
* Allow form buttons to be replaced by image buttons
**/

function phptemplate_button($element) {
// following lines are copied directly from form.inc core file:
// Make sure not to overwrite classes
if (isset($element['#attributes']['class'])) {
$element['#attributes']['class'] = 'form-'. $element['#button_type'] .' '. $element['#attributes']['class'];
}
else {
$element['#attributes']['class'] = 'form-'. $element['#button_type'];
}
// My change is type="' . (($element['#button_type'] == "image") ? 'image' : 'submit' ) . '"
return '\n";
}

Add new comment
permalinkThu, 08/02/2007 - 09:03
Lyle

Posts: 569
Joined: 09/26/2006

What's happening is that you've based your form function off uc_product_kit, but using it to override the form for uc_product. $node->products doesn't exist for a product node, so the submit function for the form is really confused.

Also, while your approach looks like it would work, I want to suggest that implementing hook_form_alter() would be a better strategy. This lets you worry only about the parts you are changing, and the rest of the form can take care of itself like it always has. To do this, you'll have to make a module to hold your customizations. An example tutorial can be found at http://drupal.org/node/508. All you need is a .info file, and your form_alter function in a .module file.

The form_alter function could look something like this:

The API page for hook_form_alter() is at http://api.drupal.org/api/function/hook_form_alter/5.