fabarea / vidi

Classes/View/MenuItem/ExportCsvMenuItem.php

Indentation +16 added lines, -16 removed lines

@@ -17,20 +17,20 @@
  */
 class ExportCsvMenuItem extends AbstractComponentView
 {
-    /**
-     * Renders a "csv export" item to be placed in the menu.
-     * Only the admin is allowed to export for now as security is not handled.
-     *
-     * @return string
-     * @throws \InvalidArgumentException
-     */
-    public function render()
-    {
-        $result = sprintf(
-            '<li><a href="#" class="dropdown-item export-csv" data-format="csv">%s %s</a></li>',
-            $this->getIconFactory()->getIcon('mimetypes-text-csv', Icon::SIZE_SMALL),
-            $this->getLanguageService()->sL('LLL:EXT:vidi/Resources/Private/Language/locallang.xlf:export-csv')
-        );
-        return $result;
-    }
+	/**
+	 * Renders a "csv export" item to be placed in the menu.
+	 * Only the admin is allowed to export for now as security is not handled.
+	 *
+	 * @return string
+	 * @throws \InvalidArgumentException
+	 */
+	public function render()
+	{
+		$result = sprintf(
+			'<li><a href="#" class="dropdown-item export-csv" data-format="csv">%s %s</a></li>',
+			$this->getIconFactory()->getIcon('mimetypes-text-csv', Icon::SIZE_SMALL),
+			$this->getLanguageService()->sL('LLL:EXT:vidi/Resources/Private/Language/locallang.xlf:export-csv')
+		);
+		return $result;
+	}
 }

Classes/View/MenuItem/ClipboardMenuItem.php

Indentation +39 added lines, -39 removed lines

@@ -19,45 +19,45 @@
  */
 class ClipboardMenuItem extends AbstractComponentView
 {
-    /**
-     * Renders a "mass delete" menu item to be placed in the grid menu.
-     *
-     * @return string
-     */
-    public function render()
-    {
-        $output = '';
-        if ($this->getMediaModule()->hasFolderTree()) {
-            $output = sprintf(
-                '<li><a href="%s" class="clipboard-save" >%s %s</a>',
-                $this->getSaveInClipboardUri(),
-                $this->getIconFactory()->getIcon('actions-document-paste-after', Icon::SIZE_SMALL),
-                $this->getLanguageService()->sL('LLL:EXT:vidi/Resources/Private/Language/locallang.xlf:save')
-            );
-        }
-        return $output;
-    }
+	/**
+	 * Renders a "mass delete" menu item to be placed in the grid menu.
+	 *
+	 * @return string
+	 */
+	public function render()
+	{
+		$output = '';
+		if ($this->getMediaModule()->hasFolderTree()) {
+			$output = sprintf(
+				'<li><a href="%s" class="clipboard-save" >%s %s</a>',
+				$this->getSaveInClipboardUri(),
+				$this->getIconFactory()->getIcon('actions-document-paste-after', Icon::SIZE_SMALL),
+				$this->getLanguageService()->sL('LLL:EXT:vidi/Resources/Private/Language/locallang.xlf:save')
+			);
+		}
+		return $output;
+	}
 
-    /**
-     * @return string
-     */
-    protected function getSaveInClipboardUri()
-    {
-        $additionalParameters = array(
-            $this->getModuleLoader()->getParameterPrefix() => array(
-                'controller' => 'Clipboard',
-                'action' => 'save',
-                'format' => 'json',
-            ),
-        );
-        return $this->getModuleLoader()->getModuleUrl($additionalParameters);
-    }
+	/**
+	 * @return string
+	 */
+	protected function getSaveInClipboardUri()
+	{
+		$additionalParameters = array(
+			$this->getModuleLoader()->getParameterPrefix() => array(
+				'controller' => 'Clipboard',
+				'action' => 'save',
+				'format' => 'json',
+			),
+		);
+		return $this->getModuleLoader()->getModuleUrl($additionalParameters);
+	}
 
-    /**
-     * @return MediaModule|object
-     */
-    protected function getMediaModule()
-    {
-        return GeneralUtility::makeInstance(MediaModule::class);
-    }
+	/**
+	 * @return MediaModule|object
+	 */
+	protected function getMediaModule()
+	{
+		return GeneralUtility::makeInstance(MediaModule::class);
+	}
 }

Classes/Signal/AfterFindContentObjectsSignalArguments.php

Indentation +183 added lines, -183 removed lines

