Fixed push_user_post_merge renders HTML prefix

DragonByte Tech

Well-known member
Affected version
2.2.8
push_user_post_merge
HTML:
{{ phrase('your_posts_in_thread_x_were_merged_together', {
    'source': prefix('thread', $extra.prefix_id) . $extra.title
}) }}
<xf:if is="$extra.reason">{{ phrase('reason:') }} {$extra.reason}</xf:if>
<push:url>{{ base_url($extra.threadLink, 'canonical') }}</push:url>

push_user_post_move
HTML:
{{ phrase('your_post_in_thread_x_was_moved_to_y', {
    'title': $extra.title,
    'target': prefix('thread', $extra.prefix_id, 'plain') . $extra.targetTitle
}) }}
<xf:if is="$extra.reason">{{ phrase('reason:') }} {$extra.reason}</xf:if>
<push:url>{{ base_url($extra.targetLink, 'canonical') }}</push:url>

Missing plain argument from push_user_post_merge :)
 

XF Bug Bot

XenForo bug fixer bot
Staff member
Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future XF release (2.2.9).

Change log:
When sending a push notification about a post being merged, avoid rendering the prefix as HTML
There may be a delay before changes are rolled out to the XenForo Community.
 
Top