typearea / dembelo

www/src/AdminBundle/Model/ImportTwine.php

Spacing +52 added lines, -52 removed lines

@@ -76,7 +76,7 @@ 
      * @param TextnodeRepositoryInterface $textnodeRepository
      * @param TopicRepositoryInterface    $topicRepository
      */
-    public function __construct(TextNodeRepositoryInterface $textnodeRepository, TopicRepositoryInterface $topicRepository)
+    public function __construct (TextNodeRepositoryInterface $textnodeRepository, TopicRepositoryInterface $topicRepository)
     {
         $this->textnodeRepository = $textnodeRepository;
         $this->topicRepository = $topicRepository;
@@ -89,7 +89,7 @@ 
      * @return bool
      * @throws Exception
      */
-    public function run(Importfile $importfile)
+    public function run (Importfile $importfile)
     {
         $this->importfile = $importfile;
 
@@ -102,7 +102,7 @@ 
         $fileHandler = fopen($filenameExtracted, "rb");
 
         if ($fileHandler === false) {
-            throw new Exception("Couldn't open file '".$this->importfile->getFilename()."'");
+            throw new Exception("Couldn't open file '" . $this->importfile->getFilename() . "'");
         }
 
         $this->xmlParser = xml_parser_create("UTF-8");
@@ -119,23 +119,23 @@ 
     /**
      * destroys the parser object
      */
-    public function parserFree()
+    public function parserFree ()
     {
         xml_parser_free($this->xmlParser);
     }
 
-    private function extractTwineFile()
+    private function extractTwineFile ()
     {
         if (is_null($this->importfile->getFilename())) {
             throw new Exception('no filename available');
         }
 
-        $extractedFile = $this->importfile->getFilename().'.extracted';
+        $extractedFile = $this->importfile->getFilename() . '.extracted';
 
         $fileHandle = fopen($this->importfile->getFilename(), 'r');
         $extractedFileHandle = fopen($extractedFile, 'w');
         if ($fileHandle === false) {
-            throw new Exception("Failed to read data from file '".$this->importfile->getFilename()."'.");
+            throw new Exception("Failed to read data from file '" . $this->importfile->getFilename() . "'.");
         }
         $writing = false;
         $matches = array();
@@ -160,7 +160,7 @@ 
         return $extractedFile;
     }
 
-    private function checkTwineFile($fileHandler)
+    private function checkTwineFile ($fileHandler)
     {
         $magicString = "<tw-storydata ";
         $magicStringLength = strlen($magicString);
@@ -168,13 +168,13 @@ 
         $peekData = fread($fileHandler, 1024);
 
         if ($peekData === false) {
-            throw new Exception("Failed to read data from file '".$this->importfile->getFilename()."'.");
+            throw new Exception("Failed to read data from file '" . $this->importfile->getFilename() . "'.");
         }
 
         $peekDataLength = strlen($peekData);
 
         if ($peekDataLength <= 0) {
-            throw new Exception("File '".$this->importfile->getFilename()."' seems to be empty.");
+            throw new Exception("File '" . $this->importfile->getFilename() . "' seems to be empty.");
         }
 
         for ($i = 0; $i < $peekDataLength; $i++) {
@@ -187,24 +187,24 @@ 
             }
 
             if ($peekDataLength - $i < $magicStringLength) {
-                throw new Exception("File '".$this->importfile->getFilename()."' isn't a Twine archive file.");
+                throw new Exception("File '" . $this->importfile->getFilename() . "' isn't a Twine archive file.");
             }
 
             if (substr($peekData, $i, $magicStringLength) !== $magicString) {
-                throw new Exception("File '".$this->importfile->getFilename()."' isn't a Twine archive file.");
+                throw new Exception("File '" . $this->importfile->getFilename() . "' isn't a Twine archive file.");
             }
 
             if (fseek($fileHandler, 0) !== 0) {
-                throw new Exception("Couldn't reset reading position after the magic string in the Twine archive file '".$this->importfile->getFilename()."' was checked.");
+                throw new Exception("Couldn't reset reading position after the magic string in the Twine archive file '" . $this->importfile->getFilename() . "' was checked.");
             }
 
             return true;
         }
 
-        throw new Exception("File '".$this->importfile->getFilename()."' doesn't seem to be a Twine archive file.");
+        throw new Exception("File '" . $this->importfile->getFilename() . "' doesn't seem to be a Twine archive file.");
     }
 
-    private function initParser($fileHandler)
+    private function initParser ($fileHandler)
     {
         xml_parser_set_option($this->xmlParser, XML_OPTION_CASE_FOLDING, 0);
 
@@ -221,7 +221,7 @@ 
             $errorCode = xml_get_error_code($this->xmlParser);
             $errorDescription = xml_error_string($errorCode);
 
-            throw new Exception("Error #".$errorCode.": '".$errorDescription."' occurred while the envelope head for the Twine archive was parsed.");
+            throw new Exception("Error #" . $errorCode . ": '" . $errorDescription . "' occurred while the envelope head for the Twine archive was parsed.");
         }
 
         do {
@@ -237,7 +237,7 @@ 
                 $errorColumnNumber = xml_get_current_column_number($this->xmlParser);
                 $errorByteIndex = xml_get_current_byte_index($this->xmlParser);
 
-                throw new Exception("Error #".$errorCode.": '".$errorDescription."' occurred while parsing the Twine archive file '".$this->importfile->getFilename()."' in line ".$errorRowNumber.", character ".$errorColumnNumber." (at byte index ".$errorByteIndex.").");
+                throw new Exception("Error #" . $errorCode . ": '" . $errorDescription . "' occurred while parsing the Twine archive file '" . $this->importfile->getFilename() . "' in line " . $errorRowNumber . ", character " . $errorColumnNumber . " (at byte index " . $errorByteIndex . ").");
             }
         } while (feof($fileHandler) === false);
 
