@@ -65,6 +65,9 @@ |
||
65 | 65 | $commandBus->handle($command); |
66 | 66 | } |
67 | 67 | |
68 | + /** |
|
69 | + * @param string $msg |
|
70 | + */ |
|
68 | 71 | protected function logVerbose($msg) |
69 | 72 | { |
70 | 73 | if ($this->output->getVerbosity() < OutputInterface::VERBOSITY_VERBOSE) return; |
@@ -67,7 +67,9 @@ |
||
67 | 67 | |
68 | 68 | protected function logVerbose($msg) |
69 | 69 | { |
70 | - if ($this->output->getVerbosity() < OutputInterface::VERBOSITY_VERBOSE) return; |
|
70 | + if ($this->output->getVerbosity() < OutputInterface::VERBOSITY_VERBOSE) { |
|
71 | + return; |
|
72 | + } |
|
71 | 73 | $this->output->writeln($msg); |
72 | 74 | } |
73 | 75 | } |
@@ -44,7 +44,7 @@ discard block |
||
44 | 44 | } |
45 | 45 | |
46 | 46 | /** |
47 | - * @param $path |
|
47 | + * @param string $path |
|
48 | 48 | * |
49 | 49 | * @return \SplFileInfo |
50 | 50 | */ |
@@ -71,6 +71,9 @@ discard block |
||
71 | 71 | return $c; |
72 | 72 | } |
73 | 73 | |
74 | + /** |
|
75 | + * @param string $markdown |
|
76 | + */ |
|
74 | 77 | protected function readProperties($markdown) |
75 | 78 | { |
76 | 79 | $properties = array(); |
@@ -81,6 +84,11 @@ discard block |
||
81 | 84 | return $properties; |
82 | 85 | } |
83 | 86 | |
87 | + /** |
|
88 | + * @param string $markdown |
|
89 | + * |
|
90 | + * @return string |
|
91 | + */ |
|
84 | 92 | protected function removeProperties($markdown) |
85 | 93 | { |
86 | 94 | return preg_replace(static::PROPERTIES_MATCH, '', $markdown); |
@@ -52,7 +52,9 @@ discard block |
||
52 | 52 | { |
53 | 53 | $contentdir = $this->contentDir->getPathname() . DIRECTORY_SEPARATOR; |
54 | 54 | $file = $contentdir . $path; |
55 | - if (!is_file($file)) throw new FileNotFoundException($path); |
|
55 | + if (!is_file($file)) { |
|
56 | + throw new FileNotFoundException($path); |
|
57 | + } |
|
56 | 58 | return new \SplFileInfo($file); |
57 | 59 | } |
58 | 60 | |
@@ -74,7 +76,9 @@ discard block |
||
74 | 76 | protected function readProperties($markdown) |
75 | 77 | { |
76 | 78 | $properties = array(); |
77 | - if (!preg_match_all(static::PROPERTIES_MATCH, $markdown, $matches, PREG_SET_ORDER)) return $properties; |
|
79 | + if (!preg_match_all(static::PROPERTIES_MATCH, $markdown, $matches, PREG_SET_ORDER)) { |
|
80 | + return $properties; |
|
81 | + } |
|
78 | 82 | foreach ($matches as $match) { |
79 | 83 | $properties[$match[1]] = $match[2]; |
80 | 84 | } |
@@ -251,7 +251,7 @@ |
||
251 | 251 | } |
252 | 252 | |
253 | 253 | /** |
254 | - * @return \DateTime|null |
|
254 | + * @return boolean |
|
255 | 255 | */ |
256 | 256 | public function getCheckinTime() |
257 | 257 | { |
@@ -34,7 +34,6 @@ |
||
34 | 34 | |
35 | 35 | /** |
36 | 36 | * @param CommandBus $commandBus |
37 | - * @param RouterInterface $router |
|
38 | 37 | * @param PaymentRepository $paymentRepo |
39 | 38 | * @param RegistrationRepository $registrationRepo |
40 | 39 | */ |
@@ -79,6 +79,12 @@ discard block |
||
79 | 79 | return $page; |
80 | 80 | } |
81 | 81 | |
82 | + /** |
|
83 | + * @param string $html |
|
84 | + * @param string $page |
|
85 | + * |
|
86 | + * @return string |
|
87 | + */ |
|
82 | 88 | protected function fixLinks($html, $page) |
83 | 89 | { |
84 | 90 | if (!preg_match_all('/src="([^"]+)"/', $html, $matches, PREG_SET_ORDER)) return $html; |
@@ -91,7 +97,7 @@ discard block |
||
91 | 97 | } |
92 | 98 | |
93 | 99 | /** |
94 | - * @param $path |
|
100 | + * @param string $path |
|
95 | 101 | * |
96 | 102 | * @return string |
97 | 103 | * @see http://www.php.net/manual/en/function.realpath.php#81935 |
@@ -48,7 +48,9 @@ discard block |
||
48 | 48 | $subNav = array(); |
49 | 49 | foreach ($this->getSubnav($this->getFilePath($path)) as $subPage) { |
50 | 50 | $s = $this->buildPage($subPage, false); |
51 | - if ($s->isHidden()) continue; |
|
51 | + if ($s->isHidden()) { |
|
52 | + continue; |
|
53 | + } |
|
52 | 54 | $n = new Nav(); |
53 | 55 | $n->setTitle($s->getProperties()->get('title')); |
54 | 56 | $n->setPath(str_replace('.md', '', $subPage)); |
@@ -81,7 +83,9 @@ discard block |
||
81 | 83 | |
82 | 84 | protected function fixLinks($html, $page) |
83 | 85 | { |
84 | - if (!preg_match_all('/src="([^"]+)"/', $html, $matches, PREG_SET_ORDER)) return $html; |
|
86 | + if (!preg_match_all('/src="([^"]+)"/', $html, $matches, PREG_SET_ORDER)) { |
|
87 | + return $html; |
|
88 | + } |
|
85 | 89 | $path = $this->contentPath . '/' . dirname($page) . '/'; |
86 | 90 | foreach ($matches as $match) { |
87 | 91 | $srcpath = $this->normalizePath($path . $match[1]); |
@@ -100,9 +104,14 @@ discard block |
||
100 | 104 | { |
101 | 105 | $out = array(); |
102 | 106 | foreach (explode('/', $path) as $i => $fold) { |
103 | - if ($fold == '' || $fold == '.') continue; |
|
104 | - if ($fold == '..' && $i > 0 && end($out) != '..') array_pop($out); |
|
105 | - else $out[] = $fold; |
|
107 | + if ($fold == '' || $fold == '.') { |
|
108 | + continue; |
|
109 | + } |
|
110 | + if ($fold == '..' && $i > 0 && end($out) != '..') { |
|
111 | + array_pop($out); |
|
112 | + } else { |
|
113 | + $out[] = $fold; |
|
114 | + } |
|
106 | 115 | } |
107 | 116 | return ($path{0} == '/' ? '/' : '') . join('/', $out); |
108 | 117 | } |
@@ -61,7 +61,9 @@ |
||
61 | 61 | foreach ($ticketRepo->getTicketsForEvent($eventRepo->getNextEvent()->getOrThrow( |
62 | 62 | new BadMethodCallException('No event.') |
63 | 63 | )) as $ticket) { |
64 | - if ($participants->contains(strtolower($ticket->getEmail()))) continue; |
|
64 | + if ($participants->contains(strtolower($ticket->getEmail()))) { |
|
65 | + continue; |
|
66 | + } |
|
65 | 67 | $participants->add(strtolower($ticket->getEmail())); |
66 | 68 | } |
67 | 69 |
@@ -33,7 +33,9 @@ |
||
33 | 33 | |
34 | 34 | protected function getFile($name) |
35 | 35 | { |
36 | - if (!preg_match('/^bcrm_content:(.+)/', $name, $match)) throw new Twig_Error_Loader(sprintf('Unknown template: %s', $name)); |
|
36 | + if (!preg_match('/^bcrm_content:(.+)/', $name, $match)) { |
|
37 | + throw new Twig_Error_Loader(sprintf('Unknown template: %s', $name)); |
|
38 | + } |
|
37 | 39 | return $match[1]; |
38 | 40 | } |
39 | 41 |
@@ -84,7 +84,9 @@ |
||
84 | 84 | |
85 | 85 | $data = array(); |
86 | 86 | foreach (explode("\n", $res) as $line) { |
87 | - if (!strpos($line, '=')) continue; |
|
87 | + if (!strpos($line, '=')) { |
|
88 | + continue; |
|
89 | + } |
|
88 | 90 | list($key, $value) = explode('=', $line); |
89 | 91 | $data[$key] = $value; |
90 | 92 | } |
@@ -112,7 +112,9 @@ |
||
112 | 112 | Ticket::DAY_SATURDAY => $formData->getSaturday(), |
113 | 113 | Ticket::DAY_SUNDAY => $formData->getSunday(), |
114 | 114 | ) as $day => $want) { |
115 | - if (!$want) continue; |
|
115 | + if (!$want) { |
|
116 | + continue; |
|
117 | + } |
|
116 | 118 | $ticketCommand = new CreateTicketCommand(); |
117 | 119 | $ticketCommand->day = $day; |
118 | 120 | $ticketCommand->event = $event; |