@@ -17,187 +17,187 @@
  */
 class AfterFindContentObjectsSignalArguments
 {
-    /**
-     * @var string
-     */
-    protected $dataType;
-
-    /**
-     * @var array
-     */
-    protected $contentObjects;
-
-    /**
-     * @var Matcher
-     */
-    protected $matcher;
-
-    /**
-     * @var Order
-     */
-    protected $order;
-
-    /**
-     * @var int
-     */
-    protected $limit;
-
-    /**
-     * @var int
-     */
-    protected $offset;
-
-    /**
-     * @var bool
-     */
-    protected $hasBeenProcessed;
-
-    /**
-     * @var int
-     */
-    protected $numberOfObjects = 0;
-
-    /**
-     * @param array $contentObjects
-     * @return $this
-     */
-    public function setContentObjects($contentObjects)
-    {
-        $this->contentObjects = $contentObjects;
-        return $this;
-    }
-
-    /**
-     * @return array
-     */
-    public function getContentObjects()
-    {
-        return $this->contentObjects;
-    }
-
-    /**
-     * @param string $dataType
-     * @return $this
-     */
-    public function setDataType($dataType)
-    {
-        $this->dataType = $dataType;
-        return $this;
-    }
-
-    /**
-     * @return string
-     */
-    public function getDataType()
-    {
-        return $this->dataType;
-    }
-
-    /**
-     * @param boolean $hasBeenProcessed
-     * @return $this
-     */
-    public function setHasBeenProcessed($hasBeenProcessed)
-    {
-        $this->hasBeenProcessed = $hasBeenProcessed;
-        return $this;
-    }
-
-    /**
-     * @return boolean
-     */
-    public function getHasBeenProcessed()
-    {
-        return $this->hasBeenProcessed;
-    }
-
-    /**
-     * @param int $limit
-     * @return $this
-     */
-    public function setLimit($limit)
-    {
-        $this->limit = $limit;
-        return $this;
-    }
-
-    /**
-     * @return int
-     */
-    public function getLimit()
-    {
-        return $this->limit;
-    }
-
-    /**
-     * @param Matcher $matcher
-     * @return $this
-     */
-    public function setMatcher($matcher)
-    {
-        $this->matcher = $matcher;
-        return $this;
-    }
-
-    /**
-     * @return Matcher
-     */
-    public function getMatcher()
-    {
-        return $this->matcher;
-    }
-
-    /**
-     * @return Order
-     */
-    public function getOrder()
-    {
-        return $this->order;
-    }
-
-    /**
-     * @param Order $order
-     * @return $this
-     */
-    public function setOrder($order)
-    {
-        $this->order = $order;
-        return $this;
-    }
-
-    /**
-     * @param int $numberOfObjects
-     * @return $this
-     */
-    public function setNumberOfObjects($numberOfObjects)
-    {
-        $this->numberOfObjects = $numberOfObjects;
-        return $this;
-    }
-
-    /**
-     * @return int
-     */
-    public function getNumberOfObjects()
-    {
-        return $this->numberOfObjects;
-    }
-
-    /**
-     * @param int $offset
-     * @return $this
-     */
-    public function setOffset($offset)
-    {
-        $this->offset = $offset;
-        return $this;
-    }
-
-    /**
-     * @return int
-     */
-    public function getOffset()
-    {
-        return $this->offset;
-    }
+	/**
+	 * @var string
+	 */
+	protected $dataType;
+
+	/**
+	 * @var array
+	 */
+	protected $contentObjects;
+
+	/**
+	 * @var Matcher
+	 */
+	protected $matcher;
+
+	/**
+	 * @var Order
+	 */
+	protected $order;
+
+	/**
+	 * @var int
+	 */
+	protected $limit;
+
+	/**
+	 * @var int
+	 */
+	protected $offset;
+
+	/**
+	 * @var bool
+	 */
+	protected $hasBeenProcessed;
+
+	/**
+	 * @var int
+	 */
+	protected $numberOfObjects = 0;
+
+	/**
+	 * @param array $contentObjects
+	 * @return $this
+	 */
+	public function setContentObjects($contentObjects)
+	{
+		$this->contentObjects = $contentObjects;
+		return $this;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getContentObjects()
+	{
+		return $this->contentObjects;
+	}
+
+	/**
+	 * @param string $dataType
+	 * @return $this
+	 */
+	public function setDataType($dataType)
+	{
+		$this->dataType = $dataType;
+		return $this;
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getDataType()
+	{
+		return $this->dataType;
+	}
+
+	/**
+	 * @param boolean $hasBeenProcessed
+	 * @return $this
+	 */
+	public function setHasBeenProcessed($hasBeenProcessed)
+	{
+		$this->hasBeenProcessed = $hasBeenProcessed;
+		return $this;
+	}
+
+	/**
+	 * @return boolean
+	 */
+	public function getHasBeenProcessed()
+	{
+		return $this->hasBeenProcessed;
+	}
+
+	/**
+	 * @param int $limit
+	 * @return $this
+	 */
+	public function setLimit($limit)
+	{
+		$this->limit = $limit;
+		return $this;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getLimit()
+	{
+		return $this->limit;
+	}
+
+	/**
+	 * @param Matcher $matcher
+	 * @return $this
+	 */
+	public function setMatcher($matcher)
+	{
+		$this->matcher = $matcher;
+		return $this;
+	}
+
+	/**
+	 * @return Matcher
+	 */
+	public function getMatcher()
+	{
+		return $this->matcher;
+	}
+
+	/**
+	 * @return Order
+	 */
+	public function getOrder()
+	{
+		return $this->order;
+	}
+
+	/**
+	 * @param Order $order
+	 * @return $this
+	 */
+	public function setOrder($order)
+	{
+		$this->order = $order;
+		return $this;
+	}
+
+	/**
+	 * @param int $numberOfObjects
+	 * @return $this
+	 */
+	public function setNumberOfObjects($numberOfObjects)
+	{
+		$this->numberOfObjects = $numberOfObjects;
+		return $this;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getNumberOfObjects()
+	{
+		return $this->numberOfObjects;
+	}
+
+	/**
+	 * @param int $offset
+	 * @return $this
+	 */
+	public function setOffset($offset)
+	{
+		$this->offset = $offset;
+		return $this;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getOffset()
+	{
+		return $this->offset;
+	}
 }

Classes/Signal/ProcessContentDataSignalArguments.php

Indentation +137 added lines, -137 removed lines

@@ -16,141 +16,141 @@
  */
 class ProcessContentDataSignalArguments
 {
-    /**
-     * @var Content
-     */
-    protected $contentObject;
-
-    /**
-     * @var array
-     */
-    protected $contentData;
-
-    /**
-     * @var string
-     */
-    protected $fieldNameAndPath;
-
-    /**
-     * @var int
-     */
-    protected $counter;
-
-    /**
-     * @var int
-     */
-    protected $savingBehavior;
-
-    /**
-     * @var int
-     */
-    protected $language;
-
-    /**
-     * @param array $contentData
-     * @return $this
-     */
-    public function setContentData($contentData)
-    {
-        $this->contentData = $contentData;
-        return $this;
-    }
-
-    /**
-     * @return array
-     */
-    public function getContentData()
-    {
-        return $this->contentData;
-    }
-
-    /**
-     * @param Content $contentObject
-     * @return $this
-     */
-    public function setContentObject($contentObject)
-    {
-        $this->contentObject = $contentObject;
-        return $this;
-    }
-
-    /**
-     * @return Content
-     */
-    public function getContentObject()
-    {
-        return $this->contentObject;
-    }
-
-    /**
-     * @param string $fieldNameAndPath
-     * @return $this
-     */
-    public function setFieldNameAndPath($fieldNameAndPath)
-    {
-        $this->fieldNameAndPath = $fieldNameAndPath;
-        return $this;
-    }
-
-    /**
-     * @return string
-     */
-    public function getFieldNameAndPath()
-    {
-        return $this->fieldNameAndPath;
-    }
-
-    /**
-     * @param int $counter
-     * @return $this
-     */
-    public function setCounter($counter)
-    {
-        $this->counter = $counter;
-        return $this;
-    }
-
-    /**
-     * @return int
-     */
-    public function getCounter()
-    {
-        return $this->counter;
-    }
-
-    /**
-     * @param int $savingBehavior
-     * @return $this
-     */
-    public function setSavingBehavior($savingBehavior)
-    {
-        $this->savingBehavior = $savingBehavior;
-        return $this;
-    }
-
-    /**
-     * @return int
-     */
-    public function getSavingBehavior()
-    {
-        return $this->savingBehavior;
-    }
-
-    /**
-     * @return int
-     */
-    public function getLanguage()
-    {
-        return $this->language;
-    }
-
-    /**
-     * @param int $language
-     * @return $this
-     */
-    public function setLanguage($language)
-    {
-        $this->language = $language;
-        return $this;
-    }
+	/**
+	 * @var Content
+	 */
+	protected $contentObject;
+
+	/**
+	 * @var array
+	 */
+	protected $contentData;
+
+	/**
+	 * @var string
+	 */
+	protected $fieldNameAndPath;
+
+	/**
+	 * @var int
+	 */
+	protected $counter;
+
+	/**
+	 * @var int
+	 */
+	protected $savingBehavior;
+
+	/**
+	 * @var int
+	 */
+	protected $language;
+
+	/**
+	 * @param array $contentData
+	 * @return $this
+	 */
+	public function setContentData($contentData)
+	{
+		$this->contentData = $contentData;
+		return $this;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getContentData()
+	{
+		return $this->contentData;
+	}
+
+	/**
+	 * @param Content $contentObject
+	 * @return $this
+	 */
+	public function setContentObject($contentObject)
+	{
+		$this->contentObject = $contentObject;
+		return $this;
+	}
+
+	/**
+	 * @return Content
+	 */
+	public function getContentObject()
+	{
+		return $this->contentObject;
+	}
+
+	/**
+	 * @param string $fieldNameAndPath
+	 * @return $this
+	 */
+	public function setFieldNameAndPath($fieldNameAndPath)
+	{
+		$this->fieldNameAndPath = $fieldNameAndPath;
+		return $this;
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getFieldNameAndPath()
+	{
+		return $this->fieldNameAndPath;
+	}
+
+	/**
+	 * @param int $counter
+	 * @return $this
+	 */
+	public function setCounter($counter)
+	{
+		$this->counter = $counter;
+		return $this;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getCounter()
+	{
+		return $this->counter;
+	}
+
+	/**
+	 * @param int $savingBehavior
+	 * @return $this
+	 */
+	public function setSavingBehavior($savingBehavior)
+	{
+		$this->savingBehavior = $savingBehavior;
+		return $this;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getSavingBehavior()
+	{
+		return $this->savingBehavior;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getLanguage()
+	{
+		return $this->language;
+	}
+
+	/**
+	 * @param int $language
+	 * @return $this
+	 */
+	public function setLanguage($language)
+	{
+		$this->language = $language;
+		return $this;
+	}
 }

Classes/Utility/BackendUtility.php

Indentation +102 added lines, -102 removed lines

@@ -19,114 +19,114 @@
  */
 class BackendUtility
 {
-    /*******************************************
+	/*******************************************
      *
      * SQL-related, selecting records, searching
      *
      *******************************************/
-    /**
-     * Returns the WHERE clause " AND NOT [tablename].[deleted-field]" if a deleted-field
-     * is configured in $GLOBALS['TCA'] for the tablename, $table
-     * This function should ALWAYS be called in the backend for selection on tables which
-     * are configured in $GLOBALS['TCA'] since it will ensure consistent selection of records,
-     * even if they are marked deleted (in which case the system must always treat them as non-existent!)
-     * In the frontend a function, ->enableFields(), is known to filter hidden-field, start- and endtime
-     * and fe_groups as well. But that is a job of the frontend, not the backend. If you need filtering
-     * on those fields as well in the backend you can use ->BEenableFields() though.
-     *
-     * @param string $table Table name present in $GLOBALS['TCA']
-     * @param string $withLogicalSeparator Table alias if any
-     * @return string WHERE clause for filtering out deleted records, eg " AND tablename.deleted=0
-     */
-    public static function deleteClause($table, $withLogicalSeparator = ' AND')
-    {
-        if (empty($GLOBALS['TCA'][$table]['ctrl']['delete'])) {
-            return '';
-        }
-        $expressionBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
-            ->getQueryBuilderForTable($table)
-            ->expr();
-        return $withLogicalSeparator . ' ' . $expressionBuilder->eq(
-            $table . '.' . $GLOBALS['TCA'][$table]['ctrl']['delete'],
-            0
-        );
-    }
-    /**
-     * Backend implementation of enableFields()
-     * Notice that "fe_groups" is not selected for - only disabled, starttime and endtime.
-     * Notice that deleted-fields are NOT filtered - you must ALSO call deleteClause in addition.
-     * $GLOBALS["SIM_ACCESS_TIME"] is used for date.
-     *
-     * @param string $table The table from which to return enableFields WHERE clause. Table name must have a 'ctrl' section in $GLOBALS['TCA'].
-     * @param bool $inv Means that the query will select all records NOT VISIBLE records (inverted selection)
-     * @return string WHERE clause part
-     */
-    public static function BEenableFields($table, $inv = false)
-    {
-        $ctrl = $GLOBALS['TCA'][$table]['ctrl'];
-        $expressionBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
-            ->getConnectionForTable($table)
-            ->getExpressionBuilder();
-        $query = $expressionBuilder->andX();
-        $invQuery = $expressionBuilder->orX();
+	/**
+	 * Returns the WHERE clause " AND NOT [tablename].[deleted-field]" if a deleted-field
+	 * is configured in $GLOBALS['TCA'] for the tablename, $table
+	 * This function should ALWAYS be called in the backend for selection on tables which
+	 * are configured in $GLOBALS['TCA'] since it will ensure consistent selection of records,
+	 * even if they are marked deleted (in which case the system must always treat them as non-existent!)
+	 * In the frontend a function, ->enableFields(), is known to filter hidden-field, start- and endtime
+	 * and fe_groups as well. But that is a job of the frontend, not the backend. If you need filtering
+	 * on those fields as well in the backend you can use ->BEenableFields() though.
+	 *
+	 * @param string $table Table name present in $GLOBALS['TCA']
+	 * @param string $withLogicalSeparator Table alias if any
+	 * @return string WHERE clause for filtering out deleted records, eg " AND tablename.deleted=0
+	 */
+	public static function deleteClause($table, $withLogicalSeparator = ' AND')
+	{
+		if (empty($GLOBALS['TCA'][$table]['ctrl']['delete'])) {
+			return '';
+		}
+		$expressionBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
+			->getQueryBuilderForTable($table)
+			->expr();
+		return $withLogicalSeparator . ' ' . $expressionBuilder->eq(
+			$table . '.' . $GLOBALS['TCA'][$table]['ctrl']['delete'],
+			0
+		);
+	}
+	/**
+	 * Backend implementation of enableFields()
+	 * Notice that "fe_groups" is not selected for - only disabled, starttime and endtime.
+	 * Notice that deleted-fields are NOT filtered - you must ALSO call deleteClause in addition.
+	 * $GLOBALS["SIM_ACCESS_TIME"] is used for date.
+	 *
+	 * @param string $table The table from which to return enableFields WHERE clause. Table name must have a 'ctrl' section in $GLOBALS['TCA'].
+	 * @param bool $inv Means that the query will select all records NOT VISIBLE records (inverted selection)
+	 * @return string WHERE clause part
+	 */
+	public static function BEenableFields($table, $inv = false)
+	{
+		$ctrl = $GLOBALS['TCA'][$table]['ctrl'];
+		$expressionBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
+			->getConnectionForTable($table)
+			->getExpressionBuilder();
+		$query = $expressionBuilder->andX();
+		$invQuery = $expressionBuilder->orX();
 
-        if (is_array($ctrl)) {
-            if (is_array($ctrl['enablecolumns'])) {
-                if ($ctrl['enablecolumns']['disabled'] ?? false) {
-                    $field = $table . '.' . $ctrl['enablecolumns']['disabled'];
-                    $query->add($expressionBuilder->eq($field, 0));
-                    $invQuery->add($expressionBuilder->neq($field, 0));
-                }
-                if ($ctrl['enablecolumns']['starttime'] ?? false) {
-                    $field = $table . '.' . $ctrl['enablecolumns']['starttime'];
-                    $query->add($expressionBuilder->lte($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp')));
-                    $invQuery->add(
-                        $expressionBuilder->andX(
-                            $expressionBuilder->neq($field, 0),
-                            $expressionBuilder->gt($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp'))
-                        )
-                    );
-                }
-                if ($ctrl['enablecolumns']['endtime'] ?? false) {
-                    $field = $table . '.' . $ctrl['enablecolumns']['endtime'];
-                    $query->add(
-                        $expressionBuilder->orX(
-                            $expressionBuilder->eq($field, 0),
-                            $expressionBuilder->gt($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp'))
-                        )
-                    );
-                    $invQuery->add(
-                        $expressionBuilder->andX(
-                            $expressionBuilder->neq($field, 0),
-                            $expressionBuilder->lte($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp'))
-                        )
-                    );
-                }
-            }
-        }
+		if (is_array($ctrl)) {
+			if (is_array($ctrl['enablecolumns'])) {
+				if ($ctrl['enablecolumns']['disabled'] ?? false) {
+					$field = $table . '.' . $ctrl['enablecolumns']['disabled'];
+					$query->add($expressionBuilder->eq($field, 0));
+					$invQuery->add($expressionBuilder->neq($field, 0));
+				}
+				if ($ctrl['enablecolumns']['starttime'] ?? false) {
+					$field = $table . '.' . $ctrl['enablecolumns']['starttime'];
+					$query->add($expressionBuilder->lte($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp')));
+					$invQuery->add(
+						$expressionBuilder->andX(
+							$expressionBuilder->neq($field, 0),
+							$expressionBuilder->gt($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp'))
+						)
+					);
+				}
+				if ($ctrl['enablecolumns']['endtime'] ?? false) {
+					$field = $table . '.' . $ctrl['enablecolumns']['endtime'];
+					$query->add(
+						$expressionBuilder->orX(
+							$expressionBuilder->eq($field, 0),
+							$expressionBuilder->gt($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp'))
+						)
+					);
+					$invQuery->add(
+						$expressionBuilder->andX(
+							$expressionBuilder->neq($field, 0),
+							$expressionBuilder->lte($field, (int)GeneralUtility::makeInstance(Context::class)->getPropertyFromAspect('date', 'timestamp'))
+						)
+					);
+				}
+			}
+		}
 
-        if ($query->count() === 0) {
-            return '';
-        }
+		if ($query->count() === 0) {
+			return '';
+		}
 
-        return ' AND ' . ($inv ? $invQuery : $query);
-    }
+		return ' AND ' . ($inv ? $invQuery : $query);
+	}
 
-    /**
-     * Returns the URL to a given module
-     *
-     * @param string $moduleName Name of the module
-     * @param array $urlParameters URL parameters that should be added as key value pairs
-     * @return string Calculated URL
-     */
-    public static function getModuleUrl($moduleName, $urlParameters = [])
-    {
-        $uriBuilder = GeneralUtility::makeInstance(UriBuilder::class);
-        try {
-            $uri = $uriBuilder->buildUriFromRoute($moduleName, $urlParameters);
-        } catch (RouteNotFoundException $e) {
-            $uri = $uriBuilder->buildUriFromRoutePath($moduleName, $urlParameters);
-        }
-        return (string)$uri;
-    }
+	/**
+	 * Returns the URL to a given module
+	 *
+	 * @param string $moduleName Name of the module
+	 * @param array $urlParameters URL parameters that should be added as key value pairs
+	 * @return string Calculated URL
+	 */
+	public static function getModuleUrl($moduleName, $urlParameters = [])
+	{
+		$uriBuilder = GeneralUtility::makeInstance(UriBuilder::class);
+		try {
+			$uri = $uriBuilder->buildUriFromRoute($moduleName, $urlParameters);
+		} catch (RouteNotFoundException $e) {
+			$uri = $uriBuilder->buildUriFromRoutePath($moduleName, $urlParameters);
+		}
+		return (string)$uri;
+	}
 }

ext_localconf.php

Indentation +28 added lines, -28 removed lines

@@ -8,32 +8,32 @@
 defined('TYPO3') or die();
 
 call_user_func(function () {
-    $configuration = GeneralUtility::makeInstance(
-        ExtensionConfiguration::class
-    )->get('vidi');
-
-    if (false === isset($configuration['autoload_typoscript']) || true === (bool)$configuration['autoload_typoscript']) {
-        ExtensionManagementUtility::addTypoScript(
-            'vidi',
-            'constants',
-            '<INCLUDE_TYPOSCRIPT: source="FILE:EXT:vidi/Configuration/TypoScript/constants.txt">'
-        );
-
-        ExtensionManagementUtility::addTypoScript(
-            'vidi',
-            'setup',
-            '<INCLUDE_TYPOSCRIPT: source="FILE:EXT:vidi/Configuration/TypoScript/setup.txt">'
-        );
-    }
-
-    // Initialize generic Vidi modules after the TCA is loaded.
-    $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['GLOBAL']['extTablesInclusion-PostProcessing'][] = 'Fab\Vidi\Configuration\VidiModulesAspect';
-
-    // Initialize generic grid TCA for all data types
-    $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['GLOBAL']['extTablesInclusion-PostProcessing'][] = 'Fab\Vidi\Configuration\TcaGridAspect';
-
-    // cache configuration, see https://docs.typo3.org/typo3cms/CoreApiReference/ApiOverview/CachingFramework/Configuration/Index.html#cache-configurations
-    $GLOBALS['TYPO3_CONF_VARS']['SYS']['caching']['cacheConfigurations']['vidi']['frontend'] = VariableFrontend::class;
-    $GLOBALS['TYPO3_CONF_VARS']['SYS']['caching']['cacheConfigurations']['vidi']['groups'] = array('all', 'vidi');
-    $GLOBALS['TYPO3_CONF_VARS']['SYS']['caching']['cacheConfigurations']['vidi']['options']['defaultLifetime'] = 2592000;
+	$configuration = GeneralUtility::makeInstance(
+		ExtensionConfiguration::class
+	)->get('vidi');
+
+	if (false === isset($configuration['autoload_typoscript']) || true === (bool)$configuration['autoload_typoscript']) {
+		ExtensionManagementUtility::addTypoScript(
+			'vidi',
+			'constants',
+			'<INCLUDE_TYPOSCRIPT: source="FILE:EXT:vidi/Configuration/TypoScript/constants.txt">'
+		);
+
+		ExtensionManagementUtility::addTypoScript(
+			'vidi',
+			'setup',
+			'<INCLUDE_TYPOSCRIPT: source="FILE:EXT:vidi/Configuration/TypoScript/setup.txt">'
+		);
+	}
+
+	// Initialize generic Vidi modules after the TCA is loaded.
+	$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['GLOBAL']['extTablesInclusion-PostProcessing'][] = 'Fab\Vidi\Configuration\VidiModulesAspect';
+
+	// Initialize generic grid TCA for all data types
+	$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['GLOBAL']['extTablesInclusion-PostProcessing'][] = 'Fab\Vidi\Configuration\TcaGridAspect';
+
+	// cache configuration, see https://docs.typo3.org/typo3cms/CoreApiReference/ApiOverview/CachingFramework/Configuration/Index.html#cache-configurations
+	$GLOBALS['TYPO3_CONF_VARS']['SYS']['caching']['cacheConfigurations']['vidi']['frontend'] = VariableFrontend::class;
+	$GLOBALS['TYPO3_CONF_VARS']['SYS']['caching']['cacheConfigurations']['vidi']['groups'] = array('all', 'vidi');
+	$GLOBALS['TYPO3_CONF_VARS']['SYS']['caching']['cacheConfigurations']['vidi']['options']['defaultLifetime'] = 2592000;
 });

Tests/Functional/Domain/Model/ContentTest.php

Indentation +104 added lines, -104 removed lines

@@ -24,118 +24,118 @@
  */
 class ContentTest extends AbstractFunctionalTestCase
 {
-    /**
-     * @var Content
-     */
-    private $fixture;
+	/**
+	 * @var Content
+	 */
+	private $fixture;
 
-    /**
-     * @var string
-     */
-    private $dataType = 'tx_foo_domain_model_bar';
+	/**
+	 * @var string
+	 */
+	private $dataType = 'tx_foo_domain_model_bar';
 
-    public function setUp()
-    {
-        parent::setUp();
-        $GLOBALS['TCA'][$this->dataType]['columns'] = array(
-            'foo' => [],
-            'foo_bar' => [],
-        );
-        $this->fixture = new Content($this->dataType);
-    }
+	public function setUp()
+	{
+		parent::setUp();
+		$GLOBALS['TCA'][$this->dataType]['columns'] = array(
+			'foo' => [],
+			'foo_bar' => [],
+		);
+		$this->fixture = new Content($this->dataType);
+	}
 
-    public function tearDown()
-    {
-        unset($this->fixture, $GLOBALS['TCA'][$this->dataType]);
-    }
+	public function tearDown()
+	{
+		unset($this->fixture, $GLOBALS['TCA'][$this->dataType]);
+	}
 
-    /**
-     * @test
-     * @dataProvider fieldNameProvider
-     */
-    public function fieldNameIsConvertedToPropertyName($fieldName, $propertyName)
-    {
-        $data = array(
-            $fieldName => 'foo data',
-        );
-        $object = new Content($this->dataType, $data);
-        $this->assertObjectHasAttribute($propertyName, $object);
-    }
+	/**
+	 * @test
+	 * @dataProvider fieldNameProvider
+	 */
+	public function fieldNameIsConvertedToPropertyName($fieldName, $propertyName)
+	{
+		$data = array(
+			$fieldName => 'foo data',
+		);
+		$object = new Content($this->dataType, $data);
+		$this->assertObjectHasAttribute($propertyName, $object);
+	}
 
-    /**
-     * @test
-     * @dataProvider fieldNameProvider
-     */
-    public function accessValueOfArrayObjectReturnsFooDataAsString($fieldName)
-    {
-        $data = array(
-            $fieldName => 'foo data',
-        );
-        $this->markTestIncomplete(); # TCA must be faked
-        #$object = new \Fab\Vidi\Domain\Model\Content($this->dataType, $data);
-        #$this->assertSame($data[$fieldName], $object[$fieldName]);
-    }
+	/**
+	 * @test
+	 * @dataProvider fieldNameProvider
+	 */
+	public function accessValueOfArrayObjectReturnsFooDataAsString($fieldName)
+	{
+		$data = array(
+			$fieldName => 'foo data',
+		);
+		$this->markTestIncomplete(); # TCA must be faked
+		#$object = new \Fab\Vidi\Domain\Model\Content($this->dataType, $data);
+		#$this->assertSame($data[$fieldName], $object[$fieldName]);
+	}
 
-    /**
-     * @test
-     * @dataProvider fieldNameProvider
-     */
-    public function getValueThroughGetterReturnsFooDataAsString($fieldName, $propertyName)
-    {
-        $data = array(
-            $fieldName => 'foo data',
-        );
-        $this->markTestIncomplete(); # TCA must be faked
-        #$object = new \Fab\Vidi\Domain\Model\Content($this->dataType, $data);
-        #$getter = 'get' . ucfirst($propertyName);
-        #$this->assertSame($data[$fieldName], $object->$getter());
-    }
+	/**
+	 * @test
+	 * @dataProvider fieldNameProvider
+	 */
+	public function getValueThroughGetterReturnsFooDataAsString($fieldName, $propertyName)
+	{
+		$data = array(
+			$fieldName => 'foo data',
+		);
+		$this->markTestIncomplete(); # TCA must be faked
+		#$object = new \Fab\Vidi\Domain\Model\Content($this->dataType, $data);
+		#$getter = 'get' . ucfirst($propertyName);
+		#$this->assertSame($data[$fieldName], $object->$getter());
+	}
 
-    /**
-     * @test
-     * @dataProvider fieldNameProvider
-     */
-    public function toArrayMethodContainsGivenFieldName($fieldName)
-    {
-        $data = array(
-            $fieldName => 'foo data',
-        );
-        $object = new Content($this->dataType, $data);
-        $array = $object->toArray();
-        $this->assertArrayHasKey($fieldName, $array);
-    }
+	/**
+	 * @test
+	 * @dataProvider fieldNameProvider
+	 */
+	public function toArrayMethodContainsGivenFieldName($fieldName)
+	{
+		$data = array(
+			$fieldName => 'foo data',
+		);
+		$object = new Content($this->dataType, $data);
+		$array = $object->toArray();
+		$this->assertArrayHasKey($fieldName, $array);
+	}
 
-    /**
-     * Provider
-     */
-    public function fieldNameProvider()
-    {
-        return array(
-            array('foo', 'foo'),
-            array('foo_bar', 'fooBar'),
-        );
-    }
+	/**
+	 * Provider
+	 */
+	public function fieldNameProvider()
+	{
+		return array(
+			array('foo', 'foo'),
+			array('foo_bar', 'fooBar'),
+		);
+	}
 
-    /**
-     * @test
-     * @dataProvider propertyProvider
-     */
-    public function testProperty($propertyName, $value)
-    {
-        $setter = 'set' . ucfirst($propertyName);
-        $getter = 'get' . ucfirst($propertyName);
-        $this->markTestIncomplete(); # TCA must be faked
-        #call_user_func_array(array($this->fixture, $setter), array($value));
-        #$this->assertEquals($value, call_user_func(array($this->fixture, $getter)));
-    }
+	/**
+	 * @test
+	 * @dataProvider propertyProvider
+	 */
+	public function testProperty($propertyName, $value)
+	{
+		$setter = 'set' . ucfirst($propertyName);
+		$getter = 'get' . ucfirst($propertyName);
+		$this->markTestIncomplete(); # TCA must be faked
+		#call_user_func_array(array($this->fixture, $setter), array($value));
+		#$this->assertEquals($value, call_user_func(array($this->fixture, $getter)));
+	}
 
-    /**
-     * Provider
-     */
-    public function propertyProvider()
-    {
-        return array(
-            array('username', 'foo'),
-        );
-    }
+	/**
+	 * Provider
+	 */
+	public function propertyProvider()
+	{
+		return array(
+			array('username', 'foo'),
+		);
+	}
 }

Tests/Functional/AbstractFunctionalTestCase.php

Indentation +6 added lines, -6 removed lines

@@ -9,12 +9,12 @@
  */
 abstract class AbstractFunctionalTestCase extends FunctionalTestCase
 {
-    protected $testExtensionsToLoad = array('typo3conf/ext/vidi');
+	protected $testExtensionsToLoad = array('typo3conf/ext/vidi');
 
-    protected $coreExtensionsToLoad = array('extbase', 'fluid', 'scheduler');
+	protected $coreExtensionsToLoad = array('extbase', 'fluid', 'scheduler');
 
-    public function setUp(): void
-    {
-        parent::setUp();
-    }
+	public function setUp(): void
+	{
+		parent::setUp();
+	}
 }

Tests/Functional/Module/ModuleLoaderTest.php

Indentation +97 added lines, -97 removed lines

@@ -24,101 +24,101 @@
  */
 class ModuleLoaderTest extends AbstractFunctionalTestCase
 {
-    /**
-     * @var ModuleLoader
-     */
-    private $fixture;
-
-    /**
-     * @var string
-     */
-    private $dataType = 'tx_foo';
-
-    /**
-     * @var string
-     */
-    private $moduleCode = 'user_VidiTxFooM1';
-
-
-    public function setUp()
-    {
-        parent::setUp();
-        $this->fixture = new ModuleLoader($this->dataType);
-        $this->fixture->register();
-    }
-
-    public function tearDown()
-    {
-        unset($this->fixture);
-    }
-
-    /**
-     * @test
-     * @dataProvider attributeValueProvider
-     */
-    public function attributeCanBeSet($attribute, $value)
-    {
-        $setter = 'set' . ucfirst($attribute);
-        $this->fixture->$setter($value);
-        $this->assertAttributeEquals($value, $attribute, $this->fixture);
-    }
-
-    /**
-     * Provider
-     */
-    public function attributeValueProvider()
-    {
-        return array(
-            array('icon', 'bar'),
-            array('moduleLanguageFile', 'bar'),
-        );
-    }
-
-    /**
-     * @test
-     * @dataProvider attributeProvider
-     */
-    public function testAttribute($attribute, $defaultValue)
-    {
-        $this->assertAttributeEquals($defaultValue, $attribute, $this->fixture);
-    }
-
-    /**
-     * Provider
-     */
-    public function attributeProvider()
-    {
-        return array(
-            array('dataType', $this->dataType),
-            array('moduleKey', 'm1'),
-            array('icon', 'EXT:vidi/ext_icon.gif')
-        );
-    }
-
-    /**
-     * @test
-     */
-    public function getModuleConfigurationReturnsArrayWithSomeKeys()
-    {
-        $moduleLoader = new ModuleLoader($this->dataType);
-        $moduleLoader->register();
-        $GLOBALS['_GET']['M'] = $this->moduleCode;
-
-        $moduleConfiguration = $moduleLoader->getModuleConfiguration();
-        $keys = array('dataType', 'additionalJavaScriptFiles', 'additionalStyleSheetFiles');
-        foreach ($keys as $key) {
-            $this->assertArrayHasKey($key, $moduleConfiguration);
-        }
-    }
-
-    /**
-     * @test
-     */
-    public function getModuleConfigurationWithParameterDataTypeReturnsDataType()
-    {
-        $moduleLoader = new ModuleLoader($this->dataType);
-        $moduleLoader->register();
-        $GLOBALS['_GET']['M'] = $this->moduleCode;
-        $this->assertEquals($this->dataType, $moduleLoader->getModuleConfiguration('dataType'));
-    }
+	/**
+	 * @var ModuleLoader
+	 */
+	private $fixture;
+
+	/**
+	 * @var string
+	 */
+	private $dataType = 'tx_foo';
+
+	/**
+	 * @var string
+	 */
+	private $moduleCode = 'user_VidiTxFooM1';
+
+
+	public function setUp()
+	{
+		parent::setUp();
+		$this->fixture = new ModuleLoader($this->dataType);
+		$this->fixture->register();
+	}
+
+	public function tearDown()
+	{
+		unset($this->fixture);
+	}
+
+	/**
+	 * @test
+	 * @dataProvider attributeValueProvider
+	 */
+	public function attributeCanBeSet($attribute, $value)
+	{
+		$setter = 'set' . ucfirst($attribute);
+		$this->fixture->$setter($value);
+		$this->assertAttributeEquals($value, $attribute, $this->fixture);
+	}
+
+	/**
+	 * Provider
+	 */
+	public function attributeValueProvider()
+	{
+		return array(
+			array('icon', 'bar'),
+			array('moduleLanguageFile', 'bar'),
+		);
+	}
+
+	/**
+	 * @test
+	 * @dataProvider attributeProvider
+	 */
+	public function testAttribute($attribute, $defaultValue)
+	{
+		$this->assertAttributeEquals($defaultValue, $attribute, $this->fixture);
+	}
+
+	/**
+	 * Provider
+	 */
+	public function attributeProvider()
+	{
+		return array(
+			array('dataType', $this->dataType),
+			array('moduleKey', 'm1'),
+			array('icon', 'EXT:vidi/ext_icon.gif')
+		);
+	}
+
+	/**
+	 * @test
+	 */
+	public function getModuleConfigurationReturnsArrayWithSomeKeys()
+	{
+		$moduleLoader = new ModuleLoader($this->dataType);
+		$moduleLoader->register();
+		$GLOBALS['_GET']['M'] = $this->moduleCode;
+
+		$moduleConfiguration = $moduleLoader->getModuleConfiguration();
+		$keys = array('dataType', 'additionalJavaScriptFiles', 'additionalStyleSheetFiles');
+		foreach ($keys as $key) {
+			$this->assertArrayHasKey($key, $moduleConfiguration);
+		}
+	}
+
+	/**
+	 * @test
+	 */
+	public function getModuleConfigurationWithParameterDataTypeReturnsDataType()
+	{
+		$moduleLoader = new ModuleLoader($this->dataType);
+		$moduleLoader->register();
+		$GLOBALS['_GET']['M'] = $this->moduleCode;
+		$this->assertEquals($this->dataType, $moduleLoader->getModuleConfiguration('dataType'));
+	}
 }