silverstripe / cwp-core

src/Extension/RichLinksExtension.php

Indentation +38 added lines, -38 removed lines

@@ -17,48 +17,48 @@
 class RichLinksExtension extends Extension
 {
 
-    /**
-     * @var array
-     */
-    private static $casting = [
-        'RichLinks' => 'HTMLText'
-    ];
+	/**
+	 * @var array
+	 */
+	private static $casting = [
+		'RichLinks' => 'HTMLText'
+	];
 
-    /**
-     * @return string
-     */
-    public function RichLinks()
-    {
-        // Note:
-        // Assume we can use Regexes because the link will always be formatted
-        // in the same way coming from the CMS.
+	/**
+	 * @return string
+	 */
+	public function RichLinks()
+	{
+		// Note:
+		// Assume we can use Regexes because the link will always be formatted
+		// in the same way coming from the CMS.
 
-        $content = $this->owner->value;
+		$content = $this->owner->value;
 
-        // Find all file links for processing.
-        preg_match_all('/<a.*href="\[file_link,id=([0-9]+)\].*".*>.*<\/a>/U', $content, $matches);
+		// Find all file links for processing.
+		preg_match_all('/<a.*href="\[file_link,id=([0-9]+)\].*".*>.*<\/a>/U', $content, $matches);
 
-        // Attach the file type and size to each of the links.
-        for ($i = 0; $i < count($matches[0]); $i++) {
-            $file = DataObject::get_by_id(File::class, $matches[1][$i]);
-            if ($file) {
-                $size = $file->getSize();
-                $ext = strtoupper($file->getExtension());
-                // Replace the closing </a> tag with the size span (and reattach the closing tag).
-                $newLink = substr($matches[0][$i], 0, strlen($matches[0][$i]) - 4)
-                    . "<span class='fileExt'> [$ext, $size]</span></a>";
-                $content = str_replace($matches[0][$i], $newLink, $content);
-            }
-        }
+		// Attach the file type and size to each of the links.
+		for ($i = 0; $i < count($matches[0]); $i++) {
+			$file = DataObject::get_by_id(File::class, $matches[1][$i]);
+			if ($file) {
+				$size = $file->getSize();
+				$ext = strtoupper($file->getExtension());
+				// Replace the closing </a> tag with the size span (and reattach the closing tag).
+				$newLink = substr($matches[0][$i], 0, strlen($matches[0][$i]) - 4)
+					. "<span class='fileExt'> [$ext, $size]</span></a>";
+				$content = str_replace($matches[0][$i], $newLink, $content);
+			}
+		}
 
-        // Inject extra attributes into the external links.
-        $pattern = '/(<a.*)(href=\"https?:\/\/[^\"]*\"[^>]*>.*)(<\/a>)/iU';
-        $replacement = sprintf(
-            '$1class="external" rel="external" $2<span class="nonvisual-indicator">(%s)</span>$3',
-            _t(__CLASS__ . '.ExternalLink', 'external link')
-        );
-        $content = preg_replace($pattern, $replacement, $content, -1);
+		// Inject extra attributes into the external links.
+		$pattern = '/(<a.*)(href=\"https?:\/\/[^\"]*\"[^>]*>.*)(<\/a>)/iU';
+		$replacement = sprintf(
+			'$1class="external" rel="external" $2<span class="nonvisual-indicator">(%s)</span>$3',
+			_t(__CLASS__ . '.ExternalLink', 'external link')
+		);
+		$content = preg_replace($pattern, $replacement, $content, -1);
 
-        return $content;
-    }
+		return $content;
+	}
 }

tests/RichLinksExtensionTest.php

Indentation +28 added lines, -28 removed lines

@@ -7,35 +7,35 @@
 
 class RichLinksExtensionTest extends SapphireTest
 {
-    public function testContentLinkInjections()
-    {
-        $field = new DBText();
+	public function testContentLinkInjections()
+	{
+		$field = new DBText();
 
-        // External links injection.
-        $field->setValue('<a href="http://newzealand.govt.nz">New Zealand Government</a>');
-        $this->assertEquals(
-            $field->RichLinks(),
-            '<a class="external" rel="external" '
-                . 'href="http://newzealand.govt.nz">New Zealand Government'
-                . '<span class="nonvisual-indicator">(external link)</span></a>',
-            'Injects attributes to external link without target.'
-        );
+		// External links injection.
+		$field->setValue('<a href="http://newzealand.govt.nz">New Zealand Government</a>');
+		$this->assertEquals(
+			$field->RichLinks(),
+			'<a class="external" rel="external" '
+				. 'href="http://newzealand.govt.nz">New Zealand Government'
+				. '<span class="nonvisual-indicator">(external link)</span></a>',
+			'Injects attributes to external link without target.'
+		);
 
-        $field->setValue('<a href="http://newzealand.govt.nz" target="_blank">New Zealand Government</a>');
-        $this->assertEquals(
-            $field->RichLinks(),
-            '<a class="external" rel="external" '
-                . 'href="http://newzealand.govt.nz" target="_blank">New Zealand Government'
-                . '<span class="nonvisual-indicator">(external link)</span></a>',
-            'Injects attributes to external link with target, while keeping the existing attributes.'
-        );
+		$field->setValue('<a href="http://newzealand.govt.nz" target="_blank">New Zealand Government</a>');
+		$this->assertEquals(
+			$field->RichLinks(),
+			'<a class="external" rel="external" '
+				. 'href="http://newzealand.govt.nz" target="_blank">New Zealand Government'
+				. '<span class="nonvisual-indicator">(external link)</span></a>',
+			'Injects attributes to external link with target, while keeping the existing attributes.'
+		);
 
-        // Check the normal links are not affected.
-        $field->setValue('<a href="[sitetree_link,id=1]">Internal</a>');
-        $this->assertEquals(
-            $field->RichLinks(),
-            '<a href="[sitetree_link,id=1]">Internal</a>',
-            'Regular link is not modified.'
-        );
-    }
+		// Check the normal links are not affected.
+		$field->setValue('<a href="[sitetree_link,id=1]">Internal</a>');
+		$this->assertEquals(
+			$field->RichLinks(),
+			'<a href="[sitetree_link,id=1]">Internal</a>',
+			'Regular link is not modified.'
+		);
+	}
 }