Logic Hook after_save fails due to Email Addresses save process

[eggsurplus]

I have a huge beef with this one. Why do email addresses get save AFTER the contact is saved and all the logic hooks are processed? This is causing huge data integrity issues here. If a logic hook needs to do some work (which can't be avoided) it will cause the Email Addresses to never save thus wiping it out. The use case here is a logic hook that checks for dups in an external system and displays a form that calls Save again. (I've avoided other scenarios by NEVER calling bean->save() but instead going directly into the db)

[eggsurplus]

Fixed it. In SugarEmailAddress.php getEmailAddressWidgetDuplicatesView() it was checking the $_POST var instead of $_REQUEST. The duplicate url is a get request.

This would mean that the built-in Sugar Duplicate check would fail as well. The fix is to change all _POST references to _REQUEST