Patch attached. There might be a better way to do this, but this patch works. The changes in this patch are reflected after applying the destination_url.patch found here: http://drupal.org/node/193460

CommentFileSizeAuthor
#5 cas-front-255086-1.patch1.29 KBredndahead
#3 logout.patch1.75 KBmetzlerd
#1 logout_front_2.patch1.35 KBredndahead
logout_front.patch1.37 KBredndahead

Comments

redndahead’s picture

redndahead commented
StatusFileSize
new logout_front_2.patch1.35 KB

Here is a better patch. It handles building the front page url in a more drupal way.

redndahead’s picture

redndahead commented

Patch will apply cleanly to head. No need to apply this: http://drupal.org/node/193460 patch first.

metzlerd’s picture

metzlerd commented
StatusFileSize
new logout.patch1.75 KB

Here's a revised patch that combines an ugly hack do deal with the differences with a strategy submitted under a different issue that also allows for a supplied destination= logout url. Thoughts on this approach?

metzlerd’s picture

metzlerd commented

Woops accidently posted to the wrong isue.

redndahead’s picture

redndahead commented
StatusFileSize
new cas-front-255086-1.patch1.29 KB

Here is an updated patch. Hopefully this could get in it's helpful for my multisite environment.

metzlerd’s picture

metzlerd commented
Status: Needs review » Fixed

Yeah sorry this languished so long. Tested and committed to HEAD.

Status: Fixed » Closed (fixed)

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

  • Commit f002cd7 on 6.x-2.x, 6.x-3.x, 7.x-1.x, master, 8.x-1.x by metzlerd: 
    #255086 Allows redirect to <front> on logout.