Completed
Push — master ( 1eb1d9...f12af0 )
by Markus
03:08
created
src/BCRM/BackendBundle/Command/CreateTicketsCommand.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -65,6 +65,9 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,9 @@
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
src/BCRM/BackendBundle/Content/FileContentReader.php 2 patches
Doc Comments   +9 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,7 +52,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
src/BCRM/BackendBundle/Entity/Event/Ticket.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -251,7 +251,7 @@
 block discarded – undo
251 251
     }
252 252
 
253 253
     /**
254
-     * @return \DateTime|null
254
+     * @return boolean
255 255
      */
256 256
     public function getCheckinTime()
257 257
     {
Please login to merge, or discard this patch.
src/BCRM/BackendBundle/Service/Concierge.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -34,7 +34,6 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
src/BCRM/WebBundle/Content/PageContentReader.php 2 patches
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,6 +79,12 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
Please login to merge, or discard this patch.
Braces   +14 added lines, -5 removed lines patch added patch discarded remove patch
@@ -48,7 +48,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
     }
Please login to merge, or discard this patch.
src/BCRM/BackendBundle/Command/MailChimpUpdateParticipantsListCommand.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,9 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/BCRM/BackendBundle/Content/TwigLoader.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,9 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/BCRM/BackendBundle/Service/Payment.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -84,7 +84,9 @@
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
src/BCRM/WebBundle/Controller/ConciergeController.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -112,7 +112,9 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.