typearea / dembelo

www/web/config.php

Braces +9 added lines, -3 removed lines

@@ -382,7 +382,10 @@ 
                         <?php if ($hasMinorProblems): ?>
                             <h2>Recommendations</h2>
                             <p>
-                                <?php if ($hasMajorProblems): ?>Additionally, to<?php else: ?>To<?php endif; ?> enhance your Symfony experience,
+                                <?php if ($hasMajorProblems): ?>Additionally, to<?php else {
+    : ?>To<?php endif;
+}
+?> enhance your Symfony experience,
                                 it’s recommended that you fix the following:
                             </p>
                             <ol>
@@ -398,9 +401,12 @@ 
                             <p id="phpini">*
                                 <?php if ($symfonyRequirements->getPhpIniConfigPath()): ?>
                                     Changes to the <strong>php.ini</strong> file must be done in "<strong><?php echo $symfonyRequirements->getPhpIniConfigPath() ?></strong>".
-                                <?php else: ?>
+                                <?php else {
+    : ?>
                                     To change settings, create a "<strong>php.ini</strong>".
-                                <?php endif; ?>
+                                <?php endif;
+}
+?>
                             </p>
                         <?php endif; ?>
 

www/src/AdminBundle/Tests/Command/ImportCommandTest.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
     {
         $this->mockObjects['importTwine']->expects($this->once())
             ->method('run')
-            ->will($this->returnCallback(function ($importfile) {
+            ->will($this->returnCallback(function($importfile) {
                 return $importfile->getFilename() === 'somefile_readable_exists.html'
                 && $importfile->getLicenseeId() === 'licenseeId'
                 && $importfile->getAuthor() === 'someauthor'
@@ -151,7 +151,7 @@ 
     {
         $this->mockObjects['importTwine']->expects($this->once())
             ->method('run')
-            ->will($this->returnCallback(function ($importfile) {
+            ->will($this->returnCallback(function($importfile) {
                 return $importfile->getFilename() === 'somefile_readable_exists.html'
                 && $importfile->getLicenseeId() === 'licenseeId'
                 && $importfile->getAuthor() === 'someauthor'

www/src/AdminBundle/Tests/Model/ImportTwineTest.php

Spacing +3 added lines, -3 removed lines

@@ -330,7 +330,7 @@ 
 
         $textnodeRepository->expects($this->once())
             ->method('save')
-            ->with($this->callback(function ($textnode) {
+            ->with($this->callback(function($textnode) {
                 return $textnode instanceof Textnode
                     && $textnode->getText() === "lorem ipsumlorem ipsum";
             }));
@@ -376,7 +376,7 @@ 
 
         $textnodeRepository->expects($this->any())
             ->method('save')
-            ->with($this->callback(function ($textnode) {
+            ->with($this->callback(function($textnode) {
                 return $textnode instanceof Textnode
                 && $textnode->getText() === "lorem ipsumlorem ipsum";
             }));
@@ -745,7 +745,7 @@ 
 
         $textnodeRepository->expects($this->any())
             ->method('save')
-            ->willReturnCallback(function ($textnode) {
+            ->willReturnCallback(function($textnode) {
                 static $counter = 0;
                 $textnode->setId('someTextnode'.$counter++);
             });

www/src/AdminBundle/Model/ImportTwine.php

Spacing +4 added lines, -4 removed lines

@@ -186,7 +186,7 @@ 
                 continue;
             }
 
-            if ($peekDataLength - $i < $magicStringLength) {
+            if ($peekDataLength-$i < $magicStringLength) {
                 throw new Exception("File '".$this->importfile->getFilename()."' isn't a Twine archive file.");
             }
 
@@ -410,13 +410,13 @@ 
                 $textnodeTextNew = substr($textnodeText, 0, $startPos);
 
                 while ($startPos !== false) {
-                    $endPos = strpos($textnodeText, "]]", $startPos + strlen("[["));
+                    $endPos = strpos($textnodeText, "]]", $startPos+strlen("[["));
 
                     if ($endPos === false) {
                         throw new Exception("The Twine archive file '".$this->importfile->getFilename()."' has a textnode named '".$twineName."' which contains a malformed link that starts with '[[' but has no corresponding ']]'.");
                     }
 
-                    $content = substr($textnodeText, $startPos + strlen("[["), $endPos - ($startPos + strlen("[[")));
+                    $content = substr($textnodeText, $startPos+strlen("[["), $endPos-($startPos+strlen("[[")));
                     $hitch = null;
                     $metadata = null;
 
@@ -519,7 +519,7 @@ 
                     $startPos = strpos($textnodeText, "[[", $endPos);
 
                     if ($startPos !== false) {
-                        $textnodeTextNew .= substr($textnodeText, $endPos, $startPos - $endPos);
+                        $textnodeTextNew .= substr($textnodeText, $endPos, $startPos-$endPos);
                     } else {
                         $textnodeTextNew .= substr($textnodeText, $endPos);
                     }

www/src/DembeloMain/Tests/Command/InstallCommandTest.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
         $containerMock = $this->getMockBuilder(ContainerInterface::class)->getMock();
         $containerMock->expects($this->any())
             ->method('get')
-            ->willReturnCallback(function ($param) {
+            ->willReturnCallback(function($param) {
                 switch ($param) {
                     case 'app.model_repository_user':
                         return $this->userRepositoryMock;

www/src/DembeloMain/Tests/Document/TextnodeTest.php

Spacing +9 added lines, -9 removed lines

@@ -183,7 +183,7 @@ 
         $hitch['status'] = Textnode::HITCH_STATUS_ACTIVE;
         $this->textnode->appendHitch($hitch);
 
-        $result = $this->textnode->getHitch($this->textnode->getHitchCount() - 1);
+        $result = $this->textnode->getHitch($this->textnode->getHitchCount()-1);
 
         $this->assertFalse(is_null($result));
 
@@ -280,7 +280,7 @@ 
      */
     public function testAppendHitchMoreThanMaximum()
     {
-        for ($i = 0; $i < Textnode::HITCHES_MAXIMUM_COUNT + 1; $i++) {
+        for ($i = 0; $i < Textnode::HITCHES_MAXIMUM_COUNT+1; $i++) {
             $hitch = array();
             $hitch['textnodeId'] = $i;
             $hitch['description'] = "Continue.";
@@ -311,12 +311,12 @@ 
         $hitch['textnodeId'] = "55f5ab3708985c4b188b4578";
         $hitch['description'] = "Abort.";
         $hitch['status'] = Textnode::HITCH_STATUS_INACTIVE;
-        $result = $this->textnode->setHitch($this->textnode->getHitchCount() - 1, $hitch);
+        $result = $this->textnode->setHitch($this->textnode->getHitchCount()-1, $hitch);
 
         $this->assertTrue($result);
         $this->assertEquals($this->textnode->getHitchCount(), 1);
 
-        $result = $this->textnode->getHitch($this->textnode->getHitchCount() - 1);
+        $result = $this->textnode->getHitch($this->textnode->getHitchCount()-1);
 
         $this->assertFalse(is_null($result));
 
@@ -367,7 +367,7 @@ 
         $hitch = array();
         $hitch['description'] = "Abort.";
         $hitch['status'] = Textnode::HITCH_STATUS_INACTIVE;
-        $result = $this->textnode->setHitch($this->textnode->getHitchCount() - 1, $hitch);
+        $result = $this->textnode->setHitch($this->textnode->getHitchCount()-1, $hitch);
 
         $this->assertFalse($result);
     }
@@ -388,7 +388,7 @@ 
         $hitch = array();
         $hitch['textnodeId'] = "55f5ab3708985c4b188b4578";
         $hitch['status'] = Textnode::HITCH_STATUS_INACTIVE;
-        $result = $this->textnode->setHitch($this->textnode->getHitchCount() - 1, $hitch);
+        $result = $this->textnode->setHitch($this->textnode->getHitchCount()-1, $hitch);
 
         $this->assertFalse($result);
     }
@@ -409,7 +409,7 @@ 
         $hitch = array();
         $hitch['textnodeId'] = "55f5ab3708985c4b188b4578";
         $hitch['description'] = "Abort.";
-        $result = $this->textnode->setHitch($this->textnode->getHitchCount() - 1, $hitch);
+        $result = $this->textnode->setHitch($this->textnode->getHitchCount()-1, $hitch);
 
         $this->assertFalse($result);
     }
@@ -431,7 +431,7 @@ 
         $hitch['textnodeId'] = null;
         $hitch['description'] = "Abort.";
         $hitch['status'] = Textnode::HITCH_STATUS_INACTIVE;
-        $result = $this->textnode->setHitch($this->textnode->getHitchCount() - 1, $hitch);
+        $result = $this->textnode->setHitch($this->textnode->getHitchCount()-1, $hitch);
 
         $this->assertFalse($result);
     }
@@ -449,7 +449,7 @@ 
         $this->assertTrue($this->textnode->appendHitch($hitch));
         $this->assertEquals($this->textnode->getHitchCount(), 1);
 
-        $this->assertTrue($this->textnode->removeHitch($this->textnode->getHitchCount() - 1));
+        $this->assertTrue($this->textnode->removeHitch($this->textnode->getHitchCount()-1));
         $this->assertEquals($this->textnode->getHitchCount(), 0);
     }
 

www/src/DembeloMain/Tests/Model/ReadpathTest.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
 
         $readpathRepositoryMock->expects($this->once())
             ->method('save')
-            ->willReturnCallback(function (ReadpathDocument $readpathDocument) use ($textnodeMockId, $userMockId) {
+            ->willReturnCallback(function(ReadpathDocument $readpathDocument) use ($textnodeMockId, $userMockId) {
                 $this->assertInstanceOf(ReadpathDocument::class, $readpathDocument);
                 $this->assertEquals($textnodeMockId, $readpathDocument->getTextnodeId());
                 $this->assertEquals($userMockId, $readpathDocument->getUserId());

scripts/install.php

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
 $releases = json_decode($data);
 
 if (is_null($releases)) {
-    echo 'Data can\'t be parsed as JSON. exit' . "\n";
+    echo 'Data can\'t be parsed as JSON. exit'."\n";
     exit(1);
 }
 
@@ -59,14 +59,14 @@ 
 }
 
 if ($latestReleaseDate === 0) {
-    echo 'No release found. exit' . "\n";
+    echo 'No release found. exit'."\n";
     exit(1);
 }
 
 if (file_exists("files/version")) {
     $installedVersion = trim(file_get_contents("files/version"));
     if ($installedVersion === $newVersion) {
-        echo 'latest version ['.$installedVersion.'] already installed. exit' . "\n";
+        echo 'latest version ['.$installedVersion.'] already installed. exit'."\n";
         exit(1);
     }
 }
@@ -81,7 +81,7 @@ 
 
 if (array_key_exists('u', $options)) {
     echo 'chown to '.$options['u']."...\n";
-    shell_exec('chown -R '.$options['u']. ' www/');
+    shell_exec('chown -R '.$options['u'].' www/');
 }
 
 echo 'clearing cache '."...\n";
@@ -92,7 +92,7 @@ 
 echo 'prepare some apache stuff'."\n";
 shell_exec('cp files/apache/htaccess www/web/.htaccess');
 if (array_key_exists('u', $options)) {
-    shell_exec('chown ' . $options['u'] . ' www/web/.htaccess');
+    shell_exec('chown '.$options['u'].' www/web/.htaccess');
 }
 echo 'finished'."\n";
 

scripts/coverage-checker.php

Spacing +2 added lines, -2 removed lines

@@ -25,8 +25,8 @@
 $coverage = round(($checkedElements / $totalElements) * 100, 2);
 
 if ($coverage < $percentage) {
-    echo 'Code coverage is ' . $coverage . '%, which is below the accepted ' . $percentage . '%' . PHP_EOL;
+    echo 'Code coverage is '.$coverage.'%, which is below the accepted '.$percentage.'%'.PHP_EOL;
     exit(1);
 }
 
-echo 'Code coverage is ' . $coverage . '% - OK!' . PHP_EOL;
+echo 'Code coverage is '.$coverage.'% - OK!'.PHP_EOL;