@@ -6,7 +6,7 @@ |
||
6 | 6 | |
7 | 7 | $client = \Comodojo\Daemon\Socket\Client::create('unix://extender.sock'); |
8 | 8 | |
9 | - $data = $client->send('scheduler:refresh',[]); |
|
9 | + $data = $client->send('scheduler:refresh', []); |
|
10 | 10 | |
11 | 11 | } catch (Exception $e) { |
12 | 12 |
@@ -48,6 +48,9 @@ discard block |
||
48 | 48 | |
49 | 49 | } |
50 | 50 | |
51 | + /** |
|
52 | + * @param integer $uid |
|
53 | + */ |
|
51 | 54 | public function init($uid) { |
52 | 55 | |
53 | 56 | $this->ipc[$uid] = []; |
@@ -72,6 +75,10 @@ discard block |
||
72 | 75 | |
73 | 76 | } |
74 | 77 | |
78 | + /** |
|
79 | + * @param integer $uid |
|
80 | + * @param string $data |
|
81 | + */ |
|
75 | 82 | public function write($uid, $data) { |
76 | 83 | |
77 | 84 | $writer = $this->ipc[$uid][self::WRITER]; |
@@ -136,7 +136,7 @@ discard block |
||
136 | 136 | /** |
137 | 137 | * Get cron expression of this schedule |
138 | 138 | * |
139 | - * @return string |
|
139 | + * @return CronExpression |
|
140 | 140 | */ |
141 | 141 | public function getExpression() { |
142 | 142 | |
@@ -147,7 +147,7 @@ discard block |
||
147 | 147 | /** |
148 | 148 | * set cron expression for this schedule |
149 | 149 | * |
150 | - * @param srting $expression A cron-compatible expression |
|
150 | + * @param CronExpression $expression A cron-compatible expression |
|
151 | 151 | * @return Schedule |
152 | 152 | */ |
153 | 153 | public function setExpression(CronExpression $expression) { |
@@ -177,7 +177,7 @@ discard block |
||
177 | 177 | /** |
178 | 178 | * Set enable/disable status |
179 | 179 | * |
180 | - * @param bool $enable |
|
180 | + * @param boolean $enabled |
|
181 | 181 | * @return Schedule |
182 | 182 | */ |
183 | 183 | public function setEnabled($enabled) { |
@@ -58,6 +58,9 @@ |
||
58 | 58 | |
59 | 59 | } |
60 | 60 | |
61 | + /** |
|
62 | + * @return \Doctrine\ORM\EntityManagerInterface |
|
63 | + */ |
|
61 | 64 | public function getEntityManager() { |
62 | 65 | |
63 | 66 | if ( $this->entity_manager === null ) { |
@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | false |
85 | 85 | ]; |
86 | 86 | |
87 | - switch (strtoupper($metadata_mode)) { |
|
87 | + switch ( strtoupper($metadata_mode) ) { |
|
88 | 88 | |
89 | 89 | case 'YAML': |
90 | 90 | $db_config = Setup::createYAMLMetadataConfiguration(...$config_args); |
@@ -178,7 +178,7 @@ discard block |
||
178 | 178 | $repos = $configuration->get('database-repositories'); |
179 | 179 | |
180 | 180 | return array_map(function($repo) use ($base_folder) { |
181 | - return substr($repo, 0, 1 ) === "/" ? $repo : "$base_folder/$repo"; |
|
181 | + return substr($repo, 0, 1) === "/" ? $repo : "$base_folder/$repo"; |
|
182 | 182 | }, $repos); |
183 | 183 | |
184 | 184 | } |
@@ -188,7 +188,7 @@ discard block |
||
188 | 188 | $base_folder = $configuration->get('base-path'); |
189 | 189 | $folder = $configuration->get('database-proxies'); |
190 | 190 | |
191 | - return substr($folder, 0, 1 ) === "/" ? $folder : "$base_folder/$folder"; |
|
191 | + return substr($folder, 0, 1) === "/" ? $folder : "$base_folder/$folder"; |
|
192 | 192 | |
193 | 193 | } |
194 | 194 |
@@ -41,7 +41,6 @@ |
||
41 | 41 | * |
42 | 42 | * @param Configuration $configuration |
43 | 43 | * @param LoggerInterface $logger |
44 | - * @param TasksTable $tasks |
|
45 | 44 | * @param EventsManager $events |
46 | 45 | * @param EntityManager $em |
47 | 46 | */ |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | |
81 | 81 | $em = $this->getEntityManager(); |
82 | 82 | |
83 | - foreach ($queue as $record) { |
|
83 | + foreach ( $queue as $record ) { |
|
84 | 84 | $em->remove($record); |
85 | 85 | } |
86 | 86 | |
@@ -106,7 +106,7 @@ discard block |
||
106 | 106 | |
107 | 107 | $records = []; |
108 | 108 | |
109 | - foreach ($queue as $name => $request) { |
|
109 | + foreach ( $queue as $name => $request ) { |
|
110 | 110 | $records[] = $request instanceof Request ? $this->doAddRequest($request, $em) : false; |
111 | 111 | } |
112 | 112 |
@@ -75,12 +75,10 @@ discard block |
||
75 | 75 | /** |
76 | 76 | * Class constructor |
77 | 77 | * |
78 | - * @param string $manager_name |
|
79 | 78 | * @param Configuration $configuration |
80 | 79 | * @param LoggerInterface $logger |
81 | 80 | * @param TasksTable $tasks |
82 | 81 | * @param EventsManager $events |
83 | - * @param EntityManager $em |
|
84 | 82 | */ |
85 | 83 | public function __construct( |
86 | 84 | Locker $locker, |
@@ -413,6 +411,9 @@ discard block |
||
413 | 411 | |
414 | 412 | } |
415 | 413 | |
414 | + /** |
|
415 | + * @param string $message |
|
416 | + */ |
|
416 | 417 | private function generateSyntheticResult($uid, $message, $jid = null, $success = true) { |
417 | 418 | |
418 | 419 | return new Result([ |
@@ -437,6 +438,9 @@ discard block |
||
437 | 438 | |
438 | 439 | } |
439 | 440 | |
441 | + /** |
|
442 | + * @param null|integer $pid |
|
443 | + */ |
|
440 | 444 | private function updateTrackerSetRunning($uid, $pid) { |
441 | 445 | |
442 | 446 | $this->tracker->setRunning($uid, $pid); |
@@ -466,6 +470,9 @@ discard block |
||
466 | 470 | |
467 | 471 | } |
468 | 472 | |
473 | + /** |
|
474 | + * @param Result $result |
|
475 | + */ |
|
469 | 476 | private function updateTrackerSetAborted($uid, $result) { |
470 | 477 | |
471 | 478 | $this->tracker->setAborted($uid, $result); |
@@ -133,9 +133,9 @@ discard block |
||
133 | 133 | |
134 | 134 | public function addBulk(array $requests) { |
135 | 135 | |
136 | - foreach ($requests as $id => $request) { |
|
136 | + foreach ( $requests as $id => $request ) { |
|
137 | 137 | |
138 | - if ($request instanceof \Comodojo\Extender\Task\Request) { |
|
138 | + if ( $request instanceof \Comodojo\Extender\Task\Request ) { |
|
139 | 139 | $this->add($request); |
140 | 140 | } else { |
141 | 141 | $this->logger->error("Skipping invalid request with local id $id: class mismatch"); |
@@ -174,7 +174,7 @@ discard block |
||
174 | 174 | |
175 | 175 | protected function cycle() { |
176 | 176 | |
177 | - foreach ($this->tracker->getQueued() as $uid => $request) { |
|
177 | + foreach ( $this->tracker->getQueued() as $uid => $request ) { |
|
178 | 178 | |
179 | 179 | if ( $this->multithread === false ) { |
180 | 180 | |
@@ -202,7 +202,7 @@ discard block |
||
202 | 202 | |
203 | 203 | if ( $this->max_childs > 0 && $this->tracker->countRunning() >= $this->max_childs ) { |
204 | 204 | |
205 | - while( $this->tracker->countRunning() >= $this->max_childs ) { |
|
205 | + while ( $this->tracker->countRunning() >= $this->max_childs ) { |
|
206 | 206 | |
207 | 207 | $this->catcher(); |
208 | 208 |
@@ -190,6 +190,15 @@ discard block |
||
190 | 190 | |
191 | 191 | } |
192 | 192 | |
193 | + /** |
|
194 | + * @param integer $uid |
|
195 | + * @param integer $puid |
|
196 | + * @param string $name |
|
197 | + * @param integer $jid |
|
198 | + * @param string $task |
|
199 | + * @param TaskParameters $parameters |
|
200 | + * @param \DateTime $start |
|
201 | + */ |
|
193 | 202 | protected function openWorklog( |
194 | 203 | $uid, |
195 | 204 | $puid, |
@@ -221,6 +230,9 @@ discard block |
||
221 | 230 | |
222 | 231 | } |
223 | 232 | |
233 | + /** |
|
234 | + * @param \DateTime $end |
|
235 | + */ |
|
224 | 236 | protected function closeWorklog( |
225 | 237 | $status, |
226 | 238 | $result, |
@@ -86,7 +86,7 @@ discard block |
||
86 | 86 | |
87 | 87 | $thetask = $this->table->get($task)->getInstance($name, $parameters); |
88 | 88 | |
89 | - $this->events->emit( new TaskEvent('start', $thetask) ); |
|
89 | + $this->events->emit(new TaskEvent('start', $thetask)); |
|
90 | 90 | |
91 | 91 | $pid = $thetask->getPid(); |
92 | 92 | |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | |
122 | 122 | } |
123 | 123 | |
124 | - $this->events->emit( new TaskEvent('stop', $thetask) ); |
|
124 | + $this->events->emit(new TaskEvent('stop', $thetask)); |
|
125 | 125 | |
126 | 126 | $this->stopwatch->stop(); |
127 | 127 |
@@ -44,7 +44,6 @@ |
||
44 | 44 | /** |
45 | 45 | * Tracker constructor |
46 | 46 | * |
47 | - * @param string $name |
|
48 | 47 | * @param Configuration $configuration |
49 | 48 | * @param LoggerInterface $logger |
50 | 49 | */ |
@@ -41,7 +41,6 @@ |
||
41 | 41 | * |
42 | 42 | * @param Configuration $configuration |
43 | 43 | * @param LoggerInterface $logger |
44 | - * @param TasksTable $tasks |
|
45 | 44 | * @param EventsManager $events |
46 | 45 | * @param EntityManager $em |
47 | 46 | */ |
@@ -144,7 +144,7 @@ |
||
144 | 144 | $records = []; |
145 | 145 | $em = $this->getEntityManager(); |
146 | 146 | |
147 | - foreach ($schedules as $key => $schedule) { |
|
147 | + foreach ( $schedules as $key => $schedule ) { |
|
148 | 148 | |
149 | 149 | try { |
150 | 150 |