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

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new 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 commented
Status: Needs review » Fixed

Good patch!

its in .

Status: Fixed » Closed (fixed)

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