Completed
Pull Request — master (#162)
by
unknown
03:27
created
Event/SettingChangeEvent.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
     /**
27 27
      * Constructor.
28 28
      *
29
-     * @param Action $action
29
+     * @param string $action
30 30
      */
31 31
     public function __construct($action)
32 32
     {
Please login to merge, or discard this patch.
Settings/General/SettingsContainer.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -120,7 +120,7 @@
 block discarded – undo
120 120
     }
121 121
 
122 122
     /**
123
-     * @param array $profiles
123
+     * @param string[] $profiles
124 124
      */
125 125
     public function setProfiles($profiles)
126 126
     {
Please login to merge, or discard this patch.
Settings/General/SettingsManager.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,6 @@
 block discarded – undo
17 17
 use ONGR\ElasticsearchBundle\ORM\Manager;
18 18
 use ONGR\SettingsBundle\Document\Setting;
19 19
 use ONGR\SettingsBundle\Event\SettingChangeEvent;
20
-use Exception;
21 20
 
22 21
 /**
23 22
  * Class SettingsManager responsible for managing settings actions.
Please login to merge, or discard this patch.
Tests/Unit/Settings/General/SettingsContainerTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@
 block discarded – undo
108 108
      * @param bool  $miss
109 109
      * @param array $settings
110 110
      *
111
-     * @return \PHPUnit_Framework_MockObject_MockObject|PoolInterface
111
+     * @return PoolInterface
112 112
      */
113 113
     private function getPool($miss = true, $settings = [])
114 114
     {
Please login to merge, or discard this patch.
Twig/HiddenExtension.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
     }
42 42
 
43 43
     /**
44
-     * @return array
44
+     * @return \Twig_SimpleFunction[]
45 45
      */
46 46
     public function getFunctions()
47 47
     {
Please login to merge, or discard this patch.
Twig/WrapperExtension.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
     /**
20 20
      * Returns list of new Twig functions.
21 21
      *
22
-     * @return array
22
+     * @return \Twig_SimpleFilter[]
23 23
      */
24 24
     public function getFilters()
25 25
     {
Please login to merge, or discard this patch.
EventListener/SettingsCookieListener.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
     ) {
39 39
         $settingsMap = $this->generalSettingsManager->getSettingsMap();
40 40
         $cookiesServiceNames = array_map(
41
-            function ($setting) {
41
+            function($setting) {
42 42
                 return $setting['cookie'];
43 43
             },
44 44
             $settingsMap
Please login to merge, or discard this patch.
Tests/Functional/Controller/UserControllerTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
         if ($shouldSucceed) {
80 80
             $this->assertContains(' already logged in', $response->getContent());
81 81
 
82
-             // Assert correct cookie has been set.
82
+                // Assert correct cookie has been set.
83 83
             /** @var Cookie $cookie */
84 84
             $cookie = $client->getCookieJar()->get('ongr_settings_user_auth');
85 85
             $this->assertSame('ongr_settings_user_auth', $cookie->getName());
Please login to merge, or discard this patch.
Tests/Functional/Controller/PersonalSettingsControllerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@
 block discarded – undo
99 99
 
100 100
         // Try to change value through change setting action.
101 101
         $data[0] = ['foo_setting_1', false];
102
-        $data[1] = ['foo_setting_non_existent', false ];
102
+        $data[1] = ['foo_setting_non_existent', false];
103 103
 
104 104
         foreach ($data as $case) {
105 105
             $client->request('get', '/admin/setting/change/' . base64_encode($case[0]));
Please login to merge, or discard this patch.