@@ -245,7 +245,7 @@ 
             $errorCode = xml_get_error_code($this->xmlParser);
             $errorDescription = xml_error_string($errorCode);
 
-            throw new Exception("Error #".$errorCode.": '".$errorDescription."' occurred while the envelope foot for the Twine archive was parsed.");
+            throw new Exception("Error #" . $errorCode . ": '" . $errorDescription . "' occurred while the envelope foot for the Twine archive was parsed.");
         }
 
         $this->parserFree();
@@ -258,10 +258,10 @@ 
         return true;
     }
 
-    private function startElementStoryData($name, array $attrs)
+    private function startElementStoryData ($name, array $attrs)
     {
         if ($this->twineRelevant === true) {
-            throw new Exception("Nested '".$name."' found in Twine archive file '".$this->importfile->getFilename()."'.");
+            throw new Exception("Nested '" . $name . "' found in Twine archive file '" . $this->importfile->getFilename() . "'.");
         }
 
         if (isset($attrs['startnode']) !== true) {
@@ -273,13 +273,13 @@ 
         }
 
         if (isset($attrs['name']) !== true) {
-            throw new Exception("There is a '".$name."' in the Twine archive file '".$this->importfile->getFilename()."' which is missing its 'name' attribute.");
+            throw new Exception("There is a '" . $name . "' in the Twine archive file '" . $this->importfile->getFilename() . "' which is missing its 'name' attribute.");
         }
 
         $twineStory = explode("-->", $attrs['name'], 2);
 
         if (count($twineStory) !== 2) {
-            throw new Exception("There is a '".$name."' in the Twine archive file '".$this->importfile->getFilename()."' which has an incomplete 'name' attribute. Twine stories must use the naming schema '?-->story name', where '?' is an existing Dembelo Topic Id. Instead, '".$attrs['name']."' was found.");
+            throw new Exception("There is a '" . $name . "' in the Twine archive file '" . $this->importfile->getFilename() . "' which has an incomplete 'name' attribute. Twine stories must use the naming schema '?-->story name', where '?' is an existing Dembelo Topic Id. Instead, '" . $attrs['name'] . "' was found.");
         }
 
         $this->topicId = $twineStory[0];
@@ -287,7 +287,7 @@ 
         $topic = $this->topicRepository->find($this->topicId);
 
         if (is_null($topic)) {
-            throw new Exception("The Dembelo Topic with Id '".$this->topicId."', referenced by Twine story '".$attrs['name']."' in the Twine archive file '".$this->importfile->getFilename()."', doesn't exist.");
+            throw new Exception("The Dembelo Topic with Id '" . $this->topicId . "', referenced by Twine story '" . $attrs['name'] . "' in the Twine archive file '" . $this->importfile->getFilename() . "', doesn't exist.");
         }
 
         $this->twineStartnodeId = $attrs['startnode'];
@@ -295,10 +295,10 @@ 
         $this->twineRelevant = true;
     }
 
-    private function getTwineId($tagString, $textnodeTitle)
+    private function getTwineId ($tagString, $textnodeTitle)
     {
         if (empty($tagString) || !is_string($tagString)) {
-            throw new Exception('no ID given for Textnode "'.$textnodeTitle.'"');
+            throw new Exception('no ID given for Textnode "' . $textnodeTitle . '"');
         }
         $tagArray = explode(" ", $tagString);
 
@@ -311,30 +311,30 @@ 
         }
 
         if ($twineId === false) {
-            throw new Exception('no ID given for Textnode "'.$textnodeTitle.'"');
+            throw new Exception('no ID given for Textnode "' . $textnodeTitle . '"');
         }
 
         return $twineId;
     }
 
-    private function startElementPassageData($name, $attrs)
+    private function startElementPassageData ($name, $attrs)
     {
         if ($this->twineText !== false) {
-            throw new Exception("Nested '".$name."' found in Twine archive file '".$this->importfile->getFilename()."'.");
+            throw new Exception("Nested '" . $name . "' found in Twine archive file '" . $this->importfile->getFilename() . "'.");
         }
 
         if (isset($attrs['pid']) !== true) {
-            throw new Exception("There is a '".$name."' in the Twine archive file '".$this->importfile->getFilename()."' which is missing its 'pid' attribute.");
+            throw new Exception("There is a '" . $name . "' in the Twine archive file '" . $this->importfile->getFilename() . "' which is missing its 'pid' attribute.");
         }
 
         if (is_numeric($attrs['pid']) !== true) {
-            throw new Exception("There is a '".$name."' in the Twine archive file '".$this->importfile->getFilename()."' which hasn't a numeric value in its 'pid' attribute ('".$attrs['pid']."' was found instead).");
+            throw new Exception("There is a '" . $name . "' in the Twine archive file '" . $this->importfile->getFilename() . "' which hasn't a numeric value in its 'pid' attribute ('" . $attrs['pid'] . "' was found instead).");
         }
 
         $this->twineId = $this->getTwineId($attrs['tags'], $attrs['name']);
 
         if (array_key_exists($this->twineId, $this->textnodeMapping) === true) {
-            throw new Exception("There is a '".$name."' in the Twine archive file '".$this->importfile->getFilename()."' which has a non unique 'id' tag [".$this->twineId."], in node '".$attrs['name']."'");
+            throw new Exception("There is a '" . $name . "' in the Twine archive file '" . $this->importfile->getFilename() . "' which has a non unique 'id' tag [" . $this->twineId . "], in node '" . $attrs['name'] . "'");
         }
 
         $this->twineTextnodeName = $attrs['name'];
@@ -366,7 +366,7 @@ 
                 $this->textnode->setAccess(true);
                 $this->accessSet = true;
             } else {
-                throw new Exception("There is more than one '".$name."' in the Twine archive file '".$this->importfile->getFilename()."' with the startnode value '".$attrs['pid']."' in its 'pid' attribute.");
+                throw new Exception("There is more than one '" . $name . "' in the Twine archive file '" . $this->importfile->getFilename() . "' with the startnode value '" . $attrs['pid'] . "' in its 'pid' attribute.");
             }
         } else {
             $this->textnode->setAccess(false);
@@ -375,7 +375,7 @@ 
         $this->twineText = true;
     }
 
