1 reply [Last post]
urwen's picture
Offline
Joined: 01/09/2008
Juice: 20
Was this information Helpful?

Hi everybody,

i would like to help on the development. I´ve modified deeply the module to feet the needs of a customer and i´ve seen some things that i could implement on ubercart.

The most light is, for example, the title on the products listing. There is no way to hide the title on the beginning of the table. I´ve commented the line 694:

if (count($catalog->products)){
if (count($links)){
$output .= theme('links', $links, array('class' => 'links inline')) ."\n";
}

//$output .= $catalog->description; // <<<< ------ title commented by me
$output .= theme('uc_catalog_products', $catalog->products);
$output .= theme('pager');
}

but there is a more elegant way to do this adding a checkable option in the administrator which offers the user to show or not to show this. Something like:

$form['catalog-display'][''] = array('#type' => 'select',
'#title' => t('Show or not the title in product listings table'),
'#default_value' => variable_get('uc_catalog_category_columns', 0),
'#options' => drupal_map_assoc(uc_range(1, 5)),
);

$form['catalog-grid']['uc_catalog_display_title'] = array(
'#type' => 'checkbox',
'#title' => t('Show or not the title in product listings table'),
'#default_value' => variable_get('uc_catalog_display_title', true),
);

and in the commented code:

if (variable_get('uc_catalog_grid_display_title', TRUE))
$output .= $catalog->description;

¿what do you think about my little suggestion? ¿How could i access the cvs to help in things like that or any other improvments in which i could help?

regards

Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: help on dev offered

Thanks, urwen. We definitely appreciate the off to pitch in and help. We don't have just an open CVS or anything for folks to commit to but work largely through suggestions, discussion, and bugfix patches. Right now in the beta phase we're trying to knock out known bugs and make sure the core is solid before putting it out as 1.0. For more information on where to pitch in and to see what's been done, check out the beta-to list.

For your example, you can actually turn off the title in the table display settings form at /admin/store/settings/tables/uc_product_table. Eye-wink