At the moment the flag status is not included:
if ($variables['after_flagging']) {
$inverse_action = ($action == 'flag' ? 'unflag' : 'flag');
$variables['message_text'] = $flag->get_label($inverse_action . '_message', $content_id);
$variables['flag_classes_array'][] = $variables['status'];
}

But I can't find any documentation about "after_flagging" ... can't we just have the flag status by default?

Comments

quicksketch’s picture

Category: feature » task

Yes that does seem a bit odd. I'll need to check what the value of $variables['status'] is. Regarding "after_flagging", that variable is set to TRUE if the template is being printed as the return value of an AJAX flagging.

joachim’s picture

Version: 6.x-2.x-dev » 7.x-3.x-dev
Category: task » feature

Seems a reasonable request, upping version.