@@ -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; |