- **status**: code-review --> in-progress
- **Comment**:

You still can delete last verified email if it is not primary.

- add two email addresses and verify both
- set the first one as primary
- delete first one (works, since you have second verified address)
- delete second one (works, but it should not, since it is last verified 
address)

Hm, checking code one more time I see that a new primary should be set when you 
delete the old one, but it does not work for me somehow...

Logs:

~~~~
08:12:25,863 WARNI [forgesf.sf_providers] "None" was given for a preferred 
email address for admin1, selecting [email protected] instead
~~~~

but `[email protected]` is the address I've deleted. Seems like 
`select_new_primary_addr`in sf provider returns deleted address somehow.

It duplicates only when your primary address is the first address in the 
addresses list. Maybe add a test for this case along with the fix?



---

** [tickets:#7523] Better to go to /auth/preferences after email addr 
verification**

**Status:** in-progress
**Milestone:** forge-jul-25
**Labels:** bitesize 
**Created:** Wed Jul 02, 2014 06:18 PM UTC by Dave Brondsema
**Last Updated:** Thu Jul 17, 2014 07:54 AM UTC
**Owner:** Dave Brondsema

The `verify_addr` method redirects to "/"; it'd be more useful to go to 
"/auth/preferences"


---

Sent from sourceforge.net because [email protected] is subscribed to 
https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a 
mailing list, you can unsubscribe from the mailing list.

Reply via email to