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

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     exit('This script is only accessible from localhost.');
 }
 
-require_once dirname(__FILE__).'/../var/SymfonyRequirements.php';
+require_once dirname(__FILE__) . '/../var/SymfonyRequirements.php';
 
 $symfonyRequirements = new SymfonyRequirements();
 

scripts/install.php

Spacing +17 added lines, -17 removed lines

@@ -26,7 +26,7 @@ 
 $options = getopt($shortOptions, $longOptions);
 
 if (array_key_exists('help', $options)) {
-    echo "dembelo install script, ".file_get_contents('./files/version')."\n";
+    echo "dembelo install script, " . file_get_contents('./files/version') . "\n";
     echo "\n";
     echo "options:\n";
     echo " -u\tuser for chmod (for example: www-data)\n";
@@ -35,7 +35,7 @@ 
 }
 
 $url = 'https://api.github.com/repos/typearea/dembelo/releases';
-$data = shell_exec('curl '.$url);
+$data = shell_exec('curl ' . $url);
 
 $releases = json_decode($data);
 
@@ -66,36 +66,36 @@ 
 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);
     }
 }
 
-echo 'download '.$downloadUrl."...\n";
-shell_exec('wget -q '.$downloadUrl);
-echo 'finished'."\n";
+echo 'download ' . $downloadUrl . "...\n";
+shell_exec('wget -q ' . $downloadUrl);
+echo 'finished' . "\n";
 
-echo 'extract '.$downloadName."...\n";
-shell_exec('unzip -o '.$downloadName);
-echo 'finished'."\n";
+echo 'extract ' . $downloadName . "...\n";
+shell_exec('unzip -o ' . $downloadName);
+echo 'finished' . "\n";
 
 if (array_key_exists('u', $options)) {
-    echo 'chown to '.$options['u']."...\n";
-    shell_exec('chown -R '.$options['u']. ' www/');
+    echo 'chown to ' . $options['u'] . "...\n";
+    shell_exec('chown -R ' . $options['u'] . ' www/');
 }
 
-echo 'clearing cache '."...\n";
+echo 'clearing cache ' . "...\n";
 shell_exec('cd www && php bin/console cache:clear --env=prod');
 shell_exec('cd www && php bin/console assetic:dump --env=prod');
-echo 'finished'."\n";
+echo 'finished' . "\n";
 
-echo 'prepare some apache stuff'."\n";
+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');
 }
-echo 'finished'."\n";
+echo 'finished' . "\n";
 
-shell_exec('rm '.$downloadName);
-echo 'installation ['.$newVersion.'] finished'."\n";
+shell_exec('rm ' . $downloadName);
+echo 'installation [' . $newVersion . '] finished' . "\n";
 exit(0);
\ No newline at end of file

www/web/app.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 use Symfony\Component\HttpFoundation\Request;
 
-$loader = require __DIR__.'/../vendor/autoload.php';
+$loader = require __DIR__ . '/../vendor/autoload.php';
 
 $kernel = new AppKernel('prod', false);
 //$kernel->loadClassCache();

www/web/app_dev.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     //exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');
 }
 
-$loader = require __DIR__.'/../vendor/autoload.php';
+$loader = require __DIR__ . '/../vendor/autoload.php';
 Debug::enable();
 
 $kernel = new AppKernel('dev', true);

www/app/autoload.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 /**
  * @var ClassLoader $loader
  */
-$loader = require __DIR__.'/../vendor/autoload.php';
+$loader = require __DIR__ . '/../vendor/autoload.php';
 
 AnnotationRegistry::registerLoader(array($loader, 'loadClass'));
 

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

Spacing +57 added lines, -57 removed lines

@@ -32,7 +32,7 @@ 
  * @param string $filename
  * @return bool
  */
-function fopen($filename)
+function fopen ($filename)
 {
     return strpos($filename, 'readable') !== false;
 }
@@ -42,7 +42,7 @@ 
  *
  * @return bool|mixed
  */