-    private function startElement($parser, $name, $attrs)
+    private function startElement ($parser, $name, $attrs)
     {
         if ($name === "tw-storydata") {
             $this->startElementStoryData($name, $attrs);
@@ -384,21 +384,21 @@ 
         }
     }
 
-    private function characterData($parser, $data)
+    private function characterData ($parser, $data)
     {
         if ($this->twineRelevant === true && $this->twineText === true) {
-            $this->textnode->setText($this->textnode->getText().$data);
+            $this->textnode->setText($this->textnode->getText() . $data);
         }
     }
 
-    private function endElementStoryData($name)
+    private function endElementStoryData ($name)
     {
         foreach ($this->textnodeMapping as $twineId => $dembeloId) {
             $textnode = $this->textnodeRepository->find($dembeloId);
             $twineName = array_search($dembeloId, $this->nodeNameMapping);
 
             if (is_null($textnode) === true) {
-                throw new Exception("The Dembelo Textnode with Id '".$dembeloId."' doesn't exist, but should by now.");
+                throw new Exception("The Dembelo Textnode with Id '" . $dembeloId . "' doesn't exist, but should by now.");
             }
 
             /** @todo The links should be exported as XML as well instead of a custom Twine inline format. */
@@ -413,7 +413,7 @@ 
                     $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 ']]'.");
+                        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("[[")));
@@ -424,13 +424,13 @@ 
                         $content = explode("-->", $content, 2);
 
                         if (strlen($content[0]) <= 0 || strlen($content[1]) <= 0) {
-                            throw new Exception("The Twine archive file '".$this->importfile->getFilename()."' contains a '".$name."' with the invalid element '[[".$content[0]."-->".$content[1]."]]'.");
+                            throw new Exception("The Twine archive file '" . $this->importfile->getFilename() . "' contains a '" . $name . "' with the invalid element '[[" . $content[0] . "-->" . $content[1] . "]]'.");
                         }
 
                         $externalTextnode = $this->textnodeRepository->find($content[1]);
 
                         if (is_null($externalTextnode)) {
-                            throw new Exception("There is a textnode named '".$twineName."' which references the external Dembelo Textnode '".$content[1]."', but a Dembelo Textnode with such an Id doesn't exist.");
+                            throw new Exception("There is a textnode named '" . $twineName . "' which references the external Dembelo Textnode '" . $content[1] . "', but a Dembelo Textnode with such an Id doesn't exist.");
                         }
 
                         $hitch = array();
@@ -441,11 +441,11 @@ 
                         $content = explode("->", $content, 2);
 
                         if (strlen($content[0]) <= 0 || strlen($content[1]) <= 0) {
-                            throw new Exception("The Twine archive file contains a '".$name."' with the invalid element '[[".$content[0]."->".$content[1]."]]'.");
+                            throw new Exception("The Twine archive file contains a '" . $name . "' with the invalid element '[[" . $content[0] . "->" . $content[1] . "]]'.");
                         }
 
                         if (array_key_exists($content[1], $this->nodeNameMapping) !== true) {
-                            throw new Exception("There is a textnode which references another textnode named '".$content[1]."', but this textnode doesn't exist within the same story.");
+                            throw new Exception("There is a textnode which references another textnode named '" . $content[1] . "', but this textnode doesn't exist within the same story.");
                         }
 
                         $hitch = array();
@@ -456,11 +456,11 @@ 
                         $content = explode("<-", $content, 2);
 
                         if (strlen($content[0]) <= 0 || strlen($content[1]) <= 0) {
-                            throw new Exception("The Twine archive file '".$this->importfile->getFilename()."' contains a '".$name."' with the invalid element '[[".$content[0]."<-".$content[1]."]]'.");
+                            throw new Exception("The Twine archive file '" . $this->importfile->getFilename() . "' contains a '" . $name . "' with the invalid element '[[" . $content[0] . "<-" . $content[1] . "]]'.");
                         }
 
                         if (array_key_exists($content[0], $this->nodeNameMapping) !== true) {
-                            throw new Exception("There is a textnode in the Twine archive file which references another textnode named '".$content[0]."', but this textnode doesn't exist within the same story.");
+                            throw new Exception("There is a textnode in the Twine archive file which references another textnode named '" . $content[0] . "', but this textnode doesn't exist within the same story.");
                         }
 
                         $hitch = array();
@@ -471,7 +471,7 @@ 
                         $content = explode(">:<", $content, 2);
 
                         if (strlen($content[0]) <= 0 || strlen($content[1]) <= 0) {
-                            throw new Exception("The Twine archive file contains a '".$name."' with the invalid element '[[".$content[0].">:<".$content[1]."]]'.");
+                            throw new Exception("The Twine archive file contains a '" . $name . "' with the invalid element '[[" . $content[0] . ">:<" . $content[1] . "]]'.");
                         }
 
                         $metadata = $textnode->getMetadata();
