Lets remove php logic from templates. i think they should be as simple as 'if something is set, print it'. so we need to make sure we implode actions into LI in preprocess function.

 <?php foreach ($message_actions as $action): ?>
            <li><?php print $action ?></li>
          <?php endforeach; ?>
CommentFileSizeAuthor
#1 privatemsg.actions.patch3.16 KBBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Active » Needs review
FileSize
privatemsg.actions.patch3.16 KB

The attached patch does use theme_links to display them. To do this, I needed to change

$vars['message_actions'][] = l(t('bla'), 'url');
// to
$vars['message_actions'][] = array('title' => t('bla'), 'href' => 'url');
litwol’s picture

litwol CreditAttribution: litwol commented
Status: Needs review » Fixed

Good patch!

its in .

Status: Fixed » Closed (fixed)

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