[Gemstracker-svn] SF.net SVN: gemstracker:[459] branches/1.5.x/library/classes/Gems/User/ DbUserDefinitionAbstract.php

[<gem...@li...>]

Revision: 459
          http://gemstracker.svn.sourceforge.net/gemstracker/?rev=459&view=rev
Author:   mennodekker
Date:     2012-02-07 15:38:49 +0000 (Tue, 07 Feb 2012)
Log Message:
-----------
Fixed #55: Do not allow reset email when person has no email

Modified Paths:
--------------
    branches/1.5.x/library/classes/Gems/User/DbUserDefinitionAbstract.php

Modified: branches/1.5.x/library/classes/Gems/User/DbUserDefinitionAbstract.php
===================================================================
--- branches/1.5.x/library/classes/Gems/User/DbUserDefinitionAbstract.php	2012-02-07 15:17:46 UTC (rev 458)
+++ branches/1.5.x/library/classes/Gems/User/DbUserDefinitionAbstract.php	2012-02-07 15:38:49 UTC (rev 459)
@@ -71,7 +71,14 @@
     {
         if ($user) {
             // Depends on the user.
-            return $user->hasEmailAddress() && $user->canSetPassword();
+            if ($user->hasEmailAddress() && $user->canSetPassword()) {
+                $email = $user->getEmailAddress();
+                if (empty($email)) {
+                    return false;
+                } else {
+                    return true;
+                }
+            }
         } else {
             return true;
         }

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.