Completed
Pull Request — develop (#359)
by Mathias
23:51
created
src/Core/Repository/DoctrineMongoODM/Event/RepositoryEventsSubscriber.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@
 block discarded – undo
91 91
      */
92 92
     protected function injectAttachableEntityManager(AttachableEntityInterface $entity)
93 93
     {
94
-        if (! isset($this->attachableEntityManagerPrototype)) {
94
+        if (!isset($this->attachableEntityManagerPrototype)) {
95 95
             $this->attachableEntityManagerPrototype = new AttachableEntityManager($this->services->get('repositories'));
96 96
         }
97 97
         
Please login to merge, or discard this patch.
module/Core/src/Core/Entity/AttachableEntityManager.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     {
54 54
         $className = get_class($entity);
55 55
         
56
-        if (! isset($key)) {
56
+        if (!isset($key)) {
57 57
             $key = $className;
58 58
         }
59 59
         
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
         $entityId = $entity->getId();
64 64
         
65 65
         // check if entity is not persisted
66
-        if (! $entityId) {
66
+        if (!$entityId) {
67 67
             // persist entity & retrieve its ID
68 68
             $this->repositories->getRepository($className)->store($entity);
69 69
             $entityId = $entity->getId();
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
      */
82 82
     public function getAttachedEntity($key)
83 83
     {
84
-        if (! isset($this->references[$key])) {
84
+        if (!isset($this->references[$key])) {
85 85
             return;
86 86
         }
87 87
         
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
         $entity = $this->repositories->getRepository($reference['repository'])
90 90
             ->find($reference['id']);
91 91
         
92
-        if (! $entity) {
92
+        if (!$entity) {
93 93
             // remove reference if entity does not exists
94 94
             unset($this->references[$key]);
95 95
         }
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/Controller/ApiJobListByChannelController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -30,12 +30,12 @@
 block discarded – undo
30 30
     public function listAction()
31 31
     {
32 32
 
33
-        $channel = $this->params()->fromRoute('channel','default');
33
+        $channel = $this->params()->fromRoute('channel', 'default');
34 34
 
35 35
         /* @var Paginator $paginator */
36
-        $paginator = $this->paginator('Jobs/Job', ['channel' => $channel ]);
36
+        $paginator = $this->paginator('Jobs/Job', ['channel' => $channel]);
37 37
 
38
-        $viewModel=new ViewModel();
38
+        $viewModel = new ViewModel();
39 39
         $viewModel->setVariables(
40 40
             [
41 41
                 'jobs' => $paginator,
Please login to merge, or discard this patch.
module/Core/src/Core/Listener/XmlRenderListener.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -73,8 +73,8 @@  discard block
 block discarded – undo
73 73
     
74 74
     public function injectXmlTemplate(MvcEvent $e)
75 75
     {
76
-        $format = $e->getRouteMatch()->getParam('format',"html");
77
-        $channel = $e->getRouteMatch()->getParam('channel',"default");
76
+        $format = $e->getRouteMatch()->getParam('format', "html");
77
+        $channel = $e->getRouteMatch()->getParam('channel', "default");
78 78
 
79 79
         if ('xml' == $format) {
80 80
             $viewModel = $e->getResult();
@@ -83,8 +83,8 @@  discard block
 block discarded – undo
83 83
             }
84 84
             $resolver = $e->getApplication()->getServiceManager()->get('ViewResolver');
85 85
 
86
-            $templateDefault = $viewModel->getTemplate() . '.xml.phtml';
87
-            $templateChannel = $viewModel->getTemplate() . '.' . $channel . '.xml.phtml';
86
+            $templateDefault = $viewModel->getTemplate().'.xml.phtml';
87
+            $templateChannel = $viewModel->getTemplate().'.'.$channel.'.xml.phtml';
88 88
 
89 89
 
90 90
             if ($channel != 'default' && $resolver->resolve($templateChannel)) {
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 
102 102
             /* @var Response $response */
103 103
             $response = $e->getResponse();
104
-            $response->getHeaders()->addHeaderLine('Content-Type','application/xml');
104
+            $response->getHeaders()->addHeaderLine('Content-Type', 'application/xml');
105 105
         }
106 106
                 
107 107
     }
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/View/Helper/ApplyUrl.php 3 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -123,10 +123,10 @@
 block discarded – undo
123 123
      * @param $options
124 124
      */
125 125
     public function setOptions($options){
126
-       foreach($options as $key=>$val) {
126
+        foreach($options as $key=>$val) {
127 127
             if (array_key_exists($this->options,$key)) {
128 128
                 $this->options[$key]=$val;
129 129
             }
130
-       }
130
+        }
131 131
     }
132 132
 }
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 
78 78
     public function __invoke(Job $jobEntity, $options = [])
79 79
     {
80
-        $options= array_merge($this->options, $options);
80
+        $options = array_merge($this->options, $options);
81 81
 
82 82
         $ats = $jobEntity->getAtsMode();
83 83
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 
92 92
         if ($ats->isIntern() || $ats->isEmail()) {
93 93
 
94
-            $query = [ 'subscriberUri' => $serverUrlHelper(array()) . '/subscriber/' . 1 ];
94
+            $query = ['subscriberUri' => $serverUrlHelper(array()).'/subscriber/'.1];
95 95
             $route = 'lang/apply';
96 96
             $params = [
97 97
                 'applyId' => $jobEntity->getApplyId(),
@@ -106,12 +106,12 @@  discard block
 block discarded – undo
106 106
             $url = $ats->getUri();
107 107
         }
108 108
 
109
-        if($options['linkOnly']){
110
-            $result=$url;
111
-            if($options['absolute']) {
109
+        if ($options['linkOnly']) {
110
+            $result = $url;
111
+            if ($options['absolute']) {
112 112
                 $result = $serverUrlHelper($url);
113 113
             }
114
-        }else{
114
+        } else {
115 115
             $translate = $this->translateHelper;
116 116
             $result = sprintf('<a href="%s" rel="nofollow">%s</a>', $url, $translate('Apply'));
117 117
         }
@@ -122,10 +122,10 @@  discard block
 block discarded – undo
122 122
     /**
123 123
      * @param $options
124 124
      */
125
-    public function setOptions($options){
126
-       foreach($options as $key=>$val) {
127
-            if (array_key_exists($this->options,$key)) {
128
-                $this->options[$key]=$val;
125
+    public function setOptions($options) {
126
+       foreach ($options as $key=>$val) {
127
+            if (array_key_exists($this->options, $key)) {
128
+                $this->options[$key] = $val;
129 129
             }
130 130
        }
131 131
     }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@
 block discarded – undo
111 111
             if($options['absolute']) {
112 112
                 $result = $serverUrlHelper($url);
113 113
             }
114
-        }else{
114
+        } else{
115 115
             $translate = $this->translateHelper;
116 116
             $result = sprintf('<a href="%s" rel="nofollow">%s</a>', $url, $translate('Apply'));
117 117
         }
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/View/Helper/JobUrl.php 3 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -102,10 +102,10 @@
 block discarded – undo
102 102
             }
103 103
         }else{
104 104
             $result = sprintf('<a href="%s" rel="%s" %s>%s</a>',
105
-                              $url,
106
-                              $options['rel'],
107
-                              $options['target']?"target=" . $options['target']:"",
108
-                              strip_tags($jobEntity->getTitle()));
105
+                                $url,
106
+                                $options['rel'],
107
+                                $options['target']?"target=" . $options['target']:"",
108
+                                strip_tags($jobEntity->getTitle()));
109 109
         }
110 110
 
111 111
         return $result;
Please login to merge, or discard this patch.
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
         if (!empty($jobEntity->getLink())) {
70 70
             $url = $jobEntity->getLink();
71 71
             $isExternalLink = true;
72
-        }elseif($options['showPendingJobs']) {
72
+        } elseif($options['showPendingJobs']) {
73 73
             $url = $urlHelper(
74 74
                 'lang/jobs/approval',
75 75
                 [],
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
                     ]
80 80
                 ], true);
81 81
 
82
-        }else{
82
+        } else{
83 83
 
84 84
             $query = [
85 85
                 'subscriberUri' => $serverUrlHelper([]) . '/subscriber/' . 1,
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
             if ($options['absolute'] && !$isExternalLink){
101 101
                 $result = $serverUrlHelper($url);
102 102
             }
103
-        }else{
103
+        } else{
104 104
             $result = sprintf('<a href="%s" rel="%s" %s>%s</a>',
105 105
                               $url,
106 106
                               $options['rel'],
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
     public function __invoke(Job $jobEntity, $options = [], $urlParams = [])
68 68
     {
69 69
 
70
-        $options= array_merge($this->options, $options);
70
+        $options = array_merge($this->options, $options);
71 71
         $paramsHelper = $this->paramsHelper;
72 72
         $urlHelper = $this->urlHelper;
73 73
         $serverUrlHelper = $this->serverUrlHelper;
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         if (!empty($jobEntity->getLink())) {
77 77
             $url = $jobEntity->getLink();
78 78
             $isExternalLink = true;
79
-        }elseif($options['showPendingJobs']) {
79
+        }elseif ($options['showPendingJobs']) {
80 80
             $url = $urlHelper(
81 81
                 'lang/jobs/approval',
82 82
                 [],
@@ -86,10 +86,10 @@  discard block
 block discarded – undo
86 86
                     ]
87 87
                 ], true);
88 88
 
89
-        }else{
89
+        } else {
90 90
 
91 91
             $query = [
92
-                'subscriberUri' => $serverUrlHelper([]) . '/subscriber/' . 1,
92
+                'subscriberUri' => $serverUrlHelper([]).'/subscriber/'.1,
93 93
                 'id' => $jobEntity->getId()
94 94
             ];
95 95
             $route = 'lang/jobs/view';
@@ -103,16 +103,16 @@  discard block
 block discarded – undo
103 103
             $url = $urlHelper($route, $params, array('query' => $query));
104 104
         }
105 105
 
106
-        if ($options['linkOnly']){
106
+        if ($options['linkOnly']) {
107 107
             $result = $url;
108
-            if ($options['absolute'] && !$isExternalLink){
108
+            if ($options['absolute'] && !$isExternalLink) {
109 109
                 $result = $serverUrlHelper($url);
110 110
             }
111
-        }else{
111
+        } else {
112 112
             $result = sprintf('<a href="%s" rel="%s" %s>%s</a>',
113 113
                               $url,
114 114
                               $options['rel'],
115
-                              $options['target']?"target=" . $options['target']:"",
115
+                              $options['target'] ? "target=".$options['target'] : "",
116 116
                               strip_tags($jobEntity->getTitle()));
117 117
         }
118 118
 
@@ -122,10 +122,10 @@  discard block
 block discarded – undo
122 122
     /**
123 123
      * @param $options
124 124
      */
125
-    public function setOptions($options){
126
-        foreach($options as $key=>$val) {
127
-            if (array_key_exists($this->options,$key)) {
128
-                $this->options[$key]=$val;
125
+    public function setOptions($options) {
126
+        foreach ($options as $key=>$val) {
127
+            if (array_key_exists($this->options, $key)) {
128
+                $this->options[$key] = $val;
129 129
             }
130 130
         }
131 131
     }
Please login to merge, or discard this patch.
module/Core/src/Core/Options/MailServiceOptions.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
         return $this;
70 70
     }
71 71
 
72
-    public function getUsername(){
72
+    public function getUsername() {
73 73
         return $this->connectionConfig['username'];
74 74
     }
75 75
 
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         return $this;
81 81
     }
82 82
 
83
-    public function getPassword(){
83
+    public function getPassword() {
84 84
         return $this->connectionConfig['password'];
85 85
     }
86 86
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
         return $this;
92 92
     }
93 93
 
94
-    public function getSsl(){
94
+    public function getSsl() {
95 95
         return $this->connectionConfig['ssl'];
96 96
     }
97 97
 }
98 98
\ No newline at end of file
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/Filter/ViewModelTemplateFilterJob.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
             'lang/jobs/view',
49 49
             [],
50 50
             [
51
-                'query' => [ 'id' => $job->getId() ],
51
+                'query' => ['id' => $job->getId()],
52 52
                 'force_canonical' => true
53 53
             ]
54 54
         );
Please login to merge, or discard this patch.
module/Core/src/Core/Form/Hydrator/Strategy/TreeSelectStrategy.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@
 block discarded – undo
97 97
     /**
98 98
      * Set the allow multiple selections flag.
99 99
      *
100
-     * @param Callable|bool $flagOrCallback When a Callable is passed, it must return bool.
100
+     * @param \Closure $flagOrCallback When a Callable is passed, it must return bool.
101 101
      *
102 102
      * @return self
103 103
      */
Please login to merge, or discard this patch.