8 Mar
2015
8 Mar
'15
11:02 p.m.
On Mar 03, 2015, at 02:15 PM, Ankush Sharma wrote:
- Later on, in the same function the following code checks if the email is matched:
*for address in user.addresses:*
if address.email == email:*
break*
Here *email* is in *preserved case* while *address.email *returns a lower case version of email and an *unmatch* occurs. ( sam@gmail.com and SAM@gmail.com do not matches ofcourse )
I think you've done a great job of finding the root cause of the bug. I'll comment further on the merge proposal, but I think your fix is essentially correct.
Thanks!
Cheers, -Barry