Erikmitk / kitodo-presentation

Classes/Validation/AbstractDlfValidationStack.php

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
     {
         foreach ($configuration as $configurationItem) {
             if (!class_exists($configurationItem["className"])) {
-                $this->logger->error('Unable to load class ' . $configurationItem["className"] . '.');
+                $this->logger->error('Unable to load class '.$configurationItem["className"].'.');
                 throw new InvalidArgumentException('Unable to load validator class.', 1723200537037);
             }
             $breakOnError = !isset($configurationItem["breakOnError"]) || $configurationItem["breakOnError"] !== "false";
@@ -83,7 +83,7 @@ 
         }
 
         if (!$validator instanceof AbstractDlfValidator) {
-            $this->logger->error($className . ' must be an instance of AbstractDlfValidator.');
+            $this->logger->error($className.' must be an instance of AbstractDlfValidator.');
             throw new InvalidArgumentException('Class must be an instance of AbstractDlfValidator.', 1723121212747);
         }
 
@@ -104,7 +104,7 @@ 
     protected function isValid($value): void
     {
         if (!$value instanceof $this->valueClassName) {
-            $this->logger->error('Value must be an instance of ' . $this->valueClassName . '.');
+            $this->logger->error('Value must be an instance of '.$this->valueClassName.'.');
             throw new InvalidArgumentException('Type of value is not valid.', 1723127564821);
         }
 

Braces +2 added lines, -4 removed lines

@@ -26,8 +26,7 @@ 
  *
  * @access public
  */
-abstract class AbstractDlfValidationStack extends AbstractDlfValidator
-{
+abstract class AbstractDlfValidationStack extends AbstractDlfValidator {
     use LoggerAwareTrait;
 
     const ITEM_KEY_TITLE = "title";
@@ -36,8 +35,7 @@ 
 
     protected array $validatorStack = [];
 
-    public function __construct(string $valueClassName)
-    {
+    public function __construct(string $valueClassName) {
         parent::__construct($valueClassName);
     }
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -74,9 +74,9 @@
      *
      * @return void
      */
-    protected function addValidator(string $className, string $title, bool $breakOnError = true, array $configuration = null): void
+    protected function addValidator(string $className, string $title, bool $breakOnError = TRUE, array $configuration = NULL): void
     {
-        if ($configuration === null) {
+        if ($configuration === NULL) {
             $validator = GeneralUtility::makeInstance($className);
         } else {
             $validator = GeneralUtility::makeInstance($className, $configuration);

Classes/Validation/SaxonXslToSvrlValidator.php

Spacing +2 added lines, -2 removed lines

@@ -61,11 +61,11 @@
     protected function process(mixed $value): string
     {
         // using source from standard input
-        $process = new Process(['java', '-jar', $this->jar, '-xsl:' . $this->xsl, '-s:-'], null, null, $value->saveXML());
+        $process = new Process(['java', '-jar', $this->jar, '-xsl:'.$this->xsl, '-s:-'], null, null, $value->saveXML());
         $process->run();
         // executes after the command finish
         if (!$process->isSuccessful()) {
-            $this->logger->error('Processing exits with code "' . $process->getExitCode() . '"');
+            $this->logger->error('Processing exits with code "'.$process->getExitCode().'"');
             throw new InvalidArgumentException('Processing was not successful.', 1724862680);
         }
         return $process->getOutput();

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
     protected function process(mixed $value): string
     {
         // using source from standard input
-        $process = new Process(['java', '-jar', $this->jar, '-xsl:' . $this->xsl, '-s:-'], null, null, $value->saveXML());
+        $process = new Process(['java', '-jar', $this->jar, '-xsl:' . $this->xsl, '-s:-'], NULL, NULL, $value->saveXML());
         $process->run();
         // executes after the command finish
         if (!$process->isSuccessful()) {

Braces +2 added lines, -4 removed lines

@@ -30,14 +30,12 @@
  *
  * @access public
  */
-class SaxonXslToSvrlValidator extends AbstractDlfValidator implements LoggerAwareInterface
-{
+class SaxonXslToSvrlValidator extends AbstractDlfValidator implements LoggerAwareInterface {
     private string $jar;
 
     private string $xsl;
 
-    public function __construct(array $configuration)
-    {
+    public function __construct(array $configuration) {
         parent::__construct(DOMDocument::class);
         $this->jar = GeneralUtility::getFileAbsFileName($configuration["jar"] ?? '');
         $this->xsl = GeneralUtility::getFileAbsFileName($configuration["xsl"] ?? '');

Classes/Middleware/DOMDocumentValidation.php

Spacing +3 added lines, -3 removed lines

@@ -78,19 +78,19 @@
         $validation = GeneralUtility::makeInstance(DOMDocumentValidationStack::class, $settings['domDocumentValidationValidators']);
 
         if (!GeneralUtility::isValidUrl($urlParam)) {
-            $this->logger->debug('Parameter "' . $urlParam . '" is not a valid url.');
+            $this->logger->debug('Parameter "'.$urlParam.'" is not a valid url.');
             throw new InvalidArgumentException('Value of url parameter is not a valid url.', 1724852611);
         }
 
         $content = GeneralUtility::getUrl($urlParam);
         if ($content === false) {
-            $this->logger->debug('Error while loading content of "' . $urlParam . '"');
+            $this->logger->debug('Error while loading content of "'.$urlParam.'"');
             throw new InvalidArgumentException('Error while loading content of url.', 1724420640);
         }
 
         $document = new DOMDocument();
         if ($document->loadXML($content) === false) {
-            $this->logger->debug('Error converting content of "' . $urlParam . '" to xml.');
+            $this->logger->debug('Error converting content of "'.$urlParam.'" to xml.');
             throw new InvalidArgumentException('Error converting content to xml.', 1724420648);
         }
 

Braces +1 added lines, -2 removed lines

@@ -33,8 +33,7 @@
  * @subpackage dlf
  * @access public
  */
-class DOMDocumentValidation implements MiddlewareInterface
-{
+class DOMDocumentValidation implements MiddlewareInterface {
     use LoggerAwareTrait;
 
     /**

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -83,13 +83,13 @@
         }
 
         $content = GeneralUtility::getUrl($urlParam);
-        if ($content === false) {
+        if ($content === FALSE) {
             $this->logger->debug('Error while loading content of "' . $urlParam . '"');
             throw new InvalidArgumentException('Error while loading content of url.', 1724420640);
         }
 
         $document = new DOMDocument();
-        if ($document->loadXML($content) === false) {
+        if ($document->loadXML($content) === FALSE) {
             $this->logger->debug('Error converting content of "' . $urlParam . '" to xml.');
             throw new InvalidArgumentException('Error converting content to xml.', 1724420648);
         }

Tests/Unit/Validation/SaxonXslToSvrlValidatorTest.php

Braces +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
  *
  * @access public
  */
-class SaxonXslToSvrlValidatorTest extends UnitTestCase
-{
+class SaxonXslToSvrlValidatorTest extends UnitTestCase {
     const SVRL = <<<SVRL
         <svrl:schematron-output
             xmlns:svrl="http://purl.oclc.org/dsdl/schematron">

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
     {
         parent::setUp();
         $this->resetSingletonInstances = true;
-        $this->dlfExtensionPath = Environment::getExtensionsPath() . 'dlf';
+        $this->dlfExtensionPath = Environment::getExtensionsPath().'dlf';
     }
 
     public function testJarFileNotFound(): void
@@ -63,12 +63,12 @@ 
         $this->expectException(InvalidArgumentException::class);
         $this->expectExceptionMessage("XSL Schematron file not found.");
         // It only checks if a file exists at the specified path, so we can use one of the test files.
-        new SaxonXslToSvrlValidator(["jar" => $this->dlfExtensionPath . '/Tests/Fixtures/Format/alto.xml']);
+        new SaxonXslToSvrlValidator(["jar" => $this->dlfExtensionPath.'/Tests/Fixtures/Format/alto.xml']);
     }
 
     public function testValidation(): void
     {
-        $saxonXslToSvrlValidator = new SaxonXslToSvrlValidator(["jar" => $this->dlfExtensionPath . '/Tests/Fixtures/Format/alto.xml', "xsl" => $this->dlfExtensionPath . '/Tests/Fixtures/Format/alto.xml']);
+        $saxonXslToSvrlValidator = new SaxonXslToSvrlValidator(["jar" => $this->dlfExtensionPath.'/Tests/Fixtures/Format/alto.xml', "xsl" => $this->dlfExtensionPath.'/Tests/Fixtures/Format/alto.xml']);
         $reflection = new ReflectionClass($saxonXslToSvrlValidator);
 
         $result = $reflection->getProperty("result");

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
     public function setUp(): void
     {
         parent::setUp();
-        $this->resetSingletonInstances = true;
+        $this->resetSingletonInstances = TRUE;
         $this->dlfExtensionPath = Environment::getExtensionsPath() . 'dlf';
     }
 
@@ -72,11 +72,11 @@ 
         $reflection = new ReflectionClass($saxonXslToSvrlValidator);
 
         $result = $reflection->getProperty("result");
-        $result->setAccessible(true);
+        $result->setAccessible(TRUE);
         $result->setValue($saxonXslToSvrlValidator, new Result());
 
         $method = $reflection->getMethod("addErrorsOfSvrl");
-        $method->setAccessible(true);
+        $method->setAccessible(TRUE);
         $method->invoke($saxonXslToSvrlValidator, self::SVRL);
 
         self::assertTrue($result->getValue($saxonXslToSvrlValidator)->hasErrors());

Tests/Unit/Validation/DOMDocumentValidationStackTest.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
  *
  * @access public
  */
-class DOMDocumentValidationStackTest extends UnitTestCase
-{
+class DOMDocumentValidationStackTest extends UnitTestCase {
     public function setUp(): void
     {
         parent::setUp();

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function setUp(): void
     {
         parent::setUp();
-        $this->resetSingletonInstances = true;
+        $this->resetSingletonInstances = TRUE;
     }
 
     public function testValueTypeException(): void

Tests/Unit/Validation/XmlSchemesValidatorTest.php

Braces +1 added lines, -2 removed lines

@@ -26,8 +26,7 @@
  *
  * @access public
  */
-class XmlSchemesValidatorTest extends UnitTestCase
-{
+class XmlSchemesValidatorTest extends UnitTestCase {
     const METS = <<<METS
         <mets:mets
             xmlns:mets="http://www.loc.gov/METS/"

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
     public function setUp(): void
     {
         parent::setUp();
-        $this->resetSingletonInstances = true;
+        $this->resetSingletonInstances = TRUE;
     }
 
     public function testValidation(): void

Classes/Controller/AbstractController.php

Spacing +10 added lines, -10 removed lines

@@ -164,11 +164,11 @@ 
                 if ($doc !== null) {
                     $this->document->setCurrentDocument($doc);
                 } else {
-                    $this->logger->error('Failed to load document with record ID "' . $this->requestData['recordId'] . '"');
+                    $this->logger->error('Failed to load document with record ID "'.$this->requestData['recordId'].'"');
                 }
             }
         } else {
-            $this->logger->error('Invalid ID "' . $documentId . '" or PID "' . $this->settings['storagePid'] . '" for document loading');
+            $this->logger->error('Invalid ID "'.$documentId.'" or PID "'.$this->settings['storagePid'].'" for document loading');
         }
     }
 
@@ -265,7 +265,7 @@ 
             && !MathUtility::canBeInterpretedAsInteger($this->requestData['id'])
             && !GeneralUtility::isValidUrl($this->requestData['id'])
         ) {
-            $this->logger->warning('Invalid ID or URI "' . $this->requestData['id'] . '" for document loading');
+            $this->logger->warning('Invalid ID or URI "'.$this->requestData['id'].'" for document loading');
             unset($this->requestData['id']);
         }
 
@@ -325,7 +325,7 @@ 
     {
         if (!array_key_exists($setting, $this->settings) || empty($this->settings[$setting])) {
             $this->settings[$setting] = $value;
-            $this->logger->warning('Setting "' . $setting . '" not set, using default value "' . $value . '". Probably FlexForm for controller "' . get_class($this) . '" is not read.');
+            $this->logger->warning('Setting "'.$setting.'" not set, using default value "'.$value.'". Probably FlexForm for controller "'.get_class($this).'" is not read.');
         } else {
             $this->settings[$setting] = (int) $this->settings[$setting];
         }
@@ -410,7 +410,7 @@ 
         $pages = [];
         $pagesSect = [];
         $aRange = [];
-        $nRange = 5;    // ToDo: should be made configurable
+        $nRange = 5; // ToDo: should be made configurable
 
         // lower limit of the range
         $nBottom = $currentPageNumber - $nRange;
@@ -425,7 +425,7 @@ 
 
         // check whether the first screen page is > 1, if yes then points must be added
         if ($aRange[0] > 1) {
-            array_push($pagesSect, ['label' => '...','startRecordNumber' => '...']);
+            array_push($pagesSect, ['label' => '...', 'startRecordNumber' => '...']);
         };
         $lastStartRecordNumberGrid = 0; // due to validity outside the loop
         foreach (range($firstPage, $lastPage) as $i) {
@@ -468,7 +468,7 @@ 
 
                 // Check if screen page is in range
                 if (in_array($i, $aRange)) {
-                    array_push($pagesSect, ['label' => $i,'startRecordNumber' => $startRecordNumber]);
+                    array_push($pagesSect, ['label' => $i, 'startRecordNumber' => $startRecordNumber]);
                 };
             };
         };
@@ -534,7 +534,7 @@ 
             // fix for count(): Argument #1 ($value) must be of type Countable|array, null given
             $this->documentArray[] = $doc;
         } else {
-            $this->logger->error('Invalid UID "' . $documentId . '" or PID "' . $this->settings['storagePid'] . '" for document loading');
+            $this->logger->error('Invalid UID "'.$documentId.'" or PID "'.$this->settings['storagePid'].'" for document loading');
         }
 
         return $doc;
@@ -572,7 +572,7 @@ 
             foreach ($this->requestData['multipleSource'] as $location) {
                 $document = AbstractDocument::getInstance($location, $this->settings, true);
                 if ($document !== null) {
-                    $this->documentArray['extra_' . $i] = $document;
+                    $this->documentArray['extra_'.$i] = $document;
                 }
                 $i++;
             }
@@ -596,7 +596,7 @@ 
 
             $this->document->setLocation($documentId);
         } else {
-            $this->logger->error('Invalid location given "' . $documentId . '" for document loading');
+            $this->logger->error('Invalid location given "'.$documentId.'" for document loading');
         }
 
         return $doc;

Upper-Lower-Casing +19 added lines, -19 removed lines

@@ -62,7 +62,7 @@ 
      * @access protected
      * @var Document|null This holds the current document
      */
-    protected ?Document $document = null;
+    protected ?Document $document = NULL;
 
     /**
      * @access protected
@@ -143,7 +143,7 @@ 
         // Try to get document format from database
         if (!empty($documentId)) {
 
-            $doc = null;
+            $doc = NULL;
 
             if (MathUtility::canBeInterpretedAsInteger($documentId)) {
                 $doc = $this->getDocumentByUid($documentId);
@@ -151,7 +151,7 @@ 
                 $doc = $this->getDocumentByUrl($documentId);
             }
 
-            if ($this->document !== null && $doc !== null) {
+            if ($this->document !== NULL && $doc !== NULL) {
                 $this->document->setCurrentDocument($doc);
             }
 
@@ -159,9 +159,9 @@ 
 
             $this->document = $this->documentRepository->findOneByRecordId($this->requestData['recordId']);
 
-            if ($this->document !== null) {
-                $doc = AbstractDocument::getInstance($this->document->getLocation(), $this->settings, true);
-                if ($doc !== null) {
+            if ($this->document !== NULL) {
+                $doc = AbstractDocument::getInstance($this->document->getLocation(), $this->settings, TRUE);
+                if ($doc !== NULL) {
                     $this->document->setCurrentDocument($doc);
                 } else {
                     $this->logger->error('Failed to load document with record ID "' . $this->requestData['recordId'] . '"');
@@ -218,7 +218,7 @@ 
      */
     protected function isDocMissing(): bool
     {
-        return $this->document === null || $this->document->getCurrentDocument() === null;
+        return $this->document === NULL || $this->document->getCurrentDocument() === NULL;
     }
 
     /**
@@ -247,7 +247,7 @@ 
         if ($this->request->hasArgument($parameterName)) {
             return $this->request->getArgument($parameterName);
         }
-        return null;
+        return NULL;
     }
 
     /**
@@ -368,7 +368,7 @@ 
             if (isset($this->settings['multiViewType']) && $this->document->getCurrentDocument()->tableOfContents[0]['type'] === $this->settings['multiViewType']) {
                 $i = 0;
                 foreach ($this->documentArray as $document) {
-                    if ($document !== null) {
+                    if ($document !== NULL) {
                         $this->requestData['docPage'][$i] = MathUtility::forceIntegerInRange((int) $this->requestData['docPage'][$i], 1, $document->numPages, 1);
                         $i++;
                     }
@@ -479,8 +479,8 @@ 
         };
 
         // Safely get the next and previous page numbers
-        $nextPageNumber = isset($pages[$currentPageNumber + 1]) ? $pages[$currentPageNumber + 1]['startRecordNumber'] : null;
-        $previousPageNumber = isset($pages[$currentPageNumber - 1]) ? $pages[$currentPageNumber - 1]['startRecordNumber'] : null;
+        $nextPageNumber = isset($pages[$currentPageNumber + 1]) ? $pages[$currentPageNumber + 1]['startRecordNumber'] : NULL;
+        $previousPageNumber = isset($pages[$currentPageNumber - 1]) ? $pages[$currentPageNumber - 1]['startRecordNumber'] : NULL;
 
         // 'startRecordNumber' is not required in GridView, only the variant for each loop is required
         // 'endRecordNumber' is not required in both views
@@ -526,11 +526,11 @@ 
     private function getDocumentByUid(int $documentId)
     {
         // TODO: implement multiView as it is in getDocumentByUrl
-        $doc = null;
+        $doc = NULL;
         $this->document = $this->documentRepository->findOneByIdAndSettings($documentId);
 
         if ($this->document) {
-            $doc = AbstractDocument::getInstance($this->document->getLocation(), $this->settings, true);
+            $doc = AbstractDocument::getInstance($this->document->getLocation(), $this->settings, TRUE);
             // fix for count(): Argument #1 ($value) must be of type Countable|array, null given
             $this->documentArray[] = $doc;
         } else {
@@ -551,13 +551,13 @@ 
      */
     protected function getDocumentByUrl(string $documentId)
     {
-        $doc = AbstractDocument::getInstance($documentId, $this->settings, true);
+        $doc = AbstractDocument::getInstance($documentId, $this->settings, TRUE);
 
         if (isset($this->settings['multiViewType']) && $doc->tableOfContents[0]['type'] === $this->settings['multiViewType']) {
             $childDocuments = $doc->tableOfContents[0]['children'];
             $i = 0;
             foreach ($childDocuments as $document) {
-                $this->documentArray[] = AbstractDocument::getInstance($document['points'], $this->settings, true);
+                $this->documentArray[] = AbstractDocument::getInstance($document['points'], $this->settings, TRUE);
                 if (!isset($this->requestData['docPage'][$i]) && isset(explode('#', $document['points'])[1])) {
                     $initPage = explode('#', $document['points'])[1];
                     $this->requestData['docPage'][$i] = $initPage;
@@ -570,21 +570,21 @@ 
         if ($this->requestData['multipleSource'] && is_array($this->requestData['multipleSource'])) {
             $i = 0;
             foreach ($this->requestData['multipleSource'] as $location) {
-                $document = AbstractDocument::getInstance($location, $this->settings, true);
-                if ($document !== null) {
+                $document = AbstractDocument::getInstance($location, $this->settings, TRUE);
+                if ($document !== NULL) {
                     $this->documentArray['extra_' . $i] = $document;
                 }
                 $i++;
             }
         }
 
-        if ($doc !== null) {
+        if ($doc !== NULL) {
             $this->document = GeneralUtility::makeInstance(Document::class);
 
             if ($doc->recordId) {
                 // find document from repository by recordId
                 $docFromRepository = $this->documentRepository->findOneByRecordId($doc->recordId);
-                if ($docFromRepository !== null) {
+                if ($docFromRepository !== NULL) {
                     $this->document = $docFromRepository;
                 }
             }

Braces +8 added lines, -10 removed lines

@@ -39,8 +39,7 @@ 
  *
  * @abstract
  */
-abstract class AbstractController extends ActionController implements LoggerAwareInterface
-{
+abstract class AbstractController extends ActionController implements LoggerAwareInterface {
     use LoggerAwareTrait;
 
     /**
@@ -248,8 +247,7 @@ 
      *
      * @return null|string|array
      */
-    protected function getParametersSafely(string $parameterName)
-    {
+    protected function getParametersSafely(string $parameterName) {
         if ($this->request->hasArgument($parameterName)) {
             return $this->request->getArgument($parameterName);
         }
@@ -444,7 +442,8 @@ 
         $lastStartRecordNumberGrid = 0; // due to validity outside the loop
         foreach (range($firstPage, $lastPage) as $i) {
             // detect which pagination is active: ListView or GridView
-            if (get_class($pagination) == 'TYPO3\CMS\Core\Pagination\SimplePagination') {  // ListView
+            if (get_class($pagination) == 'TYPO3\CMS\Core\Pagination\SimplePagination') {
+// ListView
                 $lastStartRecordNumberGrid = $i; // save last $startRecordNumber for LastPage button
 
                 $pages[$i] = [
@@ -457,7 +456,8 @@ 
                 if (in_array($i, $aRange)) {
                     array_push($pagesSect, ['label' => $i, 'startRecordNumber' => $i]);
                 };
-            } else { // GridView
+            } else {
+// GridView
                 // to calculate the values for generation the links for the pagination pages
                 /** @var \Kitodo\Dlf\Pagination\PageGridPaginator $paginator */
                 $itemsPerPage = $paginator->getPublicItemsPerPage();
@@ -537,8 +537,7 @@ 
      *
      * @return AbstractDocument
      */
-    private function getDocumentByUid(int $documentId)
-    {
+    private function getDocumentByUid(int $documentId) {
         // TODO: implement multiView as it is in getDocumentByUrl
         $doc = null;
         $this->document = $this->documentRepository->findOneByIdAndSettings($documentId);
@@ -563,8 +562,7 @@ 
      *
      * @return AbstractDocument
      */
-    protected function getDocumentByUrl(string $documentId)
-    {
+    protected function getDocumentByUrl(string $documentId) {
         $doc = AbstractDocument::getInstance($documentId, $this->settings, true);
 
         if (isset($this->settings['multiViewType']) && $doc->tableOfContents[0]['type'] === $this->settings['multiViewType']) {

Classes/Validation/AbstractDlfValidator.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function validate($value): Result
     {
         if (!$value instanceof $this->valueClassName) {
-            $this->logger->debug('Value must be an instance of ' . $this->valueClassName . '.');
+            $this->logger->debug('Value must be an instance of '.$this->valueClassName.'.');
             throw new InvalidArgumentException('Type of value is not valid.', 1723126505626);
         }
         return parent::validate($value);

Braces +2 added lines, -4 removed lines

@@ -29,8 +29,7 @@ 
  *
  * @access public
  */
-abstract class AbstractDlfValidator extends AbstractValidator
-{
+abstract class AbstractDlfValidator extends AbstractValidator {
     use LoggerAwareTrait;
 
     protected string $valueClassName;
@@ -38,8 +37,7 @@ 
     /**
      * @param $valueClassName string The class name of the value
      */
-    public function __construct(string $valueClassName)
-    {
+    public function __construct(string $valueClassName) {
         $this->logger = GeneralUtility::makeInstance(LogManager::class)->getLogger(static::class);
         $this->valueClassName = $valueClassName;
     }

Classes/Controller/AnnotationController.php

Braces +1 added lines, -2 removed lines

@@ -22,8 +22,7 @@
  * @subpackage dlf
  * @access public
  */
-class AnnotationController extends AbstractController
-{
+class AnnotationController extends AbstractController {
     /**
      * The main method of the plugin
      *

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -34,8 +34,8 @@
         $this->loadDocument();
 
         if (
-            $this->document === null
-            || $this->document->getCurrentDocument() === null
+            $this->document === NULL
+            || $this->document->getCurrentDocument() === NULL
         ) {
             // Quit without doing anything if required variables are not set.
         } else {