Fix regular-expression exception.

bruli · bruli · commit 3fba9f36487d · 2015-09-06T17:49:01.000+02:00
diff --git a/src/PhpGitHooks/Application/Config/ConfigFile.php b/src/PhpGitHooks/Application/Config/ConfigFile.php
@@ -54,13 +54,16 @@ public function getMessageCommitConfiguration()
     {
         $data = $this->getConfigurationData();
 
-        if (!isset($data['commit-msg']) ||
-            !isset($data['commit-msg']['enabled']) ||
-            !isset($data['commit-msg']['regular-expression'])
-        ) {
+        if (!isset($data['commit-msg']) || !isset($data['commit-msg']['enabled'])) {
             throw new InvalidConfigStructureException();
         }
 
+        if (true === $data['commit-msg']['enabled']) {
+            if (!isset($data['commit-msg']['regular-expression'])) {
+                throw new InvalidConfigStructureException();
+            }
+        }
+
         return $data['commit-msg'];
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -54,13 +54,16 @@ public function getMessageCommitConfiguration()`
`54`	`54`	`{`
`55`	`55`	`$data = $this->getConfigurationData();`
`56`	`56`
`57`		`- if (!isset($data['commit-msg']) \|\|`
`58`		`- !isset($data['commit-msg']['enabled']) \|\|`
`59`		`- !isset($data['commit-msg']['regular-expression'])`
`60`		`- ) {`
	`57`	`+ if (!isset($data['commit-msg']) \|\| !isset($data['commit-msg']['enabled'])) {`
`61`	`58`	`throw new InvalidConfigStructureException();`
`62`	`59`	`}`
`63`	`60`
	`61`	`+ if (true === $data['commit-msg']['enabled']) {`
	`62`	`+ if (!isset($data['commit-msg']['regular-expression'])) {`
	`63`	`+ throw new InvalidConfigStructureException();`
	`64`	`+ }`
	`65`	`+ }`
	`66`	`+`
`64`	`67`	`return $data['commit-msg'];`
`65`	`68`	`}`
`66`	`69`	`}`