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 :)
 
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 Bottom