Fixed Marking alert as read/unread does not respect $redirect

[TickTackk]

Affected version

2.2.1

First in template account_alerts_mark_read this:

    <xf:redirect />
    <xf:hiddenval name="unread">{{ $newUnreadStatus ? 1 : 0 }}</xf:hiddenval>
</xf:form>

should be:

    <xf:redirect url="{$redirect}" />
    <xf:hiddenval name="unread">{{ $newUnreadStatus ? 1 : 0 }}</xf:hiddenval>
</xf:form>

Then in XF\Pub\Controller\Account::actionAlertToggle() this:

        $redirect = $this->buildLink('account/alerts');

should be:

        $redirect = $this->getDynamicRedirect($this->buildLink('account/alerts'));

Edit: If someone were to go to page 69 and then open the "Mark as read" or "Mark as unread" for an alert in a new tab, click submit then they would be redirected back to page 1 instead of the last page they were on.

 

[TickTackk]

This also applies to account_alerts_mark_read template and XF\Pub\Controller\Account::actionAlertsMarkRead().

 

[XF Bug Bot]

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.2).

Change log:

Fix dynamic redirects for alert and conversation read state toggles

There may be a delay before changes are rolled out to the XenForo Community.