Explorar el Código

Fix missing flash message translations

Emmanuel Vella hace 7 años
padre
commit
c39403f1b8
Se han modificado 2 ficheros con 34 adiciones y 4 borrados
  1. 16 4
      Controller/CRUDController.php
  2. 18 0
      Tests/Controller/CRUDControllerTest.php

+ 16 - 4
Controller/CRUDController.php

@@ -133,10 +133,16 @@ class CRUDController extends Controller
 
 
         try {
         try {
             $modelManager->batchDelete($this->admin->getClass(), $query);
             $modelManager->batchDelete($this->admin->getClass(), $query);
-            $this->addFlash('sonata_flash_success', 'flash_batch_delete_success');
+            $this->addFlash(
+                'sonata_flash_success',
+                $this->trans('flash_batch_delete_success', array(), 'SonataAdminBundle')
+            );
         } catch (ModelManagerException $e) {
         } catch (ModelManagerException $e) {
             $this->handleModelManagerException($e);
             $this->handleModelManagerException($e);
-            $this->addFlash('sonata_flash_error', 'flash_batch_delete_error');
+            $this->addFlash(
+                'sonata_flash_error',
+                $this->trans('flash_batch_delete_error', array(), 'SonataAdminBundle')
+            );
         }
         }
 
 
         return new RedirectResponse($this->admin->generateUrl(
         return new RedirectResponse($this->admin->generateUrl(
@@ -406,7 +412,10 @@ class CRUDController extends Controller
         $datagrid->buildPager();
         $datagrid->buildPager();
 
 
         if (true !== $nonRelevantMessage) {
         if (true !== $nonRelevantMessage) {
-            $this->addFlash('sonata_flash_info', $nonRelevantMessage);
+            $this->addFlash(
+                'sonata_flash_info',
+                $this->trans($nonRelevantMessage, array(), 'SonataAdminBundle')
+            );
 
 
             return new RedirectResponse(
             return new RedirectResponse(
                 $this->admin->generateUrl(
                 $this->admin->generateUrl(
@@ -912,7 +921,10 @@ class CRUDController extends Controller
 
 
                 if ($form->isValid()) {
                 if ($form->isValid()) {
                     $adminObjectAclManipulator->$updateMethod($adminObjectAclData);
                     $adminObjectAclManipulator->$updateMethod($adminObjectAclData);
-                    $this->addFlash('sonata_flash_success', 'flash_acl_edit_success');
+                    $this->addFlash(
+                        'sonata_flash_success',
+                        $this->trans('flash_acl_edit_success', array(), 'SonataAdminBundle')
+                    );
 
 
                     return new RedirectResponse($this->admin->generateObjectUrl('acl', $object));
                     return new RedirectResponse($this->admin->generateObjectUrl('acl', $object));
                 }
                 }

+ 18 - 0
Tests/Controller/CRUDControllerTest.php

@@ -777,6 +777,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->method('getFilterParameters')
             ->method('getFilterParameters')
             ->will($this->returnValue(array('foo' => 'bar')));
             ->will($this->returnValue(array('foo' => 'bar')));
 
 
+        $this->expectTranslate('flash_batch_delete_success', array(), 'SonataAdminBundle');
+
         $result = $this->controller->batchActionDelete($this->createMock('Sonata\AdminBundle\Datagrid\ProxyQueryInterface'));
         $result = $this->controller->batchActionDelete($this->createMock('Sonata\AdminBundle\Datagrid\ProxyQueryInterface'));
 
 
         $this->assertInstanceOf('Symfony\Component\HttpFoundation\RedirectResponse', $result);
         $this->assertInstanceOf('Symfony\Component\HttpFoundation\RedirectResponse', $result);
@@ -797,6 +799,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->method('getFilterParameters')
             ->method('getFilterParameters')
             ->will($this->returnValue(array('foo' => 'bar')));
             ->will($this->returnValue(array('foo' => 'bar')));
 
 
+        $this->expectTranslate('flash_batch_delete_error', array(), 'SonataAdminBundle');
+
         $result = $this->controller->batchActionDelete($this->createMock('Sonata\AdminBundle\Datagrid\ProxyQueryInterface'));
         $result = $this->controller->batchActionDelete($this->createMock('Sonata\AdminBundle\Datagrid\ProxyQueryInterface'));
 
 
         $this->assertInstanceOf('Symfony\Component\HttpFoundation\RedirectResponse', $result);
         $this->assertInstanceOf('Symfony\Component\HttpFoundation\RedirectResponse', $result);
@@ -2885,6 +2889,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->method('getSecurityHandler')
             ->method('getSecurityHandler')
             ->will($this->returnValue($aclSecurityHandler));
             ->will($this->returnValue($aclSecurityHandler));
 
 
+        $this->expectTranslate('flash_acl_edit_success', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
 
 
         $response = $this->controller->aclAction(null, $this->request);
         $response = $this->controller->aclAction(null, $this->request);
@@ -3394,6 +3400,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->with($this->equalTo('Foo'), $this->equalTo($query), $this->equalTo(array('123', '456')))
             ->with($this->equalTo('Foo'), $this->equalTo($query), $this->equalTo(array('123', '456')))
             ->will($this->returnValue(true));
             ->will($this->returnValue(true));
 
 
+        $this->expectTranslate('flash_batch_delete_success', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
         $this->request->request->set('data', json_encode(array('action' => 'delete', 'idx' => array('123', '456'), 'all_elements' => false)));
         $this->request->request->set('data', json_encode(array('action' => 'delete', 'idx' => array('123', '456'), 'all_elements' => false)));
         $this->request->request->set('_sonata_csrf_token', 'csrf-token-123_sonata.batch');
         $this->request->request->set('_sonata_csrf_token', 'csrf-token-123_sonata.batch');
@@ -3449,6 +3457,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->with($this->equalTo('Foo'), $this->equalTo($query), $this->equalTo(array('123', '456')))
             ->with($this->equalTo('Foo'), $this->equalTo($query), $this->equalTo(array('123', '456')))
             ->will($this->returnValue(true));
             ->will($this->returnValue(true));
 
 
+        $this->expectTranslate('flash_batch_delete_success', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
         $this->request->request->set('action', 'delete');
         $this->request->request->set('action', 'delete');
         $this->request->request->set('idx', array('123', '456'));
         $this->request->request->set('idx', array('123', '456'));
@@ -3537,6 +3547,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->method('getDatagrid')
             ->method('getDatagrid')
             ->will($this->returnValue($datagrid));
             ->will($this->returnValue($datagrid));
 
 
+        $this->expectTranslate('flash_batch_empty', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
         $this->request->request->set('action', 'foo');
         $this->request->request->set('action', 'foo');
         $this->request->request->set('idx', array('789'));
         $this->request->request->set('idx', array('789'));
@@ -3571,6 +3583,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->method('getDatagrid')
             ->method('getDatagrid')
             ->will($this->returnValue($datagrid));
             ->will($this->returnValue($datagrid));
 
 
+        $this->expectTranslate('flash_foo_error', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
         $this->request->request->set('action', 'foo');
         $this->request->request->set('action', 'foo');
         $this->request->request->set('idx', array('999'));
         $this->request->request->set('idx', array('999'));
@@ -3602,6 +3616,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->method('getDatagrid')
             ->method('getDatagrid')
             ->will($this->returnValue($datagrid));
             ->will($this->returnValue($datagrid));
 
 
+        $this->expectTranslate('flash_batch_empty', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
         $this->request->request->set('action', 'delete');
         $this->request->request->set('action', 'delete');
         $this->request->request->set('idx', array());
         $this->request->request->set('idx', array());
@@ -3706,6 +3722,8 @@ class CRUDControllerTest extends PHPUnit_Framework_TestCase
             ->with($this->equalTo('Foo'), $this->equalTo($query), $this->equalTo(array('123', '456')))
             ->with($this->equalTo('Foo'), $this->equalTo($query), $this->equalTo(array('123', '456')))
             ->will($this->returnValue(true));
             ->will($this->returnValue(true));
 
 
+        $this->expectTranslate('flash_batch_delete_success', array(), 'SonataAdminBundle');
+
         $this->request->setMethod('POST');
         $this->request->setMethod('POST');
         $this->request->request->set('data', json_encode(array('action' => 'delete', 'idx' => array('123', '456'), 'all_elements' => false)));
         $this->request->request->set('data', json_encode(array('action' => 'delete', 'idx' => array('123', '456'), 'all_elements' => false)));
         $this->request->request->set('foo', 'bar');
         $this->request->request->set('foo', 'bar');