bad utilisation of 'attributes' => array('class')
Key 'class' must be an array not a string

$output .= theme('table__diff__revisions', array(
    'header' => $header,
    'rows' => $rows,
    'sticky' => FALSE,
    'attributes' => array('class' => 'diff-revisions'),
  ));

Must be an array

$output .= theme('table__diff__revisions', array(
    'header' => $header,
    'rows' => $rows,
    'sticky' => FALSE,
    'attributes' => array('class' => array('diff-revisions')),
  ));

file : diff.theme.inc
line : 63

CommentFileSizeAuthor
#3 diff-1894120-3-attributes-class-should-be-array.patch893 bytesAlan D.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

michael_cristal’s picture

michael_cristal CreditAttribution: michael_cristal commented

same issue in diff.module file
function diff_node_form_build_preview_changes($form, &$form_state)
line 433

'attributes' => array('class' => 'diff'),

must be array

'attributes' => array('class' => array('diff')),

hass’s picture

hass CreditAttribution: hass commented
Assigned: michael_cristal » Unassigned

Looks like you are not working on a patch

Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented
Priority: Normal » Minor
Status: Active » Needs review
FileSize
diff-1894120-3-attributes-class-should-be-array.patch893 bytes

These?

Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented
Title: bad utilisation of 'attributes' => array('class') » Use of scalar class instead of an array in attributes array.

Thanks for reporting. Pushed to dev.

16cd400

Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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