Merge pull request #131 from samuele-mrapps/patch-1

fix issue #91 and reduce flush
Kunstmaan · Jan 27, 2015 · 4b02802 · 4b02802
2 parents 1439220 + a265fe0
commit 4b02802
Showing 1 changed file with 7 additions and 13 deletions.
diff --git a/src/Kunstmaan/UserManagementBundle/Controller/UsersController.php b/src/Kunstmaan/UserManagementBundle/Controller/UsersController.php
@@ -106,12 +106,9 @@ public function addAction(Request $request)
         if ($request->isMethod('POST')) {
             $form->handleRequest($request);
             if ($form->isValid()) {
-                $em->persist($user);
-                $em->flush();
-
-                /* @var UserManipulator $manipulator */
-                $manipulator = $this->get('fos_user.util.user_manipulator');
-                $manipulator->changePassword($user->getUsername(), $user->getPlainpassword());
+                /* @var UserManager $userManager */
+                $userManager = $this->container->get('fos_user.user_manager');
+                $userManager->updateUser($user, true);
 
                 $this->get('session')->getFlashBag()->add(
                     'success',
@@ -167,13 +164,10 @@ public function editAction(Request $request, $id)
         if ($request->isMethod('POST')) {
             $form->handleRequest($request);
             if ($form->isValid()) {
-                if ($user->getPlainpassword() != "") {
-                    $manipulator = $this->get('fos_user.util.user_manipulator');
-                    $manipulator->changePassword($user->getUsername(), $user->getPlainpassword());
-                }
-                $user->setPlainpassword("");
-                $em->persist($user);
-                $em->flush();
+                /* @var UserManager $userManager */
+                $userManager = $this->container->get('fos_user.user_manager');
+                $userManager->updateUser($user, true);
+
                 $this->get('session')->getFlashBag()->add(
                     'success',
                     'User \'' . $user->getUsername() . '\' has been edited!'