Fix EZP-30383: Change password with "legacy_mode: false" does not pro…

…perly clear the cache (ezsystems#139)
peterkeung · Apr 2, 2019 · fc5a970 · fc5a970
1 parent f59b0dd
commit fc5a970
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/kernel/classes/datatypes/ezuser/ezusertype.php b/kernel/classes/datatypes/ezuser/ezusertype.php
@@ -301,7 +301,7 @@ function onPublish( $contentObjectAttribute, $contentObject, $publishedNodes )
      * @param eZUser $user
      * @return string
      */
-    private function serializeDraft( eZUser $user )
+    public static function serializeDraft( eZUser $user )
     {
         return json_encode(
             array(

diff --git a/kernel/user/ezuseroperationcollection.php b/kernel/user/ezuseroperationcollection.php
@@ -317,6 +317,12 @@ static public function password( $userID, $newPassword )
             $user->setAttribute( 'password_hash', $newHash );
             $user->setAttribute( 'password_hash_type', $type );
             $user->store();
+
+            // "Draft" must be in sync with the PersistentObject
+            self::updateUserDraft( $user );
+
+            eZContentCacheManager::clearContentCacheIfNeeded( $userID );
+
             return array( 'status' => true );
         }
         else
@@ -326,6 +332,26 @@ static public function password( $userID, $newPassword )
         }
     }
 
+    /**
+     * Update the "draft" of a given user
+     *
+     * @param $user eZUser
+     */
+    static private function updateUserDraft( $user )
+    {
+        $userObject = eZContentObject::fetch( $user->id() );
+
+        foreach ( $userObject->dataMap() as $attribute )
+        {
+            if ( $attribute->ContentClassAttributeIdentifier == 'user_account' )
+            {
+                $attribute->setAttribute( 'data_text', eZUserType::serializeDraft( $user ) );
+                $attribute->store();
+                break;
+            }
+        }
+    }
+
    /**
      * Generate forgotpassword object
      *