Add common CSS class to all product node pages

Hmm, pickaxe doesn't show me any results for 'node-type-product' anywhere in the Ubercart source code history. Specifically when in the past was this working for you? Perhaps you had a custom template for the product class that added this?

template_preprocess_html() in core has the following code:

  // If on an individual node page, add the node type to body classes.
  if ($node = menu_get_object()) {
    $variables['classes_array'][] = drupal_html_class('node-type-' . $node->type);
  }

I guess we could add another preprocess function to add another body class for Ubercart products; I'd prefer not to reuse node-type-product but a separate class so themers can target the actual "product" type, or all products.

Product classes use node--product.tpl.php because Ubercart implements uc_product_preprocess_node() to allow this. But if you ever do define a node--myclass.tpl.php for a "myclass" product type, this more specific template will take precedence.

Repurposed this as a feature request. The attached patch adds a "uc-product-node" class to the body tag of all product node pages, including a test to prove this.

Committed #5

If this worked "in the past" then you must have had some custom code to do it - #5 is a pure D7 solution, and is not something that was ever implemented in Ubercart.