Cancel checkid_setup failing

sanduhrs - September 22, 2008 - 11:02
Project:OpenID Provider
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The Cancel button in the openid_provider_form doesn't have any effect.

Pressing the button calls openid_redirect($return_to, $response) , where $return_to is empty, because it get's filled with $form_state['values']['openid.return_to'] which always is empty, as the desired value is stored in $form_state['storage']['response']['openid.return_to'] .

Patch attached, please review.

AttachmentSize
openid_prvider_cancel_checkid_setup.patch748 bytes

#1

darren.ferguson - October 9, 2008 - 14:12
Status:needs review» reviewed & tested by the community

Patch is good, it takes you back and provides the cancel status for the request.

Have tested based on the code from contributions.

#2

walkah - February 21, 2009 - 03:39
Status:reviewed & tested by the community» fixed

fixed, thanks sanduhrs!

#3

System Message - March 7, 2009 - 03:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.