#4552: [with patch, needs review] notebook -- when email system on, registration
is broken
----------------------------+-----------------------------------------------
 Reporter:  TimothyClemans  |       Owner:  boothby   
     Type:  defect          |      Status:  new       
 Priority:  major           |   Milestone:  sage-4.1.2
Component:  notebook        |    Keywords:            
 Reviewer:                  |      Author:            
   Merged:                  |  
----------------------------+-----------------------------------------------

Comment(by wjp):

 I modified the attached patch to remove the dependency on #4551. (Only
 apply {{{trac_4552-notebook_account_email.patch}}})

 To elaborate on the problem a little bit: the problem was that the value
 of the 'email' input box was never read from the form. As a result, when
 the verification code tries to access it, the server gets a KeyError.

 For what it's worth, I give a positive review to adding 'email' to
 'input_boxes', which is how Timothy's patch fixes the problem.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4552#comment:4>
Sage <http://sagemath.org/>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to