The destination URL (/user) should be fetched from user.page route.

CommentFileSizeAuthor
#4 after_test-2.png153.5 KBmsuthars
#4 after_test-1.png195.49 KBmsuthars
#4 before_test-2.png155.01 KBmsuthars
#4 before_test-1.png198.68 KBmsuthars
#2 drupal-logout-destination-3124769-2.patch781 byteschi

Comments

Chi created an issue. See original summary.

chi’s picture

chi commented
Status: Active » Needs review
StatusFileSize
new drupal-logout-destination-3124769-2.patch781 bytes
msuthars’s picture

msuthars
Location Pune
commented
Assigned: Unassigned » msuthars
msuthars’s picture

msuthars
Location Pune
commented
StatusFileSize
new before_test-1.png198.68 KB
new before_test-2.png155.01 KB
new after_test-1.png195.49 KB
new after_test-2.png153.5 KB

@Chi Thanks for the patch. I run the PHPUnit test for it & it is working as expected for me with Drupal core 8.9.x. PHPUnit test before/after screen-shots are attached.

msuthars’s picture

msuthars
Location Pune
commented
Status: Needs review » Reviewed & tested by the community
msuthars’s picture

msuthars
Location Pune
commented
Assigned: msuthars » Unassigned

  • catch committed 576652b on 9.1.x 
    Issue #3124769 by Chi, msuthars: Fix hardcoded destination in...

  • catch committed 859df06 on 9.0.x 
    Issue #3124769 by Chi, msuthars: Fix hardcoded destination in...

  • catch committed d71705f on 8.9.x 
    Issue #3124769 by Chi, msuthars: Fix hardcoded destination in...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed 576652b and pushed to 9.1.x, cherry-picked back to 8.9.x. Thanks!

Status: Fixed » Closed (fixed)

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