@@ -481,17 +481,17 @@ 
                         }
 
                         if (array_key_exists($content[0], $metadata) === true) {
-                            throw new Exception("There is a textnode in the Twine archive file which contains the metadata field '".$content[0]."' twice or would overwrite the already existing value of that field.");
+                            throw new Exception("There is a textnode in the Twine archive file which contains the metadata field '" . $content[0] . "' twice or would overwrite the already existing value of that field.");
                         }
 
                         $metadata[$content[0]] = $content[1];
                     } else {
                         if (strlen($content) <= 0) {
-                            throw new Exception("The Twine archive file contains a '".$name."' with the invalid element '[[".$content."]]'.");
+                            throw new Exception("The Twine archive file contains a '" . $name . "' with the invalid element '[[" . $content . "]]'.");
                         }
 
                         if (array_key_exists($content, $this->nodeNameMapping) !== true) {
-                            throw new Exception("There is a textnode in the Twine archive file which references another textnode named '".$content."', but this textnode doesn't exist within the same story.");
+                            throw new Exception("There is a textnode in the Twine archive file which references another textnode named '" . $content . "', but this textnode doesn't exist within the same story.");
                         }
 
                         $hitch = array();
@@ -502,11 +502,11 @@ 
 
                     if ($hitch !== null) {
                         if ($textnode->getHitchCount() >= Textnode::HITCHES_MAXIMUM_COUNT) {
-                            throw new Exception("There is a textnode named '".$twineName."' in the Twine archive file which has more than ".Textnode::HITCHES_MAXIMUM_COUNT." links.");
+                            throw new Exception("There is a textnode named '" . $twineName . "' in the Twine archive file which has more than " . Textnode::HITCHES_MAXIMUM_COUNT . " links.");
                         }
 
                         if ($textnode->appendHitch($hitch) !== true) {
-                            throw new Exception("Failed to append hitch for the textnode named '".$twineName."' from the Twine archive file.");
+                            throw new Exception("Failed to append hitch for the textnode named '" . $twineName . "' from the Twine archive file.");
                         }
                     }
 
@@ -560,7 +560,7 @@ 
         $this->accessSet = false;
     }
 
