lochmueller / ink

Classes/Rendering/Menu.php

Doc Comments +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 	/**
 	 * Render the given element
 	 *
-	 * @return array
+	 * @return string
 	 */
 	public function renderInternal() {
 		$str = $this->contentObject->cObjGetSingle($this->configuration['menu'], $this->configuration['menu.']);

Indentation +42 added lines, -42 removed lines

@@ -16,52 +16,52 @@
  */
 class Menu extends AbstractRendering {
 
-	/**
-	 * Render the given element
-	 *
-	 * @return array
-	 */
-	public function renderInternal() {
-		$str = $this->contentObject->cObjGetSingle($this->configuration['menu'], $this->configuration['menu.']);
-		$str = $this->breakBulletList($this->contentObject, trim(strip_tags(preg_replace('/<br\s*\/?>/i', chr(10), $this->parseBody($str)))));
-		return $str;
-	}
+    /**
+     * Render the given element
+     *
+     * @return array
+     */
+    public function renderInternal() {
+        $str = $this->contentObject->cObjGetSingle($this->configuration['menu'], $this->configuration['menu.']);
+        $str = $this->breakBulletList($this->contentObject, trim(strip_tags(preg_replace('/<br\s*\/?>/i', chr(10), $this->parseBody($str)))));
+        return $str;
+    }
 
-	/**
-	 * Breaks content lines into a bullet list
-	 *
-	 * @param ContentObjectRenderer $contentObject
-	 * @param string                $str Content string to make into a bullet list
-	 *
-	 * @return string Processed value
-	 */
-	function breakBulletList($contentObject, $str) {
-		$type = $contentObject->data['layout'];
-		$type = MathUtility::forceIntegerInRange($type, 0, 3);
+    /**
+     * Breaks content lines into a bullet list
+     *
+     * @param ContentObjectRenderer $contentObject
+     * @param string                $str Content string to make into a bullet list
+     *
+     * @return string Processed value
+     */
+    function breakBulletList($contentObject, $str) {
+        $type = $contentObject->data['layout'];
+        $type = MathUtility::forceIntegerInRange($type, 0, 3);
 
-		$tConf = $this->configuration['bulletlist.'][$type . '.'];
+        $tConf = $this->configuration['bulletlist.'][$type . '.'];
 
-		$cParts = explode(chr(10), $str);
-		$lines = array();
-		$c = 0;
+        $cParts = explode(chr(10), $str);
+        $lines = array();
+        $c = 0;
 
-		foreach ($cParts as $substrs) {
-			if (!strlen($substrs)) {
-				continue;
-			}
-			$c++;
-			$bullet = $tConf['bullet'] ? $tConf['bullet'] : ' - ';
-			$bLen = strlen($bullet);
-			$bullet = substr(str_replace('#', $c, $bullet), 0, $bLen);
-			$secondRow = substr($tConf['secondRow'] ? $tConf['secondRow'] : str_pad('', strlen($bullet), ' '), 0, $bLen);
+        foreach ($cParts as $substrs) {
+            if (!strlen($substrs)) {
+                continue;
+            }
+            $c++;
+            $bullet = $tConf['bullet'] ? $tConf['bullet'] : ' - ';
+            $bLen = strlen($bullet);
+            $bullet = substr(str_replace('#', $c, $bullet), 0, $bLen);
+            $secondRow = substr($tConf['secondRow'] ? $tConf['secondRow'] : str_pad('', strlen($bullet), ' '), 0, $bLen);
 
-			$lines[] = $bullet . $this->breakLines($substrs, chr(10) . $secondRow, Configuration::getPlainTextWith() - $bLen);
+            $lines[] = $bullet . $this->breakLines($substrs, chr(10) . $secondRow, Configuration::getPlainTextWith() - $bLen);
 
-			$blanks = MathUtility::forceIntegerInRange($tConf['blanks'], 0, 1000);
-			if ($blanks) {
-				$lines[] = str_pad('', $blanks - 1, chr(10));
-			}
-		}
-		return implode(chr(10), $lines);
-	}
+            $blanks = MathUtility::forceIntegerInRange($tConf['blanks'], 0, 1000);
+            if ($blanks) {
+                $lines[] = str_pad('', $blanks - 1, chr(10));
+            }
+        }
+        return implode(chr(10), $lines);
+    }
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
 		$type = $contentObject->data['layout'];
 		$type = MathUtility::forceIntegerInRange($type, 0, 3);
 
-		$tConf = $this->configuration['bulletlist.'][$type . '.'];
+		$tConf = $this->configuration['bulletlist.'][$type.'.'];
 
 		$cParts = explode(chr(10), $str);
 		$lines = array();
@@ -55,7 +55,7 @@ 
 			$bullet = substr(str_replace('#', $c, $bullet), 0, $bLen);
 			$secondRow = substr($tConf['secondRow'] ? $tConf['secondRow'] : str_pad('', strlen($bullet), ' '), 0, $bLen);
 
-			$lines[] = $bullet . $this->breakLines($substrs, chr(10) . $secondRow, Configuration::getPlainTextWith() - $bLen);
+			$lines[] = $bullet.$this->breakLines($substrs, chr(10).$secondRow, Configuration::getPlainTextWith() - $bLen);
 
 			$blanks = MathUtility::forceIntegerInRange($tConf['blanks'], 0, 1000);
 			if ($blanks) {

Classes/Rendering/Text.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 	/**
 	 * Render the current content
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function renderInternal() {
 

Indentation +9 added lines, -9 removed lines

@@ -12,15 +12,15 @@
  */
 class Text extends AbstractRendering {
 
-	/**
-	 * Render the current content
-	 *
-	 * @return array
-	 */
-	public function renderInternal() {
+    /**
+     * Render the current content
+     *
+     * @return array
+     */
+    public function renderInternal() {
 
-		$lines[] = trim($this->breakContent(strip_tags($this->parseBody($this->contentObject->data['bodytext']))), CRLF . TAB);
-		return $lines;
+        $lines[] = trim($this->breakContent(strip_tags($this->parseBody($this->contentObject->data['bodytext']))), CRLF . TAB);
+        return $lines;
 
-	}
+    }
 }
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 	 */
 	public function renderInternal() {
 
-		$lines[] = trim($this->breakContent(strip_tags($this->parseBody($this->contentObject->data['bodytext']))), CRLF . TAB);
+		$lines[] = trim($this->breakContent(strip_tags($this->parseBody($this->contentObject->data['bodytext']))), CRLF.TAB);
 		return $lines;
 
 	}

Classes/Configuration.php

Indentation +24 added lines, -24 removed lines

@@ -12,31 +12,31 @@
  */
 class Configuration {
 
-	/**
-	 * get the plaintext width
-	 *
-	 * @return int
-	 */
-	static public function getPlainTextWith() {
-		return isset($GLOBALS['TSFE']->config['config']['plainTextWith']) ? (int)$GLOBALS['TSFE']->config['config']['plainTextWith'] : 76;
-	}
+    /**
+     * get the plaintext width
+     *
+     * @return int
+     */
+    static public function getPlainTextWith() {
+        return isset($GLOBALS['TSFE']->config['config']['plainTextWith']) ? (int)$GLOBALS['TSFE']->config['config']['plainTextWith'] : 76;
+    }
 
-	/**
-	 * get the table mode
-	 *
-	 * @return string
-	 */
-	static public function getTableMode() {
-		return isset($GLOBALS['TSFE']->config['config']['tableMode']) && trim($GLOBALS['TSFE']->config['config']['tableMode']) !== '' ? trim($GLOBALS['TSFE']->config['config']['tableMode']) : 'default';
-	}
+    /**
+     * get the table mode
+     *
+     * @return string
+     */
+    static public function getTableMode() {
+        return isset($GLOBALS['TSFE']->config['config']['tableMode']) && trim($GLOBALS['TSFE']->config['config']['tableMode']) !== '' ? trim($GLOBALS['TSFE']->config['config']['tableMode']) : 'default';
+    }
 
-	/**
-	 * check if the plain tables are 100 displayed
-	 *
-	 * @return string
-	 */
-	static public function isPlainTable100() {
-		return isset($GLOBALS['TSFE']->config['config']['plainTable100']) && trim($GLOBALS['TSFE']->config['config']['plainTable100']) !== '' ? (bool)$GLOBALS['TSFE']->config['config']['plainTable100'] : TRUE;
-	}
+    /**
+     * check if the plain tables are 100 displayed
+     *
+     * @return string
+     */
+    static public function isPlainTable100() {
+        return isset($GLOBALS['TSFE']->config['config']['plainTable100']) && trim($GLOBALS['TSFE']->config['config']['plainTable100']) !== '' ? (bool)$GLOBALS['TSFE']->config['config']['plainTable100'] : TRUE;
+    }
 
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 	 * @return int
 	 */
 	static public function getPlainTextWith() {
-		return isset($GLOBALS['TSFE']->config['config']['plainTextWith']) ? (int)$GLOBALS['TSFE']->config['config']['plainTextWith'] : 76;
+		return isset($GLOBALS['TSFE']->config['config']['plainTextWith']) ? (int) $GLOBALS['TSFE']->config['config']['plainTextWith'] : 76;
 	}
 
 	/**
@@ -36,7 +36,7 @@ 
 	 * @return string
 	 */
 	static public function isPlainTable100() {
-		return isset($GLOBALS['TSFE']->config['config']['plainTable100']) && trim($GLOBALS['TSFE']->config['config']['plainTable100']) !== '' ? (bool)$GLOBALS['TSFE']->config['config']['plainTable100'] : TRUE;
+		return isset($GLOBALS['TSFE']->config['config']['plainTable100']) && trim($GLOBALS['TSFE']->config['config']['plainTable100']) !== '' ? (bool) $GLOBALS['TSFE']->config['config']['plainTable100'] : TRUE;
 	}
 
 }
\ No newline at end of file

Classes/Postprocessing/RemoveJavaScript.php

Indentation +11 added lines, -11 removed lines

@@ -12,15 +12,15 @@
  */
 class RemoveJavaScript implements PostprocessingInterface {
 
-	/**
-	 * Run the replacement
-	 *
-	 * @param string $content
-	 *
-	 * @return string
-	 * @todo remove also onchange etc.
-	 */
-	public function process($content) {
-		return preg_replace('#<script(.*?)>(.*?)</script>#is', '', $content);
-	}
+    /**
+     * Run the replacement
+     *
+     * @param string $content
+     *
+     * @return string
+     * @todo remove also onchange etc.
+     */
+    public function process($content) {
+        return preg_replace('#<script(.*?)>(.*?)</script>#is', '', $content);
+    }
 }

Classes/Postprocessing/RemoveGenerator.php

Indentation +10 added lines, -10 removed lines

@@ -12,14 +12,14 @@
  */
 class RemoveGenerator implements PostprocessingInterface {
 
-	/**
-	 * Remove the generator tag
-	 *
-	 * @param string $content
-	 *
-	 * @return string
-	 */
-	public function process($content) {
-		return preg_replace('/<meta name="?generator"?.+?>/is', '', $content);
-	}
+    /**
+     * Remove the generator tag
+     *
+     * @param string $content
+     *
+     * @return string
+     */
+    public function process($content) {
+        return preg_replace('/<meta name="?generator"?.+?>/is', '', $content);
+    }
 }

Classes/Postprocessing/RemoveHtmlComments.php

Indentation +10 added lines, -10 removed lines

@@ -12,14 +12,14 @@
  */
 class RemoveHtmlComments implements PostprocessingInterface {
 
-	/**
-	 * Run the replacement of the comments
-	 *
-	 * @param string $content
-	 *
-	 * @return string
-	 */
-	public function process($content) {
-		return preg_replace('/<!--(.*?)-->/s', '', $content);
-	}
+    /**
+     * Run the replacement of the comments
+     *
+     * @param string $content
+     *
+     * @return string
+     */
+    public function process($content) {
+        return preg_replace('/<!--(.*?)-->/s', '', $content);
+    }
 }

Classes/Postprocessing/RemoveMultipleEmptyLines.php

Indentation +10 added lines, -10 removed lines

@@ -12,14 +12,14 @@
  */
 class RemoveMultipleEmptyLines implements PostprocessingInterface {
 
-	/**
-	 * Run the cleanup of the empty lines
-	 *
-	 * @param string $content
-	 *
-	 * @return string
-	 */
-	public function process($content) {
-		return preg_replace('/\n{4,}/', "\n\n\n", $content);
-	}
+    /**
+     * Run the cleanup of the empty lines
+     *
+     * @param string $content
+     *
+     * @return string
+     */
+    public function process($content) {
+        return preg_replace('/\n{4,}/', "\n\n\n", $content);
+    }
 }

Classes/Postprocessing/InlineCss.php

Indentation +42 added lines, -42 removed lines

@@ -18,49 +18,49 @@
  */
 class InlineCss implements PostprocessingInterface {
 
-	/**
-	 * Call the inliner CSS processor
-	 *
-	 * @param string $content
-	 *
-	 * @return string
-	 */
-	public function process($content) {
-		GeneralUtility::requireOnce(ExtensionManagementUtility::extPath('ink', 'Resources/Private/Php/vendor/autoload.php'));
-		$pattern = '%<(link|style)(?=[^<>]*?(?:type="(text/css)"|>))(?=[^<>]*?(?:media="([^<>"]*)"|>))(?=[^<>]*?(?:href="(.*?)"|>))(?=[^<>]*(?:rel="([^<>"]*)"|>))(?:.*?</\1>|[^<>]*>)%si';
-		$matches = array();
-		$css = '';
-		preg_match_all($pattern, $content, $matches);
-		if (isset($matches[0])) {
-			foreach ($matches[0] as $key => $match) {
-				if ($matches[1][$key] === 'style') {
-					$css .= strip_tags($match);
-				} elseif (strpos($match, 'type="text/css"') !== FALSE) {
+    /**
+     * Call the inliner CSS processor
+     *
+     * @param string $content
+     *
+     * @return string
+     */
+    public function process($content) {
+        GeneralUtility::requireOnce(ExtensionManagementUtility::extPath('ink', 'Resources/Private/Php/vendor/autoload.php'));
+        $pattern = '%<(link|style)(?=[^<>]*?(?:type="(text/css)"|>))(?=[^<>]*?(?:media="([^<>"]*)"|>))(?=[^<>]*?(?:href="(.*?)"|>))(?=[^<>]*(?:rel="([^<>"]*)"|>))(?:.*?</\1>|[^<>]*>)%si';
+        $matches = array();
+        $css = '';
+        preg_match_all($pattern, $content, $matches);
+        if (isset($matches[0])) {
+            foreach ($matches[0] as $key => $match) {
+                if ($matches[1][$key] === 'style') {
+                    $css .= strip_tags($match);
+                } elseif (strpos($match, 'type="text/css"') !== FALSE) {
 
-					$file = preg_replace('/^(.+)\.(\d+)\.css$/', '$1.css', $matches[4][$key]);
-					$parts = parse_url($file);
-					if (isset($parts['query'])) {
-						unset($parts['query']);
-					}
-					if (!isset($parts['host'])) {
-						$parts['path'] = ltrim($parts['path'], '/');
-					}
+                    $file = preg_replace('/^(.+)\.(\d+)\.css$/', '$1.css', $matches[4][$key]);
+                    $parts = parse_url($file);
+                    if (isset($parts['query'])) {
+                        unset($parts['query']);
+                    }
+                    if (!isset($parts['host'])) {
+                        $parts['path'] = ltrim($parts['path'], '/');
+                    }
 
-					if($parts['host'] === GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY')) {
-						unset($parts['scheme']);
-						unset($parts['host']);
-						$parts['path'] = ltrim($parts['path'], '/');
-					}
+                    if($parts['host'] === GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY')) {
+                        unset($parts['scheme']);
+                        unset($parts['host']);
+                        $parts['path'] = ltrim($parts['path'], '/');
+                    }
 
-					$file = HttpUtility::buildUrl($parts);
-					$css .= GeneralUtility::getUrl($file);
-				} else {
-					continue;
-				}
-				$content = str_replace($match, '', $content);
-			}
-		}
-		$format = new CssToInlineStyles($content, $css);
-		return $format->convert();
-	}
+                    $file = HttpUtility::buildUrl($parts);
+                    $css .= GeneralUtility::getUrl($file);
+                } else {
+                    continue;
+                }
+                $content = str_replace($match, '', $content);
+            }
+        }
+        $format = new CssToInlineStyles($content, $css);
+        return $format->convert();
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 						$parts['path'] = ltrim($parts['path'], '/');
 					}
 
-					if($parts['host'] === GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY')) {
+					if ($parts['host'] === GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY')) {
 						unset($parts['scheme']);
 						unset($parts['host']);
 						$parts['path'] = ltrim($parts['path'], '/');

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use TYPO3\CMS\Core\Utility\ExtensionManagementUtility;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Core\Utility\HttpUtility;
-use TYPO3\CMS\Extbase\Utility\DebuggerUtility;
 
 /**
  * Inliner for CSS

Classes/Postprocessing/PostprocessingInterface.php

Indentation +8 added lines, -8 removed lines

@@ -11,13 +11,13 @@
  */
 interface PostprocessingInterface {
 
-	/**
-	 * Run the process
-	 *
-	 * @param string $content
-	 *
-	 * @return string
-	 */
-	public function process($content);
+    /**
+     * Run the process
+     *
+     * @param string $content
+     *
+     * @return string
+     */
+    public function process($content);
 
 }
\ No newline at end of file