-function fgets()
+function fgets ()
 {
     if (empty(ImportTwineTest::$fgetsStack)) {
         return false;
@@ -57,7 +57,7 @@ 
  * @param Resource $handle
  * @param string   $string
  */
-function fputs($handle, $string)
+function fputs ($handle, $string)
 {
     ImportTwineTest::$fputsStack[] = $string;
 }
@@ -65,7 +65,7 @@ 
 /**
  * mock function
  */
-function fclose()
+function fclose ()
 {
 }
 
@@ -74,7 +74,7 @@ 
  *
  * @return bool|mixed
  */
-function fread()
+function fread ()
 {
     if (empty(ImportTwineTest::$freadStack)) {
         return false;
@@ -88,7 +88,7 @@ 
  *
  * @return bool
  */
-function feof()
+function feof ()
 {
     return empty(ImportTwineTest::$freadStack);
 }
@@ -98,7 +98,7 @@ 
  *
  * @return int
  */
-function fseek()
+function fseek ()
 {
     return 0;
 }
@@ -106,7 +106,7 @@ 
 /**
  * @param Resource $parser
  */
-function xml_parser_free($parser)
+function xml_parser_free ($parser)
 {
     ImportTwineTest::$parserFreeCalled = true;
 
@@ -142,7 +142,7 @@ 
     /**
      * resets some variables
      */
-    public function setUp()
+    public function setUp ()
     {
         self::$freadStack = self::$fgetsStack = self::$fputsStack = [];
         $this->mocks = [];
@@ -153,7 +153,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage Failed to read data from file 'somefilename'
      */
-    public function testRunWithFopenIsFalse()
+    public function testRunWithFopenIsFalse ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -168,7 +168,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage Failed to read data from file 'somefilename_readable'
      */
-    public function testRunWithFreadReturnFalse()
+    public function testRunWithFreadReturnFalse ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -187,7 +187,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage File 'somefilename_readable' isn't a Twine archive file
      */
-    public function testRunWithWrongFileFormat()
+    public function testRunWithWrongFileFormat ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -209,7 +209,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage File 'somefilename_readable' seems to be empty.
      */
-    public function testRunWithEmptyFirstLine()
+    public function testRunWithEmptyFirstLine ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -232,7 +232,7 @@ 
      *
      * @throws \Exception
      */
-    public function testRunWithCorrectButIncompleteData()
+    public function testRunWithCorrectButIncompleteData ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -256,7 +256,7 @@ 
      *
      * @throws \Exception
      */
-    public function testRunButNoTextnodeIsWritten()
+    public function testRunButNoTextnodeIsWritten ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -265,8 +265,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName" tags="Freigegeben ID:foobar" position="104,30">lorem impsum',
             'lorem impsum</tw-passagedata></tw-storydata>',
         ];
@@ -301,7 +301,7 @@ 
      *
      * @throws \Exception
      */
-    public function testRunWithSingleNodeWithText()
+    public function testRunWithSingleNodeWithText ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -310,8 +310,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="Freigegeben ID:foobar" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -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";
             }));
@@ -346,7 +346,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage The Twine archive file 'somefilename_readable' has a textnode named 'someNodeName1' which contains a malformed link that starts with '[[' but has no corresponding ']]'.
      */
-    public function testRunWithUnfinishedLinkTag()
+    public function testRunWithUnfinishedLinkTag ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -355,8 +355,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="Freigegeben ID:foobar" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -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";
             }));
@@ -394,7 +394,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage no licensee available
      */
-    public function testRunWithExceptionWhenNoLicenseeIsAvailable()
+    public function testRunWithExceptionWhenNoLicenseeIsAvailable ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -411,7 +411,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage no filename available
      */
-    public function testRunWithExceptionWhenNoFilenameIsAvailable()
+    public function testRunWithExceptionWhenNoFilenameIsAvailable ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -426,7 +426,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage no ID given for Textnode "someNodeName1"
      */
-    public function testRunWithTextnodeWithoutTwineID()
+    public function testRunWithTextnodeWithoutTwineID ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -435,8 +435,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="Freigegeben" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -464,7 +464,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage no ID given for Textnode "someNodeName1"
      */
-    public function testRunWithTextnodeWithoutAnyTag()
+    public function testRunWithTextnodeWithoutAnyTag ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -473,8 +473,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -502,7 +502,7 @@ 
      * tests identifying a textnode by a twine ID
      * @throws \Exception
      */