-    private function endElementPassageData()
+    private function endElementPassageData ()
     {
         $this->textnodeRepository->save($this->textnode);
 
@@ -579,7 +579,7 @@ 
      * @param string $name
      * @throws Exception
      */
-    private function endElement($parser, $name)
+    private function endElement ($parser, $name)
     {
         if ($name === "tw-storydata") {
             $this->endElementStoryData($name);

www/src/AdminBundle/Controller/DefaultController.php

Spacing +31 added lines, -31 removed lines

@@ -49,7 +49,7 @@ 
      *
      * @return string
      */
-    public function indexAction()
+    public function indexAction ()
     {
         $mainMenuData = [
             ['id' => "1", 'type' => "folder", 'value' => "Benutzer", 'css' => "folder_music"],
@@ -70,7 +70,7 @@ 
      * @param Request $request
      * @return String
      */
-    public function usersAction(Request $request)
+    public function usersAction (Request $request)
     {
         $repository = $this->get('app.model_repository_user');
 
@@ -86,7 +86,7 @@ 
                     //$value = $value === 'aktiv' ? 1 : 0;
                     $query->field($field)->equals((int) $value);
                 } else {
-                    $query->field($field)->equals(new \MongoRegex('/.*'.$value.'.*/i'));
+                    $query->field($field)->equals(new \MongoRegex('/.*' . $value . '.*/i'));
                 }
             }
         }
@@ -117,7 +117,7 @@ 
      *
      * @return String
      */
-    public function licenseesAction()
+    public function licenseesAction ()
     {
         $repository = $this->get('app.model_repository_licensee');
 
@@ -141,7 +141,7 @@ 
      * @param Request $request
      * @return String
      */
-    public function licenseeSuggestAction(Request $request)
+    public function licenseeSuggestAction (Request $request)
     {
         $filter = $request->query->get('filter');
 
@@ -151,7 +151,7 @@ 
         /* @var $repository \Doctrine\ODM\MongoDB\DocumentRepository */
         $repository = $mongo->getRepository('DembeloMain:Licensee');
 
-        $licensees = $repository->findBy(array('name' => new \MongoRegex('/'.$searchString.'/')), null, 10);
+        $licensees = $repository->findBy(array('name' => new \MongoRegex('/' . $searchString . '/')), null, 10);
 
         $output = array();
         /* @var $licensee \DembeloMain\Document\Licensee */
@@ -170,7 +170,7 @@ 
      *
      * @return String
      */
-    public function topicsAction()
+    public function topicsAction ()
     {
         $mongo = $this->get('doctrine_mongodb');
         /* @var $repository \Doctrine\ODM\MongoDB\DocumentRepository */
@@ -196,7 +196,7 @@ 
      * @param Request $request
      * @return String
      */
-    public function formsaveAction(Request $request)
+    public function formsaveAction (Request $request)
     {
         $params = $request->request->all();
 
@@ -209,7 +209,7 @@ 
         $formtype = $params['formtype'];
 
         /* @var $repository AbstractRepository */
-        $repository = $this->get('app.model_repository_'.$formtype);
+        $repository = $this->get('app.model_repository_' . $formtype);
 
         if (isset($params['id']) && $params['id'] == 'new') {
             $className = $repository->getClassName();
@@ -235,7 +235,7 @@ 
             } elseif ($param === 'imported' && $value === '') {
                 $value = null;
             }
-            $method = 'set'.ucfirst($param);
+            $method = 'set' . ucfirst($param);
             if (method_exists($item, $method)) {
                 $item->$method($value);
             }
@@ -270,7 +270,7 @@ 
      * @param Request $request
      * @return String
      */
-    public function useractivationmailAction(Request $request)
+    public function useractivationmailAction (Request $request)
     {
         $userId = $request->request->get('userId');
 
@@ -286,7 +286,7 @@ 
         if (null === $user) {
             return new Response(\json_encode(['error' => false]));
         }
-        $user->setActivationHash(sha1($user->getEmail().$user->getPassword().\time()));
+        $user->setActivationHash(sha1($user->getEmail() . $user->getPassword() . \time()));
 
         $dm->persist($user);
         $dm->flush();
@@ -313,7 +313,7 @@ 
      *
      * @return Response
      */
-    public function importfilesAction()
+    public function importfilesAction ()
     {
         $mongo = $this->get('doctrine_mongodb');
         /* @var $repository \Doctrine\ODM\MongoDB\DocumentRepository */
@@ -343,7 +343,7 @@ 
      *
      * @return Response
      */
-    public function uploadImportfileAction()
+    public function uploadImportfileAction ()
     {
         $output = array();
 
@@ -357,9 +357,9 @@ 
 
         $directory = $this->container->getParameter('twine_directory');
 
-        $filename = md5(uniqid().$file['name']);
+        $filename = md5(uniqid() . $file['name']);
 
-        move_uploaded_file($file["tmp_name"], $directory.$filename);
+        move_uploaded_file($file["tmp_name"], $directory . $filename);
 
         $output['filename'] = $filename;
         $output['orgname'] = $file['name'];
@@ -374,7 +374,7 @@ 
      *
      * @return Response
      */
-    public function textnodesAction()
+    public function textnodesAction ()
     {
         $repository = $this->get('app.model_repository_textNode');
         $textnodes = $repository->findAll();
@@ -393,7 +393,7 @@ 
             $obj->access = $textnode->getAccess() ? 'ja' : 'nein';
             $obj->licensee = $licenseeIndex[$textnode->getLicenseeId()];
             $obj->importfile = isset($importfileIndex[$textnode->getImportfileId()]) ? $importfileIndex[$textnode->getImportfileId()] : 'unbekannt';
-            $obj->beginning = substr(htmlentities(strip_tags($textnode->getText())), 0, 200)."...";
+            $obj->beginning = substr(htmlentities(strip_tags($textnode->getText())), 0, 200) . "...";
             $obj->financenode = $textnode->isFinanceNode() ? 'ja' : 'nein';
             $obj->twineId = $textnode->getTwineId();
             $obj->metadata = $this->formatMetadata($textnode->getMetadata());
@@ -410,7 +410,7 @@ 
      *
      * @return Response
      */
-    public function importAction(Request $request)
+    public function importAction (Request $request)
     {
         $importfileId = $request->get('importfileId');
 
@@ -449,26 +449,26 @@ 
      * @param string $filename filename hash
      * @param string $orgname original name
      */
-    private function saveFile(Importfile $item, $filename, $orgname)
+    private function saveFile (Importfile $item, $filename, $orgname)
     {
         if (empty($filename) || empty($orgname)) {
             return;
         }
 
         $directory = $this->container->getParameter('twine_directory');
-        $finalDirectory = $directory.$item->getLicenseeId().'/';
+        $finalDirectory = $directory . $item->getLicenseeId() . '/';
         if (!is_dir($finalDirectory)) {
             mkdir($finalDirectory);
         }
-        $finalName = $finalDirectory.$item->getId();
-        $file = $directory.$filename;
+        $finalName = $finalDirectory . $item->getId();
+        $file = $directory . $filename;
         rename($file, $finalName);
 
         $item->setOriginalname($orgname);
         $item->setFilename($finalName);
     }
 
-    private function buildLicenseeIndex()
+    private function buildLicenseeIndex ()
     {
         $repository = $this->get('app.model_repository_licensee');
         $licensees = $repository->findAll();
@@ -480,7 +480,7 @@ 
         return $index;
     }
 
-    private function buildImportfileIndex()
+    private function buildImportfileIndex ()
     {
         $repository = $this->get('app.model_repository_importfile');
         $importfiles = $repository->findAll();
@@ -499,28 +499,28 @@ 
      * @param string $filename filename hash
      * @param string $orgname original name
      */
-    private function saveTopicImage(Topic $item, $filename, $orgname)
+    private function saveTopicImage (Topic $item, $filename, $orgname)
     {
         if (empty($filename) || empty($orgname)) {
             return;
         }
         $directory = $this->container->getParameter('topic_image_directory');
-        $finalDirectory = $directory.$item->getId().'/';
+        $finalDirectory = $directory . $item->getId() . '/';
         if (!is_dir($finalDirectory)) {
             mkdir($finalDirectory);
         }
-        $finalName = $finalDirectory.$orgname;
-        $file = $directory.$filename;
+        $finalName = $finalDirectory . $orgname;
+        $file = $directory . $filename;
         rename($file, $finalName);
         $item->setOriginalImageName($orgname);
         $item->setImageFilename($finalName);
     }
 
-    private function formatMetadata(array $metadata)
+    private function formatMetadata (array $metadata)
     {
         $string = '';
         foreach ($metadata as $key => $value) {
-            $string .= $key.': '.$value."\n";
+            $string .= $key . ': ' . $value . "\n";
         }
 
         return $string;

www/src/DembeloMain/Model/FavoriteManager.php

Spacing +5 added lines, -5 removed lines

@@ -38,7 +38,7 @@ 
      * FavoriteManager constructor.
      * @param Session $session
      */
-    public function __construct(Session $session)
+    public function __construct (Session $session)
     {
         $this->session = $session;
     }
@@ -48,10 +48,10 @@ 
      * @param Textnode  $textnode
      * @param User|null $user
      */
-    public function setFavorite(Textnode $textnode, User $user = null): void
+    public function setFavorite (Textnode $textnode, User $user = null): void
     {
         if (null === $user) {
-            $this->session->set('favorite_'.$textnode->getTopicId(), $textnode->getArbitraryId());
+            $this->session->set('favorite_' . $textnode->getTopicId(), $textnode->getArbitraryId());
 
             return;
         }
@@ -64,10 +64,10 @@ 
      * @param User|null $user
      * @return string
      */
-    public function getFavorite(Topic $topic, User $user = null): string
+    public function getFavorite (Topic $topic, User $user = null): string
     {
         if (null === $user) {
-            return $this->session->get('favorite_'.$topic->getId());
+            return $this->session->get('favorite_' . $topic->getId());
         }
 
         return $user->getFavorite($topic->getId());

www/src/DembeloMain/Model/Repository/TopicRepositoryInterface.php

Spacing +6 added lines, -6 removed lines

@@ -34,13 +34,13 @@ 
      * @param string $id
      * @return Topic
      */
-    public function find($id);
+    public function find ($id);
 
     /**
      * Find all topics
      * @return Topic[]
      */
-    public function findAll();
+    public function findAll ();
 
     /**
      * @param array      $criteria
@@ -49,13 +49,13 @@ 
      * @param integer    $skip
      * @return Topic[]
      */
-    public function findBy(array $criteria, array $sort = null, $limit = null, $skip = null);
+    public function findBy (array $criteria, array $sort = null, $limit = null, $skip = null);
 
     /**
      * Find all active topics
      * @return Topic[]
      */
-    public function findByStatusActive();
+    public function findByStatusActive ();
 
     /**
      * finds filtered topics
@@ -63,12 +63,12 @@ 
      * @param array $orderBy
      * @return mixed
      */
-    public function findFiltered(array $filters = array(), array $orderBy = array());
+    public function findFiltered (array $filters = array(), array $orderBy = array());
 
     /**
      * Save topic
      * @param Topic $topic
      * @return Topic
      */
-    public function save($topic);
+    public function save ($topic);
 }

www/src/DembeloMain/Controller/DefaultController.php

Spacing +9 added lines, -9 removed lines

@@ -46,7 +46,7 @@ 
      * @throws \LogicException
      * @throws \Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException
      */
-    public function readTopicAction($topicId)
+    public function readTopicAction ($topicId)
     {
         if ($this->container->get('app.feature_toggle')->hasFeature('login_needed') && !$this->isGranted('ROLE_USER')) {
             return $this->redirectToRoute('login_route');
@@ -64,7 +64,7 @@ 
         $textnode = $textnodeRepository->getTextnodeToRead($topicId);
 
         if (is_null($textnode)) {
-            throw $this->createNotFoundException('No Textnode for Topic \''.$topicId.'\' found.');
+            throw $this->createNotFoundException('No Textnode for Topic \'' . $topicId . '\' found.');
         }
 
         if ($textnode->isFinanceNode()) {
@@ -81,7 +81,7 @@ 
      *
      * @return string
      */
-    public function readTextnodeAction($textnodeArbitraryId)
+    public function readTextnodeAction ($textnodeArbitraryId)
     {
         if ($this->container->get('app.feature_toggle')->hasFeature('login_needed') && !$this->isGranted('ROLE_USER')) {
             return $this->redirectToRoute('login_route');
@@ -91,7 +91,7 @@ 
         $textnode = $textnodeRepository->findOneActiveByArbitraryId($textnodeArbitraryId);
 
         if (is_null($textnode)) {
-            throw $this->createNotFoundException('No Textnode with arbitrary ID \''.$textnode->getArbitraryId().'\' found.');
+            throw $this->createNotFoundException('No Textnode with arbitrary ID \'' . $textnode->getArbitraryId() . '\' found.');
         }
 
         if ($textnode->isFinanceNode()) {
@@ -137,7 +137,7 @@ 
      *
      * @return string
      */
-    public function paywallAction($textnodeId, $hitchIndex)
+    public function paywallAction ($textnodeId, $hitchIndex)
     {
         $hitchedTextnode = $this->getTextnodeForHitchIndex($textnodeId, $hitchIndex);
 
@@ -158,19 +158,19 @@ 
      *
      * @return string
      */
-    public function imprintAction()
+    public function imprintAction ()
     {
         return $this->render('DembeloMain::default/imprint.html.twig');
     }
 
-    private function getTextnodeForHitchIndex($textnodeId, $hitchIndex)
+    private function getTextnodeForHitchIndex ($textnodeId, $hitchIndex)
     {
         $textnodeRepository = $this->get('app.model_repository_textNode');
 
         $textnode = $textnodeRepository->findOneActiveById($textnodeId);
 
         if (is_null($textnode)) {
-            throw $this->createNotFoundException('No Textnode with ID \''.$textnodeId.'\' found.');
+            throw $this->createNotFoundException('No Textnode with ID \'' . $textnodeId . '\' found.');
         }
 
         $hitch = $textnode->getHitch($hitchIndex);
@@ -178,7 +178,7 @@ 
         return $this->getTextnodeForTextnodeId($hitch['textnodeId']);
     }
 
-    private function getTextnodeForTextnodeId($textnodeId)
+    private function getTextnodeForTextnodeId ($textnodeId)
     {
         $textnodeRepository = $this->get('app.model_repository_textNode');
         $linkedTextnode = $textnodeRepository->findOneActiveById($textnodeId);

www/src/DembeloMain/Command/InstallCommand.php

Spacing +20 added lines, -20 removed lines

@@ -45,7 +45,7 @@ 
 
     private $dummyData = array();
 
-    protected function configure()
+    protected function configure ()
     {
         $this
             ->setName('dembelo:install')
@@ -64,7 +64,7 @@ 
             );
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute (InputInterface $input, OutputInterface $output)
     {
 
         if ($input->getOption('purge-db')) {
@@ -86,7 +86,7 @@ 
         //$dm->flush();
     }
 
-    protected function purgeDB()
+    protected function purgeDB ()
     {
         $collectionClasses = array(
             'DembeloMain\Document\Licensee',
@@ -105,13 +105,13 @@ 
         }
     }
 
-    protected function installDefaultUsers(OutputInterface $output)
+    protected function installDefaultUsers (OutputInterface $output)
     {
         $this->installAdminUser();
         $output->writeln('admin user installed');
     }
 
-    protected function installAdminUser()
+    protected function installAdminUser ()
     {
         $users = array(
             array(
@@ -129,7 +129,7 @@ 
         $this->installUsers($users);
     }
 
-    protected function installDummyData(OutputInterface $output)
+    protected function installDummyData (OutputInterface $output)
     {
 
         $mongo = $this->getContainer()->get('doctrine_mongodb');
@@ -152,7 +152,7 @@ 
         $output->writeln("Hitches installed...");
     }
 
-    private function createLicensees(ManagerRegistry $mongo, DocumentManager $dm)
+    private function createLicensees (ManagerRegistry $mongo, DocumentManager $dm)
     {
         $repository = $mongo->getRepository('DembeloMain:Licensee');
 
@@ -166,7 +166,7 @@ 
         foreach ($licensees as $licenseeData) {
             $licensee = $repository->findOneByName($licenseeData['name']);
 
-            if (null ===$licensee) {
+            if (null === $licensee) {
                 $licensee = new Licensee();
                 $licensee->setName($licenseeData['name']);
                 $dm->persist($licensee);
@@ -175,7 +175,7 @@ 
         }
     }
 
-    private function createUsers()
+    private function createUsers ()
     {
         $users = array(
             array(
@@ -203,7 +203,7 @@ 
         $this->installUsers($users);
     }
 
-    private function installUsers(array $users)
+    private function installUsers (array $users)
     {
         /* @var \DembeloMain\Model\Repository\UserRepositoryInterface */
         $userRepository = $this->getContainer()->get('app.model_repository_user');
@@ -240,7 +240,7 @@ 
         }
     }
 
-    private function createTopics(ManagerRegistry $mongo, DocumentManager $dm)
+    private function createTopics (ManagerRegistry $mongo, DocumentManager $dm)
     {
         $repository = $mongo->getRepository('DembeloMain:Topic');
 
@@ -257,14 +257,14 @@ 
             array('name' => 'Themenfeld 9', 'status' => Topic::STATUS_ACTIVE),
         );
 
-        $imagesSrcFolder = $this->getContainer()->getParameter('kernel.root_dir').'/../src/DembeloMain/Resources/public/images/';
+        $imagesSrcFolder = $this->getContainer()->getParameter('kernel.root_dir') . '/../src/DembeloMain/Resources/public/images/';
         $imagesTargetFolder = $this->getContainer()->getParameter('topic_image_directory');
 
         $sortKey = 1;
         foreach ($topicData as $topicDatum) {
             $topic = $repository->findOneByName($topicDatum['name']);
             if (is_null($topic)) {
-                $imagename = 'bg0'.$sortKey.'.jpg';
+                $imagename = 'bg0' . $sortKey . '.jpg';
                 $topic = new Topic();
                 $topic->setName($topicDatum['name']);
                 $topic->setStatus($topicDatum['status']);
@@ -272,16 +272,16 @@ 
                 $topic->setOriginalImageName($imagename);
                 $topic->setImageFilename($imagename);
                 $dm->persist($topic);
-                $topicFolder = $imagesTargetFolder.'/'.$topic->getId().'/';
+                $topicFolder = $imagesTargetFolder . '/' . $topic->getId() . '/';
                 mkdir($topicFolder);
-                copy($imagesSrcFolder.$imagename, $topicFolder.'/'.$imagename);
+                copy($imagesSrcFolder . $imagename, $topicFolder . '/' . $imagename);
                 $sortKey++;
             }
             $this->dummyData['topics'][] = $topic;
         }
     }
 
-    private function createTextnodes()
+    private function createTextnodes ()
     {
         $loremIpsumLength = 3500;
 
@@ -378,7 +378,7 @@ 
         }
     }
 
-    private function createHitches(DocumentManager $dm)
+    private function createHitches (DocumentManager $dm)
     {
         if (isset($this->dummyData['textnodes']) !== true) {
             return;
@@ -407,11 +407,11 @@ 
         $dm->persist($this->dummyData['textnodes'][0]);
     }
 
-    private function cleanImageDirectories()
+    private function cleanImageDirectories ()
     {
-        $topicImageDirectory = $this->getContainer()->getParameter('topic_image_directory').'/';
+        $topicImageDirectory = $this->getContainer()->getParameter('topic_image_directory') . '/';
         if (is_dir($topicImageDirectory)) {
-            shell_exec('rm -r '.$topicImageDirectory.'*');
+            shell_exec('rm -r ' . $topicImageDirectory . '*');
         }
     }
 }

www/src/DembeloMain/Document/Story.php

Spacing +8 added lines, -8 removed lines

@@ -61,7 +61,7 @@ 
     /**
      * @return mixed
      */
-    public function getId()
+    public function getId ()
     {
         return $this->id;
     }
@@ -69,7 +69,7 @@ 
     /**
      * @param string $id
      */
-    public function setId($id)
+    public function setId ($id)
     {
         $this->id = $id;
     }
@@ -77,7 +77,7 @@ 
     /**
      * @return string
      */
-    public function getName()
+    public function getName ()
     {
         return $this->name;
     }
@@ -85,7 +85,7 @@ 
     /**
      * @param string $name
      */
-    public function setName(string $name): void
+    public function setName (string $name): void
     {
         $this->name = $name;
     }
@@ -93,7 +93,7 @@ 
     /**
      * @return mixed
      */
-    public function getTopicId()
+    public function getTopicId ()
     {
         return $this->topicId;
     }
@@ -101,7 +101,7 @@ 
     /**
      * @param mixed $topicId
      */
-    public function setTopicId($topicId)
+    public function setTopicId ($topicId)
     {
         $this->topicId = $topicId;
     }
@@ -109,7 +109,7 @@ 
     /**
      * @return int
      */
-    public function getStatus(): int
+    public function getStatus (): int
     {
         return $this->status;
     }
@@ -117,7 +117,7 @@ 
     /**
      * @param int $status
      */
-    public function setStatus(int $status)
+    public function setStatus (int $status)
     {
         $this->status = $status;
     }

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;
 }
@@ -77,7 +77,7 @@ 
     /**
      * @inheritdoc
      */
-    protected function setUp(): void
+    protected function setUp (): void
     {
         self::bootKernel();
         $application = new Application(self::$kernel);
@@ -91,11 +91,11 @@ 
     /**
      * tests the execute method
      */
-    public function testExecute(): void
+    public function testExecute (): void
     {
         $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'
@@ -119,7 +119,7 @@ 
     /**
      * tests execute method with unreadable file
      */
-    public function testExecuteWithUnreadableFile(): void
+    public function testExecuteWithUnreadableFile (): void
     {
         $this->mockObjects['importTwine']->expects($this->never())
             ->method('run');
@@ -141,7 +141,7 @@ 
     /**
      * tests execute() method with non existing file
      */
-    public function testExecuteWithFileNotExisting(): void
+    public function testExecuteWithFileNotExisting (): void
     {
         $this->mockObjects['importTwine']->expects($this->never())
             ->method('run');
@@ -163,11 +163,11 @@ 
     /**
      * tests execute() method with exception thrown by importTwine
      */
-    public function testExecuteWithExeptionInImportTwine(): void
+    public function testExecuteWithExeptionInImportTwine (): void
     {
         $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'
@@ -195,7 +195,7 @@ 
     /**
      * @expectedException \Exception
      */
-    public function testExecuteWithLicenseeNotFound(): void
+    public function testExecuteWithLicenseeNotFound (): void
     {
         $this->mockObjects['importTwine']->expects($this->never())
             ->method('run');
@@ -221,7 +221,7 @@ 
      * @param string $arg
      * @return null|\PHPUnit_Framework_MockObject_MockObject
      */
-    public function findOneByNameCallback($arg): ?Licensee
+    public function findOneByNameCallback ($arg): ?Licensee
     {
         if ($arg === 'somelicensee') {
             $licenseeMock = $this->getMockBuilder('DembeloMain\Document\Licensee')->disableOriginalConstructor()->getMock();
@@ -238,7 +238,7 @@ 
     /**
      * @return array
      */
-    private function getMockObjects(): array
+    private function getMockObjects (): array
     {
         $container = $this->getMockBuilder('Symfony\Component\DependencyInjection\ContainerInterface')->getMock();
 

www/src/DembeloMain/Model/Repository/Doctrine/ODM/TextNodeRepository.php

Spacing +10 added lines, -10 removed lines

@@ -39,7 +39,7 @@ 
      * @param string $importfileId
      * @return Textnode[]
      */
-    public function findByImportfileId($importfileId)
+    public function findByImportfileId ($importfileId)
     {
         return $this->findBy(array('importfileId' => new MongoId($importfileId)));
     }
@@ -50,7 +50,7 @@ 
      * @param string     $twineId
      * @return Textnode
      */
-    public function findByTwineId(Importfile $importfile, $twineId)
+    public function findByTwineId (Importfile $importfile, $twineId)
     {
         $textnode = $this->findOneBy(
             array(
@@ -68,7 +68,7 @@ 
      * @param array $existingTextnodeIds array of textnodeIds
      * @throws \Doctrine\ODM\MongoDB\MongoDBException
      */
-    public function disableOrphanedNodes(Importfile $importfile, array $existingTextnodeIds)
+    public function disableOrphanedNodes (Importfile $importfile, array $existingTextnodeIds)
     {
         $this->getDocumentManager()->createQueryBuilder(Textnode::class)
             ->update()
@@ -84,7 +84,7 @@ 
      * @param string $arbitraryId textnode arbitrary id
      * @return Textnode|null
      */
-    public function findOneActiveByArbitraryId($arbitraryId): ?Textnode
+    public function findOneActiveByArbitraryId ($arbitraryId): ?Textnode
     {
         return $this->findOneBy(
             array(
@@ -98,7 +98,7 @@ 
      * @param string $id Textnode Id
      * @return Textnode
      */
-    public function findOneActiveById($id)
+    public function findOneActiveById ($id)
     {
         return $this->findOneBy(
             array(
@@ -113,7 +113,7 @@ 
      *
      * @param Textnode $textnode
      */
-    public function setHyphenatedText(Textnode $textnode)
+    public function setHyphenatedText (Textnode $textnode)
     {
         $hyphenator = new Hyphenator();
         $hyphenator->registerPatterns('de');
@@ -129,7 +129,7 @@ 
      *
      * @return Textnode|null
      */
-    public function getTextnodeToRead($topicId): ?Textnode
+    public function getTextnodeToRead ($topicId): ?Textnode
     {
         return $this->createQueryBuilder()
             ->field('topicId')->equals(new \MongoId($topicId))
@@ -141,7 +141,7 @@ 
     /**
      * @param Textnode $object
      */
-    protected function beforeSave($object)
+    protected function beforeSave ($object)
     {
         parent::beforeSave($object);
         if (null === $object->getArbitraryId()) {
@@ -154,9 +154,9 @@ 
      *
      * @return string
      */
-    private function createArbitraryId(Textnode $object): string
+    private function createArbitraryId (Textnode $object): string
     {
-        $id = substr(md5(time().$object->getTwineId().substr($object->getText(), 0, 100)), 0, 15);
+        $id = substr(md5(time() . $object->getTwineId() . substr($object->getText(), 0, 100)), 0, 15);
         $exists = count($this->findBy(array('arbitraryId' => $id))) > 0;
 
         if ($exists) {