Completed
Push — develop ( 037ef6...a650ad )
by Carsten
06:58
created
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.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
     public function __invoke(Job $jobEntity, $options = [])
61 61
     {
62 62
 
63
-        $options= array_merge($this->options, $options);
63
+        $options = array_merge($this->options, $options);
64 64
         $paramsHelper = $this->paramsHelper;
65 65
         $urlHelper = $this->urlHelper;
66 66
         $serverUrlHelper = $this->serverUrlHelper;
@@ -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,10 +79,10 @@  discard block
 block discarded – undo
79 79
                     ]
80 80
                 ], true);
81 81
 
82
-        }else{
82
+        } else {
83 83
 
84 84
             $query = [
85
-                'subscriberUri' => $serverUrlHelper([]) . '/subscriber/' . 1,
85
+                'subscriberUri' => $serverUrlHelper([]).'/subscriber/'.1,
86 86
                 'id' => $jobEntity->getId()
87 87
             ];
88 88
             $route = 'lang/jobs/view';
@@ -95,16 +95,16 @@  discard block
 block discarded – undo
95 95
             $url = $urlHelper($route, $params, array('query' => $query));
96 96
         }
97 97
 
98
-        if ($options['linkOnly']){
98
+        if ($options['linkOnly']) {
99 99
             $result = $url;
100
-            if ($options['absolute'] && !$isExternalLink){
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'],
107
-                              $options['target']?"target=" . $options['target']:"",
107
+                              $options['target'] ? "target=".$options['target'] : "",
108 108
                               strip_tags($jobEntity->getTitle()));
109 109
         }
110 110
 
@@ -114,10 +114,10 @@  discard block
 block discarded – undo
114 114
     /**
115 115
      * @param $options
116 116
      */
117
-    public function setOptions($options){
118
-        foreach($options as $key=>$val) {
119
-            if (array_key_exists($this->options,$key)) {
120
-                $this->options[$key]=$val;
117
+    public function setOptions($options) {
118
+        foreach ($options as $key=>$val) {
119
+            if (array_key_exists($this->options, $key)) {
120
+                $this->options[$key] = $val;
121 121
             }
122 122
         }
123 123
     }
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.
module/Jobs/src/Jobs/Controller/TemplateController.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
     public function viewAction()
57 57
     {
58 58
         $id = $this->params()->fromQuery('id');
59
-        $channel = $this->params()->fromRoute('channel','default');
59
+        $channel = $this->params()->fromRoute('channel', 'default');
60 60
         $response = $this->getResponse();
61 61
         /* @var \Jobs\Entity\Job $job */
62 62
         $job = $this->jobRepository->find($id);
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
             $this->auth()->isAdmin()
84 84
         ) {
85 85
             $applicationViewModel->setTemplate('iframe/iFrameInjection');
86
-        }elseif(Status::EXPIRED == $job->getStatus() or  Status::INACTIVE == $job->getStatus()) {
86
+        }elseif (Status::EXPIRED == $job->getStatus() or Status::INACTIVE == $job->getStatus()) {
87 87
             $response->setStatusCode(Response::STATUS_CODE_410);
88 88
             $model->setTemplate('jobs/error/expired');
89 89
             $model->setVariables(
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
     protected function editTemplateAction()
112 112
     {
113 113
         $id = $this->params('id');
114
-        $formIdentifier=$this->params()->fromQuery('form');
114
+        $formIdentifier = $this->params()->fromQuery('form');
115 115
         $job = $this->jobRepository->find($id);
116 116
         $this->acl($job, 'edit');
117 117
 
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 
143 143
             $instanceForm = $formTemplate->get($formIdentifier);
144 144
             if (!isset($instanceForm)) {
145
-                throw new \RuntimeException('No form found for "' . $formIdentifier . '"');
145
+                throw new \RuntimeException('No form found for "'.$formIdentifier.'"');
146 146
             }
147 147
 
148 148
             // the id is part of the postData, but it never should be altered
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@
 block discarded – undo
83 83
             $this->auth()->isAdmin()
84 84
         ) {
85 85
             $applicationViewModel->setTemplate('iframe/iFrameInjection');
86
-        }elseif(Status::EXPIRED == $job->getStatus() or  Status::INACTIVE == $job->getStatus()) {
86
+        } elseif(Status::EXPIRED == $job->getStatus() or  Status::INACTIVE == $job->getStatus()) {
87 87
             $response->setStatusCode(Response::STATUS_CODE_410);
88 88
             $model->setTemplate('jobs/error/expired');
89 89
             $model->setVariables(
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 2 patches
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.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@
 block discarded – undo
123 123
     public function extract($value)
124 124
     {
125 125
         if (!$value instanceOf AbstractLeafs) {
126
-            throw new \InvalidArgumentException('$value must be an instance of ' . AbstractLeafs::class);
126
+            throw new \InvalidArgumentException('$value must be an instance of '.AbstractLeafs::class);
127 127
         }
128 128
         /* @var AbstractLeafs $value
129 129
          * @var NodeInterface $item */
Please login to merge, or discard this patch.
module/Core/src/Core/Form/Hydrator/TreeHydrator.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@
 block discarded – undo
93 93
      * @param  array  $data Form values
94 94
      * @param  Collection $object
95 95
      *
96
-     * @return object
96
+     * @return NodeInterface
97 97
      */
98 98
     public function hydrate(array $data, $object)
99 99
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
         $data = [];
54 54
         $this->flattenTree($object, $data);
55 55
 
56
-        return ['items' => $data ];
56
+        return ['items' => $data];
57 57
 
58 58
     }
59 59
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 
80 80
         if ($tree->hasChildren()) {
81 81
             foreach ($tree->getChildren() as $i => $child) {
82
-                $this->flattenTree($child, $data, $curId . '-' . ($i + 1) );
82
+                $this->flattenTree($child, $data, $curId.'-'.($i + 1));
83 83
             }
84 84
         }
85 85
     }
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
          * unflatten tree
117 117
          */
118 118
         $items = $data['items'];
119
-        $tree = [ '__root__' => array_shift($items) ];
119
+        $tree = ['__root__' => array_shift($items)];
120 120
 
121 121
         foreach ($items as $item) {
122 122
             $parent = substr($item['current'], 0, strrpos($item['current'], '-'));
Please login to merge, or discard this patch.
module/Core/src/Core/Repository/RepositoryService.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
             $name = "\\$namespace\\Entity\\$entityName";
39 39
         }
40 40
         
41
-        $repository  = $this->dm->getRepository($name);
41
+        $repository = $this->dm->getRepository($name);
42 42
         return $repository;
43 43
     }
44 44
     
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -24,6 +24,9 @@
 block discarded – undo
24 24
         $this->dm = $dm;
25 25
     }
26 26
     
27
+    /**
28
+     * @param string $name
29
+     */
27 30
     public function get($name)
28 31
     {
29 32
         if (!class_exists($name)) {
Please login to merge, or discard this patch.
module/Core/src/Core/Form/View/Helper/FormWizardContainer.php 2 patches
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -90,24 +90,24 @@
 block discarded – undo
90 90
             $tabId = $containerId . '-' . strtolower($tabElement->getName());
91 91
             $tabsNav .= '<li><a data-toggle="tab" href="#' . $tabId . '">' . $translate($tabElement->getLabel()) . '</a></li>';
92 92
             $tabsContent .= '<div class="tab-pane" id="' . $tabId . '">'
93
-                          . $formContainer($tabElement, $layout, $parameter)
94
-                          . '</div>';
93
+                            . $formContainer($tabElement, $layout, $parameter)
94
+                            . '</div>';
95 95
         }
96 96
 
97 97
         $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="' . $containerId . '">'
98
-                  . '<ul>' . $tabsNav . '</ul>'
99
-                  . '<div class="tab-content">' . $tabsContent . '</div>';
98
+                    . '<ul>' . $tabsNav . '</ul>'
99
+                    . '<div class="tab-content">' . $tabsContent . '</div>';
100 100
         if ($containerParams['pager']) {
101 101
             $content .='<ul class="pager wizard">'
102
-                  . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
103
-                  . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
104
-                  . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
105
-                  . (false !== $containerParams['finish_label']
102
+                    . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
103
+                    . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
104
+                    . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
105
+                    . (false !== $containerParams['finish_label']
106 106
                      ? '<a class="pull-right" href="' . $containerParams['finish_href'] . '">'
107
-                       . $translate($containerParams['finish_label']) . ' &bull;</a>'
107
+                        . $translate($containerParams['finish_label']) . ' &bull;</a>'
108 108
                      : ''
109 109
                     )
110
-                  . '</li></ul>';
110
+                    . '</li></ul>';
111 111
         }
112 112
         $content .= '</div>';
113 113
 
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -83,28 +83,28 @@
 block discarded – undo
83 83
 
84 84
         $containerId = $container->getAttribute('id');
85 85
         if (!$containerId) {
86
-            $containerId = 'wizardcontainer-' . strtolower(str_replace('\\', '-', get_class($container)));
86
+            $containerId = 'wizardcontainer-'.strtolower(str_replace('\\', '-', get_class($container)));
87 87
         }
88 88
 
89 89
         foreach ($container as $tabElement) {
90
-            $tabId = $containerId . '-' . strtolower($tabElement->getName());
91
-            $tabsNav .= '<li><a data-toggle="tab" href="#' . $tabId . '">' . $translate($tabElement->getLabel()) . '</a></li>';
92
-            $tabsContent .= '<div class="tab-pane" id="' . $tabId . '">'
90
+            $tabId = $containerId.'-'.strtolower($tabElement->getName());
91
+            $tabsNav .= '<li><a data-toggle="tab" href="#'.$tabId.'">'.$translate($tabElement->getLabel()).'</a></li>';
92
+            $tabsContent .= '<div class="tab-pane" id="'.$tabId.'">'
93 93
                           . $formContainer($tabElement, $layout, $parameter)
94 94
                           . '</div>';
95 95
         }
96 96
 
97
-        $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="' . $containerId . '">'
98
-                  . '<ul>' . $tabsNav . '</ul>'
99
-                  . '<div class="tab-content">' . $tabsContent . '</div>';
97
+        $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="'.$containerId.'">'
98
+                  . '<ul>'.$tabsNav.'</ul>'
99
+                  . '<div class="tab-content">'.$tabsContent.'</div>';
100 100
         if ($containerParams['pager']) {
101
-            $content .='<ul class="pager wizard">'
102
-                  . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
103
-                  . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
104
-                  . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
101
+            $content .= '<ul class="pager wizard">'
102
+                  . '<li class="previous"><a href="javascript:;">&larr; '.$translate('previous').'</a></li>'
103
+                  . '<li class="next"><a href="javascript:;">'.$translate('Next').' &rarr;</a></li>'
104
+                  . '<li class="finish'.($containerParams['finish_enabled'] ? '' : ' disabled').'">'
105 105
                   . (false !== $containerParams['finish_label']
106
-                     ? '<a class="pull-right" href="' . $containerParams['finish_href'] . '">'
107
-                       . $translate($containerParams['finish_label']) . ' &bull;</a>'
106
+                     ? '<a class="pull-right" href="'.$containerParams['finish_href'].'">'
107
+                       . $translate($containerParams['finish_label']).' &bull;</a>'
108 108
                      : ''
109 109
                     )
110 110
                   . '</li></ul>';
Please login to merge, or discard this patch.