-    public function testRunWithTextnodeWithTwineID()
+    public function testRunWithTextnodeWithTwineID ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -511,8 +511,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="Freigegeben ID:foobar" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -541,7 +541,7 @@ 
      * tests the freeing of xml parser
      * @throws \Exception
      */
-    public function testParserFree()
+    public function testParserFree ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -550,8 +550,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="Freigegeben ID:foobar" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -582,7 +582,7 @@ 
      * tests disabling of a textnode that is no longer found in import file
      * @throws \Exception
      */
-    public function testRunWithTextnodeDeletedInImportfile()
+    public function testRunWithTextnodeDeletedInImportfile ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -591,8 +591,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="Freigegeben ID:foobar" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata></tw-storydata>',
         ];
@@ -627,7 +627,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage There is a 'tw-passagedata' in the Twine archive file 'somefilename_readable' which has a non unique 'id' tag [someTwineId], in node 'someNodeName2'
      */
-    public function testRunWithDuplicateTwineId()
+    public function testRunWithDuplicateTwineId ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -636,8 +636,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="ID:someTwineId" position="104,30">lorem ipsum',
             'lorem ipsum</tw-passagedata>',
             '<tw-passagedata pid="2" name="someNodeName2" tags="ID:someTwineId" position="104,30">lorem ipsum',
@@ -668,7 +668,7 @@ 
      * @expectedException Exception
      * @expectedExceptionMessage There is a textnode which references another textnode named 'someNodeName3', but this textnode doesn't exist within the same story.
      */
-    public function testRunWithLinkToNonExistingTextnode()
+    public function testRunWithLinkToNonExistingTextnode ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -677,8 +677,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="ID:someTwineId1" position="104,30">lorem ipsum',
             'lorem ipsum [[Linkdata->someNodeName3]]</tw-passagedata>',
             '</tw-storydata>',
@@ -708,7 +708,7 @@ 
      * tests a hitch to another textnode
      * @throws \Exception
      */
