This came up while working on #1939088: Convert theme_meter() to Twig, after applying the patch there which replaces theme_meter() with a Twig template.

Per http://api.drupal.org/api/drupal/developer%21topics%21forms_api_referenc... the class should be an array.

Without this patch I got a fatal error when this code in template_preprocess was hit:

// Initialize html class attribute for the current hook.
$variables['attributes']['class'][] = drupal_html_class($hook);
CommentFileSizeAuthor
#1 poll-1966090-1.patch406 bytesstar-szr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

star-szr’s picture

Status: Active » Needs review
FileSize
406 bytes

Status: Needs review » Needs work

The last submitted patch, poll-1966090-1.patch, failed testing.

adammalone’s picture

Status: Needs work » Fixed

Thanks for the pick up and patch - fixed.

star-szr’s picture

Thank you very much for the quick fix, this will make our manual testing much easier.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Update report