-    public function testRunWithLinkToAnotherTextnode()
+    public function testRunWithLinkToAnotherTextnode ()
     {
         $textnodeRepository = $this->getTextnodeRepositoryMock();
         $topicRepository = $this->getTopicRepositoryMock();
@@ -717,8 +717,8 @@ 
 
         self::$freadStack = [
             '<tw-storydata ',
-            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>'."\n",
-            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>'."\n",
+            '<tw-storydata name="someTopicId-->someStoryName" startnode="1" creator="Twine" creator-version="2.0.8" ifid="8E30D51C-4980-4161-B57F-B11C752E879A" format="Harlowe" options=""><style role="stylesheet" id="twine-user-stylesheet" type="text/twine-css"></style>' . "\n",
+            '<script role="script" id="twine-user-script" type="text/twine-javascript"></script>' . "\n",
             '<tw-passagedata pid="1" name="someNodeName1" tags="ID:someTwineId1" position="104,30">lorem ipsum',
             'lorem ipsum [[Linkdata->someNodeName2]]</tw-passagedata>',
             '<tw-passagedata pid="2" name="someNodeName2" tags="ID:someTwineId2" position="104,30">lorem ipsum',
@@ -745,9 +745,9 @@ 
 
         $textnodeRepository->expects($this->any())
             ->method('save')
-            ->willReturnCallback(function ($textnode) {
+            ->willReturnCallback(function($textnode) {
                 static $counter = 0;
-                $textnode->setId('someTextnode'.$counter++);
+                $textnode->setId('someTextnode' . $counter++);
             });
 
         $importTwine = new ImportTwine($textnodeRepository, $topicRepository);
@@ -756,7 +756,7 @@ 
         $this->assertTrue($returnValue);
     }
 
-    private function getDummyImportfile(array $data = [])
+    private function getDummyImportfile (array $data = [])
     {
         $default = [
             'filename'   => 'somefilename_readable',
@@ -789,17 +789,17 @@ 
         return $importfile;
     }
 
-    private function getTextnodeRepositoryMock()
+    private function getTextnodeRepositoryMock ()
     {
         return $this->getMockBuilder(TextNodeRepository::class)->disableOriginalConstructor()->setMethods(['createQueryBuilder', 'field', 'equals', 'getQuery', 'save', 'find', 'findByTwineId', 'disableOrphanedNodes', 'setHyphenatedText'])->getMock();
     }
 
-    private function getTopicRepositoryMock()
+    private function getTopicRepositoryMock ()
     {
         return $this->getMockBuilder(TopicRepository::class)->disableOriginalConstructor()->setMethods(['find'])->getMock();
     }
 
-    private function getDmMock()
+    private function getDmMock ()
     {
         return $this->getMockBuilder('dmMock')->setMethods(['flush', 'persist', 'find'])->getMock();
     }

www/src/AdminBundle/Tests/Controller/DefaultControllerTest.php

Spacing +20 added lines, -20 removed lines

@@ -56,7 +56,7 @@ 
     /**
      * tests the index action
      */
-    public function testIndexAction()
+    public function testIndexAction ()
     {
         $client = static::createClient();
         $crawler = $client->request('GET', '/admin/');
@@ -68,7 +68,7 @@ 
     /**
      * tests controller's userAction with no users in db
      */
-    public function testUserAction()
+    public function testUserAction ()
     {
         $request = $this->getMockBuilder(Request::class)->disableOriginalConstructor()->getMock();
         $postMock = $this->getMockBuilder(ParameterBag::class)->disableOriginalConstructor()->getMock();
@@ -105,7 +105,7 @@ 
     /**
      * tests controller's userAction with two users in db
      */
-    public function testUserActionWithUsers()
+    public function testUserActionWithUsers ()
     {
         $request = $this->getMockBuilder(Request::class)->disableOriginalConstructor()->getMock();
         $postMock = $this->getMockBuilder(ParameterBag::class)->disableOriginalConstructor()->getMock();
@@ -152,14 +152,14 @@ 
         /* @var $response \Symfony\Component\HttpFoundation\Response */
         $response = $controller->usersAction($request);
         $this->assertInstanceOf(Response::class, $response);
-        $this->assertJsonStringEqualsJsonString('[{"id":"id1","gender":null,"email":"email1","roles":"ROLE_ADMIN","licenseeId":"lic1","status":null,"source":null,"reason":null,"created":"'.date('Y-m-d H:i:s', 0).'","updated":"'.date('Y-m-d H:i:s', 0).'"},{"id":"id2","email":"email2","roles":"ROLE_USER","licenseeId":"lic2","status":null,"source":null,"reason":null,"gender":null,"created":"'.date('Y-m-d H:i:s', 0).'","updated":"'.date('Y-m-d H:i:s', 0).'"}]', $response->getContent());
+        $this->assertJsonStringEqualsJsonString('[{"id":"id1","gender":null,"email":"email1","roles":"ROLE_ADMIN","licenseeId":"lic1","status":null,"source":null,"reason":null,"created":"' . date('Y-m-d H:i:s', 0) . '","updated":"' . date('Y-m-d H:i:s', 0) . '"},{"id":"id2","email":"email2","roles":"ROLE_USER","licenseeId":"lic2","status":null,"source":null,"reason":null,"gender":null,"created":"' . date('Y-m-d H:i:s', 0) . '","updated":"' . date('Y-m-d H:i:s', 0) . '"}]', $response->getContent());
         $this->assertEquals('200', $response->getStatusCode());
     }
 
     /**
      * tear down method
      */
-    public function tearDown()
+    public function tearDown ()
     {
         $this->container = null;
         $this->repository = null;
@@ -169,7 +169,7 @@ 
     /**
      * tests the formsaveAction without parameters
      */
-    public function testFormsaveActionWithoutParameters()
+    public function testFormsaveActionWithoutParameters ()
     {
         $request = $this->getMockBuilder(Request::class)->disableOriginalConstructor()->getMock();
         $postMock = $this->getMockBuilder(ParameterBag::class)->disableOriginalConstructor()->getMock();
@@ -194,7 +194,7 @@ 
     /**
      * tests the formsaveAction with wrong parameters
      */
-    public function testFormsaveActionWithWrongParameters()
+    public function testFormsaveActionWithWrongParameters ()
     {
         $request = $this->getMockBuilder(Request::class)->disableOriginalConstructor()->getMock();
         $postMock = $this->getMockBuilder(ParameterBag::class)->disableOriginalConstructor()->getMock();
@@ -221,7 +221,7 @@ 
     /**
      * tests the formsaveAction with an existing user
      */
-    public function testFormsaveActionExistingUser()
+    public function testFormsaveActionExistingUser ()
     {
         $user = new User();
         $user->setId('someId');
@@ -261,7 +261,7 @@ 
     /**
      * tests the formsaveAction with a nonexisting user
      */
-    public function testFormsaveActionNotExistingUser()
+    public function testFormsaveActionNotExistingUser ()
     {
         $user = new User();
         $user->setId('someId');
@@ -300,7 +300,7 @@ 
     /**
      * tests the formsaveAction without admin permission
      */
-    public function testFormsaveActionWithoutAdminPermission()
+    public function testFormsaveActionWithoutAdminPermission ()
     {
         $client = static::createClient();
         $crawler = $client->request('GET', '/admin/save');
@@ -313,7 +313,7 @@ 
     /**
      * tests the formsaveAction with missing id parameter
      */
-    public function testFormsaveActionWithMissingIdParameter()
+    public function testFormsaveActionWithMissingIdParameter ()
     {
         $request = $this->getMockBuilder(Request::class)->disableOriginalConstructor()->getMock();
         $postMock = $this->getMockBuilder(ParameterBag::class)->disableOriginalConstructor()->getMock();
@@ -341,7 +341,7 @@ 
     /**
      * tests licenseeAction with no licensees
      */
-    public function testLicenseeActionWithNoLicensees()
+    public function testLicenseeActionWithNoLicensees ()
     {
         $repository = $this->getMockBuilder(LicenseeRepository::class)->disableOriginalConstructor()->setMethods(['findAll'])->getMock();
         $repository->expects($this->once())
@@ -367,7 +367,7 @@ 
     /**
      * tests licenseeAction() with one licensee
      */
-    public function testLicenseeActionWithOneLicensee()
+    public function testLicenseeActionWithOneLicensee ()
     {
         $licensee = new Licensee();
         $licensee->setName('someName');
@@ -397,7 +397,7 @@ 
     /**
      * tests textnode action
      */
-    public function testTextnodesAction()
+    public function testTextnodesAction ()
     {
         $textnode = new Textnode();
         $textnode->setId('someId');
@@ -464,7 +464,7 @@ 
         $this->assertJsonStringEqualsJsonString($expectedJson, $response->getContent());
     }
 
-    private function getLicenseeRepositoryMock()
+    private function getLicenseeRepositoryMock ()
     {
         $repository = $this->getMockBuilder(LicenseeRepository::class)
             ->disableOriginalConstructor()
@@ -474,7 +474,7 @@ 
         return $repository;
     }
 
-    private function getImportfileRepositoryMock()
+    private function getImportfileRepositoryMock ()
     {
         $repository = $this->getMockBuilder(ImportfileRepository::class)
             ->disableOriginalConstructor()
@@ -487,7 +487,7 @@ 
     /**
      * load mockoed container and mocked mongodb repository
      */
-    private function loadMongoContainer($repository)
+    private function loadMongoContainer ($repository)
     {
         $this->loadMockedContainer();
         $this->loadMockedMongoRepository($repository);
@@ -496,7 +496,7 @@ 
     /**
      * load mocked container
      */
-    private function loadMockedContainer()
+    private function loadMockedContainer ()
     {
         $this->container = $this->createMock(ContainerInterface::class);
     }
@@ -504,13 +504,13 @@ 
     /**
      * load mocked mongodb repository
      */
-    private function loadMockedMongoRepository($repository)
+    private function loadMockedMongoRepository ($repository)
     {
         $this->service = $this->getMockBuilder(ManagerRegistry::class)->disableOriginalConstructor()->getMock();
         $this->repository = $this->getMockBuilder(DocumentRepository::class)->disableOriginalConstructor()->getMock();
         $this->container->expects($this->once())
             ->method("get")
-            ->with($this->equalTo('app.model_repository_'.$repository))
+            ->with($this->equalTo('app.model_repository_' . $repository))
             ->will($this->returnValue($this->repository));
     }
 }

www/src/AdminBundle/Tests/Controller/TopicControllerTest.php

Spacing +6 added lines, -6 removed lines

@@ -36,7 +36,7 @@ 
     /**
      * tests topicAction() with no topics
      */
-    public function testListActionWithNoTopics()
+    public function testListActionWithNoTopics ()
     {
         $container = $this->getMockBuilder(ContainerInterface::class)->getMock();
 
@@ -62,7 +62,7 @@ 
     /**
      * tests topicAction with one topic
      */
-    public function testListActionWithOneTopic()
+    public function testListActionWithOneTopic ()
     {
         $topic = new Topic();
         $topic->setName('someName');
@@ -95,7 +95,7 @@ 
     /**
      * tests topicAction with some filters
      */
-    public function testListActionWithSomeFilters()
+    public function testListActionWithSomeFilters ()
     {
         $topic = new Topic();
         $topic->setName('someName');
@@ -136,7 +136,7 @@ 
     /**
      * tests uploadImageAction with an error in $_FILES
      */
-    public function testUploadImageActionWithNoFileError()
+    public function testUploadImageActionWithNoFileError ()
     {
         $repository = $this->getMockBuilder(TopicRepository::class)->disableOriginalConstructor()->setMethods(['findBy'])->getMock();
 
@@ -160,7 +160,7 @@ 
     /**
      * tests uploadImageAction
      */
-    public function testUploadImageActionWithNoError()
+    public function testUploadImageActionWithNoError ()
     {
         $repository = $this->getMockBuilder(TopicRepository::class)->disableOriginalConstructor()->setMethods(['findBy'])->getMock();
 
@@ -188,7 +188,7 @@ 
     /**
      * tear down method
      */
-    public function tearDown()
+    public function tearDown ()
     {
         $this->container = null;
         $this->repository = null;

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

Spacing +12 added lines, -12 removed lines

@@ -29,7 +29,7 @@ 
  * @param string $filename
  * @return bool
  */
-function is_readable($filename)
+function is_readable ($filename)
 {
     return \strpos($filename, 'readable') !== false;
 }
@@ -38,7 +38,7 @@ 
  * @param string $filename
  * @return bool
  */
-function file_exists($filename)
+function file_exists ($filename)
 {
     return \strpos($filename, 'exists') !== false;
 }
@@ -61,7 +61,7 @@ 
     private $command;
     private $commandTester;
 
-    protected function setUp()
+    protected function setUp ()
     {
         self::bootKernel();
         $application = new Application(self::$kernel);
@@ -75,11 +75,11 @@ 
     /**
      * tests the execute method
      */
-    public function testExecute()
+    public function testExecute ()
     {
         $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'
@@ -103,7 +103,7 @@ 
     /**
      * tests execute method with unreadable file
      */
-    public function testExecuteWithUnreadableFile()
+    public function testExecuteWithUnreadableFile ()
     {
         $this->mockObjects['importTwine']->expects($this->never())
             ->method('run');
@@ -125,7 +125,7 @@ 
     /**
      * tests execute() method with non existing file
      */
-    public function testExecuteWithFileNotExisting()
+    public function testExecuteWithFileNotExisting ()
     {
         $this->mockObjects['importTwine']->expects($this->never())
             ->method('run');
@@ -147,11 +147,11 @@ 
     /**
      * tests execute() method with exception thrown by importTwine
      */
-    public function testExecuteWithExeptionInImportTwine()
+    public function testExecuteWithExeptionInImportTwine ()
     {
         $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'
@@ -179,7 +179,7 @@ 
     /**
      * @expectedException Exception
      */
-    public function testExecuteWithLicenseeNotFound()
+    public function testExecuteWithLicenseeNotFound ()
     {
         $this->mockObjects['importTwine']->expects($this->never())
             ->method('run');
@@ -205,7 +205,7 @@ 
      * @param string $arg
      * @return null|\PHPUnit_Framework_MockObject_MockObject
      */
-    public function findOneByNameCallback($arg)
+    public function findOneByNameCallback ($arg)
     {
         if ($arg === 'somelicensee') {
             $licenseeMock = $this->getMockBuilder('DembeloMain\Document\Licensee')->disableOriginalConstructor()->getMock();
@@ -219,7 +219,7 @@ 
         return null;
     }
 
-    private function getMockObjects()
+    private function getMockObjects ()
     {
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerInterface')->getMock();