nextcloud / server

tests/lib/TemplateFunctionsTest.php

Indentation +240 added lines, -240 removed lines

@@ -9,290 +9,290 @@
 namespace Test;
 
 class TemplateFunctionsTest extends \Test\TestCase {
-	protected function setUp(): void {
-		parent::setUp();
-
-		require_once \OC::$SERVERROOT . '/lib/private/Template/functions.php';
-	}
-
-	public function testPJavaScript(): void {
-		$this->expectOutputString('<img onload="alert(1)" />');
-		p('<img onload="alert(1)" />');
-	}
-
-	public function testPJavaScriptWithScriptTags(): void {
-		$this->expectOutputString('&lt;script&gt;alert(&#039;Hacked!&#039;);&lt;/script&gt;');
-		p("<script>alert('Hacked!');</script>");
-	}
-
-	public function testPNormalString(): void {
-		$string = 'This is a good string without HTML.';
-		$this->expectOutputString($string);
-		p($string);
-	}
-
-	public function testPrintUnescaped(): void {
-		$htmlString = "<script>alert('xss');</script>";
-		$this->expectOutputString($htmlString);
-		print_unescaped($htmlString);
-	}
-
-	public function testPrintUnescapedNormalString(): void {
-		$string = 'This is a good string!';
-		$this->expectOutputString($string);
-		print_unescaped($string);
-	}
-
-	public function testEmitScriptTagWithContent(): void {
-		$this->expectOutputRegex('/<script nonce="[^"]+">\nalert\(\)\n<\/script>\n?/');
-		emit_script_tag('', 'alert()');
-	}
-
-	public function testEmitScriptTagWithSource(): void {
-		$this->expectOutputRegex('/<script nonce=".*" defer src="some.js"><\/script>/');
-		emit_script_tag('some.js');
-	}
-
-	public function testEmitScriptTagWithModuleSource(): void {
-		$this->expectOutputRegex('/<script nonce=".*" type="module" src="some.mjs"><\/script>/');
-		emit_script_tag('some.mjs', '', 'module');
-	}
-
-	public function testEmitImportMap(): void {
-		$this->expectOutputRegex('/^<script[^>]+type="importmap">\n{"imports":{"\/some\/path\/file\.mjs":"\/some\/path\/file\.mjs\?v=123"}}\n<\/script>$/m');
-		emit_import_map(['jsfiles' => ['/some/path/file.mjs?v=123']]);
-	}
-
-	// only create import map for modules with versioning
-	public function testEmitImportMapMixedScripts(): void {
-		$this->expectOutputRegex('/^<script[^>]+type="importmap">\n{"imports":{"\/some\/path\/module\.mjs":"\/some\/path\/module\.mjs\?v=123"}}\n<\/script>$/m');
-		emit_import_map(['jsfiles' => ['/some/path/module.mjs?v=123', '/some/path/classic.js?v=123']]);
-	}
-
-	public function testEmitImportMapNoOutputWithoutVersion(): void {
-		$this->expectOutputString('');
-		emit_import_map(['jsfiles' => ['some.mjs']]);
-	}
-
-	public function testEmitImportMapNoOutputWithClassicScript(): void {
-		$this->expectOutputString('');
-		emit_import_map(['jsfiles' => ['some.js?v=123']]);
-	}
-
-	public function testEmitScriptLoadingTags(): void {
-		// Test mjs js and inline content
-		$pattern = '/type="module"[^>]+src="some\.mjs"[^>]*>.+\n'; // some.mjs with type = module
-		$pattern .= '<script[^>]+src="other\.js"[^>]*>.+\n'; // other.js as plain javascript
-		$pattern .= '<script[^>]*>\n?.*inline.*\n?<\/script>'; // inline content
-		$pattern .= '/'; // no flags
-
-		$this->expectOutputRegex($pattern);
-		emit_script_loading_tags([
-			'jsfiles' => ['some.mjs', 'other.js'],
-			'inline_ocjs' => '// inline'
-		]);
-	}
-
-	public function testEmitScriptLoadingTagsWithVersion(): void {
-		// Test mjs js and inline content
-		$pattern = '/type="module"[^>]+src="some\.mjs\?v=ab123cd"[^>]*>.+\n'; // some.mjs with type = module
-		$pattern .= '<script[^>]+src="other\.js\?v=12abc34"[^>]*>.+\n'; // other.js as plain javascript
-		$pattern .= '/'; // no flags
-
-		$this->expectOutputRegex($pattern);
-		emit_script_loading_tags([
-			'jsfiles' => ['some.mjs?v=ab123cd', 'other.js?v=12abc34'],
-		]);
-	}
-
-	// ---------------------------------------------------------------------------
-	// Test relative_modified_date with dates only
-	// ---------------------------------------------------------------------------
-	public function testRelativeDateToday(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
-
-		$this->assertEquals('today', $result);
-
-		// 2 hours ago is still today
-		$elementTime = $currentTime - 2 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
-
-		$this->assertEquals('today', $result);
-	}
-
-	public function testRelativeDateYesterday(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 24 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
-
-		$this->assertEquals('yesterday', $result);
-
-		// yesterday - 2 hours is still yesterday
-		$elementTime = $currentTime - 26 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
-
-		$this->assertEquals('yesterday', $result);
-	}
-
-	public function testRelativeDate2DaysAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 48 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+    protected function setUp(): void {
+        parent::setUp();
+
+        require_once \OC::$SERVERROOT . '/lib/private/Template/functions.php';
+    }
+
+    public function testPJavaScript(): void {
+        $this->expectOutputString('&lt;img onload=&quot;alert(1)&quot; /&gt;');
+        p('<img onload="alert(1)" />');
+    }
+
+    public function testPJavaScriptWithScriptTags(): void {
+        $this->expectOutputString('&lt;script&gt;alert(&#039;Hacked!&#039;);&lt;/script&gt;');
+        p("<script>alert('Hacked!');</script>");
+    }
+
+    public function testPNormalString(): void {
+        $string = 'This is a good string without HTML.';
+        $this->expectOutputString($string);
+        p($string);
+    }
+
+    public function testPrintUnescaped(): void {
+        $htmlString = "<script>alert('xss');</script>";
+        $this->expectOutputString($htmlString);
+        print_unescaped($htmlString);
+    }
+
+    public function testPrintUnescapedNormalString(): void {
+        $string = 'This is a good string!';
+        $this->expectOutputString($string);
+        print_unescaped($string);
+    }
+
+    public function testEmitScriptTagWithContent(): void {
+        $this->expectOutputRegex('/<script nonce="[^"]+">\nalert\(\)\n<\/script>\n?/');
+        emit_script_tag('', 'alert()');
+    }
+
+    public function testEmitScriptTagWithSource(): void {
+        $this->expectOutputRegex('/<script nonce=".*" defer src="some.js"><\/script>/');
+        emit_script_tag('some.js');
+    }
+
+    public function testEmitScriptTagWithModuleSource(): void {
+        $this->expectOutputRegex('/<script nonce=".*" type="module" src="some.mjs"><\/script>/');
+        emit_script_tag('some.mjs', '', 'module');
+    }
+
+    public function testEmitImportMap(): void {
+        $this->expectOutputRegex('/^<script[^>]+type="importmap">\n{"imports":{"\/some\/path\/file\.mjs":"\/some\/path\/file\.mjs\?v=123"}}\n<\/script>$/m');
+        emit_import_map(['jsfiles' => ['/some/path/file.mjs?v=123']]);
+    }
+
+    // only create import map for modules with versioning
+    public function testEmitImportMapMixedScripts(): void {
+        $this->expectOutputRegex('/^<script[^>]+type="importmap">\n{"imports":{"\/some\/path\/module\.mjs":"\/some\/path\/module\.mjs\?v=123"}}\n<\/script>$/m');
+        emit_import_map(['jsfiles' => ['/some/path/module.mjs?v=123', '/some/path/classic.js?v=123']]);
+    }
+
+    public function testEmitImportMapNoOutputWithoutVersion(): void {
+        $this->expectOutputString('');
+        emit_import_map(['jsfiles' => ['some.mjs']]);
+    }
+
+    public function testEmitImportMapNoOutputWithClassicScript(): void {
+        $this->expectOutputString('');
+        emit_import_map(['jsfiles' => ['some.js?v=123']]);
+    }
+
+    public function testEmitScriptLoadingTags(): void {
+        // Test mjs js and inline content
+        $pattern = '/type="module"[^>]+src="some\.mjs"[^>]*>.+\n'; // some.mjs with type = module
+        $pattern .= '<script[^>]+src="other\.js"[^>]*>.+\n'; // other.js as plain javascript
+        $pattern .= '<script[^>]*>\n?.*inline.*\n?<\/script>'; // inline content
+        $pattern .= '/'; // no flags
+
+        $this->expectOutputRegex($pattern);
+        emit_script_loading_tags([
+            'jsfiles' => ['some.mjs', 'other.js'],
+            'inline_ocjs' => '// inline'
+        ]);
+    }
+
+    public function testEmitScriptLoadingTagsWithVersion(): void {
+        // Test mjs js and inline content
+        $pattern = '/type="module"[^>]+src="some\.mjs\?v=ab123cd"[^>]*>.+\n'; // some.mjs with type = module
+        $pattern .= '<script[^>]+src="other\.js\?v=12abc34"[^>]*>.+\n'; // other.js as plain javascript
+        $pattern .= '/'; // no flags
+
+        $this->expectOutputRegex($pattern);
+        emit_script_loading_tags([
+            'jsfiles' => ['some.mjs?v=ab123cd', 'other.js?v=12abc34'],
+        ]);
+    }
+
+    // ---------------------------------------------------------------------------
+    // Test relative_modified_date with dates only
+    // ---------------------------------------------------------------------------
+    public function testRelativeDateToday(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
+
+        $this->assertEquals('today', $result);
+
+        // 2 hours ago is still today
+        $elementTime = $currentTime - 2 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
+
+        $this->assertEquals('today', $result);
+    }
+
+    public function testRelativeDateYesterday(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 24 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
+
+        $this->assertEquals('yesterday', $result);
+
+        // yesterday - 2 hours is still yesterday
+        $elementTime = $currentTime - 26 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
+
+        $this->assertEquals('yesterday', $result);
+    }
+
+    public function testRelativeDate2DaysAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 48 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('2 days ago', $result);
+        $this->assertEquals('2 days ago', $result);
 
-		// 2 days ago minus 4 hours is still 2 days ago
-		$elementTime = $currentTime - 52 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+        // 2 days ago minus 4 hours is still 2 days ago
+        $elementTime = $currentTime - 52 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('2 days ago', $result);
-	}
+        $this->assertEquals('2 days ago', $result);
+    }
 
-	public function testRelativeDateLastMonth(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 31;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+    public function testRelativeDateLastMonth(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 31;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('last month', $result);
+        $this->assertEquals('last month', $result);
 
-		$elementTime = $currentTime - 86400 * 35;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+        $elementTime = $currentTime - 86400 * 35;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('last month', $result);
-	}
+        $this->assertEquals('last month', $result);
+    }
 
-	public function testRelativeDateMonthsAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 65;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+    public function testRelativeDateMonthsAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 65;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('2 months ago', $result);
+        $this->assertEquals('2 months ago', $result);
 
-		$elementTime = $currentTime - 86400 * 130;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+        $elementTime = $currentTime - 86400 * 130;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('4 months ago', $result);
-	}
+        $this->assertEquals('4 months ago', $result);
+    }
 
-	public function testRelativeDateLastYear(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 365;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+    public function testRelativeDateLastYear(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 365;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('last year', $result);
+        $this->assertEquals('last year', $result);
 
-		$elementTime = $currentTime - 86400 * 450;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+        $elementTime = $currentTime - 86400 * 450;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('last year', $result);
-	}
+        $this->assertEquals('last year', $result);
+    }
 
-	public function testRelativeDateYearsAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 365.25 * 2;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+    public function testRelativeDateYearsAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 365.25 * 2;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('2 years ago', $result);
+        $this->assertEquals('2 years ago', $result);
 
-		$elementTime = $currentTime - 86400 * 365.25 * 3;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+        $elementTime = $currentTime - 86400 * 365.25 * 3;
+        $result = (string)relative_modified_date($elementTime, $currentTime, true);
 
-		$this->assertEquals('3 years ago', $result);
-	}
+        $this->assertEquals('3 years ago', $result);
+    }
 
-	// ---------------------------------------------------------------------------
-	// Test relative_modified_date with timestamps only (date + time value)
-	// ---------------------------------------------------------------------------
+    // ---------------------------------------------------------------------------
+    // Test relative_modified_date with timestamps only (date + time value)
+    // ---------------------------------------------------------------------------
 
-	public function testRelativeTimeSecondsAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 5;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeSecondsAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 5;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('seconds ago', $result);
-	}
+        $this->assertEquals('seconds ago', $result);
+    }
 
-	public function testRelativeTimeMinutesAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 190;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeMinutesAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 190;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('3 minutes ago', $result);
-	}
+        $this->assertEquals('3 minutes ago', $result);
+    }
 
-	public function testRelativeTimeHoursAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 7500;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeHoursAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 7500;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('2 hours ago', $result);
-	}
+        $this->assertEquals('2 hours ago', $result);
+    }
 
-	public function testRelativeTime2DaysAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 48 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTime2DaysAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 48 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('2 days ago', $result);
+        $this->assertEquals('2 days ago', $result);
 
-		// 2 days ago minus 4 hours is still 2 days ago
-		$elementTime = $currentTime - 52 * 3600;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+        // 2 days ago minus 4 hours is still 2 days ago
+        $elementTime = $currentTime - 52 * 3600;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('2 days ago', $result);
-	}
+        $this->assertEquals('2 days ago', $result);
+    }
 
-	public function testRelativeTimeLastMonth(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 31;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeLastMonth(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 31;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('last month', $result);
+        $this->assertEquals('last month', $result);
 
-		$elementTime = $currentTime - 86400 * 35;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+        $elementTime = $currentTime - 86400 * 35;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('last month', $result);
-	}
+        $this->assertEquals('last month', $result);
+    }
 
-	public function testRelativeTimeMonthsAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 65;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeMonthsAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 65;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('2 months ago', $result);
+        $this->assertEquals('2 months ago', $result);
 
-		$elementTime = $currentTime - 86400 * 130;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+        $elementTime = $currentTime - 86400 * 130;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('4 months ago', $result);
-	}
+        $this->assertEquals('4 months ago', $result);
+    }
 
-	public function testRelativeTimeLastYear(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 365;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeLastYear(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 365;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('last year', $result);
+        $this->assertEquals('last year', $result);
 
-		$elementTime = $currentTime - 86400 * 450;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+        $elementTime = $currentTime - 86400 * 450;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('last year', $result);
-	}
+        $this->assertEquals('last year', $result);
+    }
 
-	public function testRelativeTimeYearsAgo(): void {
-		$currentTime = 1380703592;
-		$elementTime = $currentTime - 86400 * 365.25 * 2;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+    public function testRelativeTimeYearsAgo(): void {
+        $currentTime = 1380703592;
+        $elementTime = $currentTime - 86400 * 365.25 * 2;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('2 years ago', $result);
+        $this->assertEquals('2 years ago', $result);
 
-		$elementTime = $currentTime - 86400 * 365.25 * 3;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+        $elementTime = $currentTime - 86400 * 365.25 * 3;
+        $result = (string)relative_modified_date($elementTime, $currentTime, false);
 
-		$this->assertEquals('3 years ago', $result);
-	}
+        $this->assertEquals('3 years ago', $result);
+    }
 }

lib/private/Template/functions.php

Indentation +112 added lines, -112 removed lines

@@ -17,7 +17,7 @@ 
  * @param string $string
  */
 function p($string): void {
-	print(Util::sanitizeHTML($string));
+    print(Util::sanitizeHTML($string));
 }
 
 /**
@@ -26,14 +26,14 @@ 
  * @param string $opts, additional optional options
  */
 function emit_css_tag($href, $opts = ''): void {
-	$s = '<link rel="stylesheet"';
-	if (!empty($href)) {
-		$s .= ' href="' . $href . '"';
-	}
-	if (!empty($opts)) {
-		$s .= ' ' . $opts;
-	}
-	print_unescaped($s . ">\n");
+    $s = '<link rel="stylesheet"';
+    if (!empty($href)) {
+        $s .= ' href="' . $href . '"';
+    }
+    if (!empty($opts)) {
+        $s .= ' ' . $opts;
+    }
+    print_unescaped($s . ">\n");
 }
 
 /**
@@ -41,12 +41,12 @@ 
  * @param array $obj all the script information from template
  */
 function emit_css_loading_tags($obj): void {
-	foreach ($obj['cssfiles'] as $css) {
-		emit_css_tag($css);
-	}
-	foreach ($obj['printcssfiles'] as $css) {
-		emit_css_tag($css, 'media="print"');
-	}
+    foreach ($obj['cssfiles'] as $css) {
+        emit_css_tag($css);
+    }
+    foreach ($obj['printcssfiles'] as $css) {
+        emit_css_tag($css, 'media="print"');
+    }
 }
 
 /**
@@ -58,21 +58,21 @@ 
  * @since 27.0.0 added the $content_type parameter
  */
 function emit_script_tag(string $src, string $script_content = '', string $content_type = ''): void {
-	$nonceManager = Server::get(ContentSecurityPolicyNonceManager::class);
+    $nonceManager = Server::get(ContentSecurityPolicyNonceManager::class);
 
-	$defer_str = $content_type === '' ? ' defer' : ''; // "defer" only works with classic scripts
-	$type = $content_type !== '' ? ' type="' . $content_type . '"' : '';
+    $defer_str = $content_type === '' ? ' defer' : ''; // "defer" only works with classic scripts
+    $type = $content_type !== '' ? ' type="' . $content_type . '"' : '';
 
-	$s = '<script nonce="' . $nonceManager->getNonce() . '"' . $type;
-	if (!empty($src)) {
-		// emit script tag for deferred loading from $src
-		$s .= $defer_str . ' src="' . $src . '">';
-	} else {
-		// emit script tag for inline script from $script_content without defer (see MDN)
-		$s .= ">\n" . $script_content . "\n";
-	}
-	$s .= '</script>';
-	print_unescaped($s . "\n");
+    $s = '<script nonce="' . $nonceManager->getNonce() . '"' . $type;
+    if (!empty($src)) {
+        // emit script tag for deferred loading from $src
+        $s .= $defer_str . ' src="' . $src . '">';
+    } else {
+        // emit script tag for inline script from $script_content without defer (see MDN)
+        $s .= ">\n" . $script_content . "\n";
+    }
+    $s .= '</script>';
+    print_unescaped($s . "\n");
 }
 
 /**
@@ -80,16 +80,16 @@ 
  * @param array $obj all the script information from template
  */
 function emit_script_loading_tags($obj): void {
-	emit_import_map($obj);
+    emit_import_map($obj);
 
-	foreach ($obj['jsfiles'] as $jsfile) {
-		$fileName = explode('?', $jsfile, 2)[0];
-		$type = str_ends_with($fileName, '.mjs') ? 'module' : '';
-		emit_script_tag($jsfile, '', $type);
-	}
-	if (!empty($obj['inline_ocjs'])) {
-		emit_script_tag('', $obj['inline_ocjs']);
-	}
+    foreach ($obj['jsfiles'] as $jsfile) {
+        $fileName = explode('?', $jsfile, 2)[0];
+        $type = str_ends_with($fileName, '.mjs') ? 'module' : '';
+        emit_script_tag($jsfile, '', $type);
+    }
+    if (!empty($obj['inline_ocjs'])) {
+        emit_script_tag('', $obj['inline_ocjs']);
+    }
 }
 
 /**
@@ -101,18 +101,18 @@ 
  * @param $obj all the script information from template
  */
 function emit_import_map(array $obj): void {
-	$modules = [];
-	foreach ($obj['jsfiles'] as $jsfile) {
-		$fileName = explode('?', $jsfile, 2)[0];
-		if (str_ends_with($fileName, '.mjs') && $jsfile !== $fileName) {
-			// its a module and we have a cache buster available
-			$modules[$fileName] = $jsfile;
-		}
-	}
-	if (!empty($modules)) {
-		$json = json_encode(['imports' => $modules], JSON_UNESCAPED_SLASHES | JSON_FORCE_OBJECT);
-		emit_script_tag('', $json, 'importmap');
-	}
+    $modules = [];
+    foreach ($obj['jsfiles'] as $jsfile) {
+        $fileName = explode('?', $jsfile, 2)[0];
+        if (str_ends_with($fileName, '.mjs') && $jsfile !== $fileName) {
+            // its a module and we have a cache buster available
+            $modules[$fileName] = $jsfile;
+        }
+    }
+    if (!empty($modules)) {
+        $json = json_encode(['imports' => $modules], JSON_UNESCAPED_SLASHES | JSON_FORCE_OBJECT);
+        emit_script_tag('', $json, 'importmap');
+    }
 }
 
 /**
@@ -121,7 +121,7 @@ 
  * @param string $string the string which will be printed as it is
  */
 function print_unescaped($string): void {
-	print($string);
+    print($string);
 }
 
 /**
@@ -137,13 +137,13 @@ 
  *                              if an array is given it will add all scripts
  */
 function script($app, $file = null): void {
-	if (is_array($file)) {
-		foreach ($file as $script) {
-			Util::addScript($app, $script, 'core');
-		}
-	} else {
-		Util::addScript($app, $file, 'core');
-	}
+    if (is_array($file)) {
+        foreach ($file as $script) {
+            Util::addScript($app, $script, 'core');
+        }
+    } else {
+        Util::addScript($app, $file, 'core');
+    }
 }
 
 /**
@@ -153,13 +153,13 @@ 
  *                              if an array is given it will add all styles
  */
 function style($app, $file = null): void {
-	if (is_array($file)) {
-		foreach ($file as $f) {
-			Util::addStyle($app, $f);
-		}
-	} else {
-		Util::addStyle($app, $file);
-	}
+    if (is_array($file)) {
+        foreach ($file as $f) {
+            Util::addStyle($app, $f);
+        }
+    } else {
+        Util::addStyle($app, $file);
+    }
 }
 
 /**
@@ -170,13 +170,13 @@ 
  * @deprecated 32.0.0
  */
 function vendor_style($app, $file = null): void {
-	if (is_array($file)) {
-		foreach ($file as $f) {
-			OC_Util::addVendorStyle($app, $f);
-		}
-	} else {
-		OC_Util::addVendorStyle($app, $file);
-	}
+    if (is_array($file)) {
+        foreach ($file as $f) {
+            OC_Util::addVendorStyle($app, $f);
+        }
+    } else {
+        OC_Util::addVendorStyle($app, $file);
+    }
 }
 
 /**
@@ -185,7 +185,7 @@ 
  *                    if an array is given it will add all styles
  */
 function translation($app): void {
-	Util::addTranslations($app);
+    Util::addTranslations($app);
 }
 
 /**
@@ -198,7 +198,7 @@ 
  * For further information have a look at \OCP\IURLGenerator::linkTo
  */
 function link_to($app, $file, $args = []) {
-	return Server::get(IURLGenerator::class)->linkTo($app, $file, $args);
+    return Server::get(IURLGenerator::class)->linkTo($app, $file, $args);
 }
 
 /**
@@ -206,7 +206,7 @@ 
  * @return string url to the online documentation
  */
 function link_to_docs($key) {
-	return Server::get(IURLGenerator::class)->linkToDocs($key);
+    return Server::get(IURLGenerator::class)->linkToDocs($key);
 }
 
 /**
@@ -218,7 +218,7 @@ 
  * For further information have a look at \OCP\IURLGenerator::imagePath
  */
 function image_path($app, $image) {
-	return Server::get(IURLGenerator::class)->imagePath($app, $image);
+    return Server::get(IURLGenerator::class)->imagePath($app, $image);
 }
 
 /**
@@ -227,7 +227,7 @@ 
  * @return string link to the image
  */
 function mimetype_icon($mimetype) {
-	return Server::get(IMimeTypeDetector::class)->mimeTypeIcon($mimetype);
+    return Server::get(IMimeTypeDetector::class)->mimeTypeIcon($mimetype);
 }
 
 /**
@@ -237,7 +237,7 @@ 
  * @return string link to the preview
  */
 function preview_icon($path) {
-	return Server::get(IURLGenerator::class)->linkToRoute('core.Preview.getPreview', ['x' => 32, 'y' => 32, 'file' => $path]);
+    return Server::get(IURLGenerator::class)->linkToRoute('core.Preview.getPreview', ['x' => 32, 'y' => 32, 'file' => $path]);
 }
 
 /**
@@ -246,7 +246,7 @@ 
  * @return string
  */
 function publicPreview_icon($path, $token) {
-	return Server::get(IURLGenerator::class)->linkToRoute('files_sharing.PublicPreview.getPreview', ['x' => 32, 'y' => 32, 'file' => $path, 'token' => $token]);
+    return Server::get(IURLGenerator::class)->linkToRoute('files_sharing.PublicPreview.getPreview', ['x' => 32, 'y' => 32, 'file' => $path, 'token' => $token]);
 }
 
 /**
@@ -258,7 +258,7 @@ 
  * For further information have a look at Util::humanFileSize
  */
 function human_file_size($bytes) {
-	return Util::humanFileSize($bytes);
+    return Util::humanFileSize($bytes);
 }
 
 /**
@@ -267,9 +267,9 @@ 
  * @return int timestamp without time value
  */
 function strip_time($timestamp) {
-	$date = new \DateTime("@{$timestamp}");
-	$date->setTime(0, 0, 0);
-	return (int)$date->format('U');
+    $date = new \DateTime("@{$timestamp}");
+    $date->setTime(0, 0, 0);
+    return (int)$date->format('U');
 }
 
 /**
@@ -281,12 +281,12 @@ 
  * @return string timestamp
  */
 function relative_modified_date($timestamp, $fromTime = null, $dateOnly = false): string {
-	$formatter = Server::get(IDateTimeFormatter::class);
+    $formatter = Server::get(IDateTimeFormatter::class);
 
-	if ($dateOnly) {
-		return $formatter->formatDateSpan($timestamp, $fromTime);
-	}
-	return $formatter->formatTimeSpan($timestamp, $fromTime);
+    if ($dateOnly) {
+        return $formatter->formatDateSpan($timestamp, $fromTime);
+    }
+    return $formatter->formatTimeSpan($timestamp, $fromTime);
 }
 
 /**
@@ -295,29 +295,29 @@ 
  * @param array $params
  */
 function html_select_options($options, $selected, $params = []): string {
-	if (!is_array($selected)) {
-		$selected = [$selected];
-	}
-	if (isset($params['combine']) && $params['combine']) {
-		$options = array_combine($options, $options);
-	}
-	$value_name = $label_name = false;
-	if (isset($params['value'])) {
-		$value_name = $params['value'];
-	}
-	if (isset($params['label'])) {
-		$label_name = $params['label'];
-	}
-	$html = '';
-	foreach ($options as $value => $label) {
-		if ($value_name && is_array($label)) {
-			$value = $label[$value_name];
-		}
-		if ($label_name && is_array($label)) {
-			$label = $label[$label_name];
-		}
-		$select = in_array($value, $selected) ? ' selected="selected"' : '';
-		$html .= '<option value="' . Util::sanitizeHTML($value) . '"' . $select . '>' . Util::sanitizeHTML($label) . '</option>' . "\n";
-	}
-	return $html;
+    if (!is_array($selected)) {
+        $selected = [$selected];
+    }
+    if (isset($params['combine']) && $params['combine']) {
+        $options = array_combine($options, $options);
+    }
+    $value_name = $label_name = false;
+    if (isset($params['value'])) {
+        $value_name = $params['value'];
+    }
+    if (isset($params['label'])) {
+        $label_name = $params['label'];
+    }
+    $html = '';
+    foreach ($options as $value => $label) {
+        if ($value_name && is_array($label)) {
+            $value = $label[$value_name];
+        }
+        if ($label_name && is_array($label)) {
+            $label = $label[$label_name];
+        }
+        $select = in_array($value, $selected) ? ' selected="selected"' : '';
+        $html .= '<option value="' . Util::sanitizeHTML($value) . '"' . $select . '>' . Util::sanitizeHTML($label) . '</option>' . "\n";
+    }
+    return $html;
 }

Spacing +10 added lines, -10 removed lines

@@ -28,12 +28,12 @@ 
 function emit_css_tag($href, $opts = ''): void {
 	$s = '<link rel="stylesheet"';
 	if (!empty($href)) {
-		$s .= ' href="' . $href . '"';
+		$s .= ' href="'.$href.'"';
 	}
 	if (!empty($opts)) {
-		$s .= ' ' . $opts;
+		$s .= ' '.$opts;
 	}
-	print_unescaped($s . ">\n");
+	print_unescaped($s.">\n");
 }
 
 /**
@@ -61,18 +61,18 @@ 
 	$nonceManager = Server::get(ContentSecurityPolicyNonceManager::class);
 
 	$defer_str = $content_type === '' ? ' defer' : ''; // "defer" only works with classic scripts
-	$type = $content_type !== '' ? ' type="' . $content_type . '"' : '';
+	$type = $content_type !== '' ? ' type="'.$content_type.'"' : '';
 
-	$s = '<script nonce="' . $nonceManager->getNonce() . '"' . $type;
+	$s = '<script nonce="'.$nonceManager->getNonce().'"'.$type;
 	if (!empty($src)) {
 		// emit script tag for deferred loading from $src
-		$s .= $defer_str . ' src="' . $src . '">';
+		$s .= $defer_str.' src="'.$src.'">';
 	} else {
 		// emit script tag for inline script from $script_content without defer (see MDN)
-		$s .= ">\n" . $script_content . "\n";
+		$s .= ">\n".$script_content."\n";
 	}
 	$s .= '</script>';
-	print_unescaped($s . "\n");
+	print_unescaped($s."\n");
 }
 
 /**
@@ -269,7 +269,7 @@ 
 function strip_time($timestamp) {
 	$date = new \DateTime("@{$timestamp}");
 	$date->setTime(0, 0, 0);
-	return (int)$date->format('U');
+	return (int) $date->format('U');
 }
 
 /**
@@ -317,7 +317,7 @@ 
 			$label = $label[$label_name];
 		}
 		$select = in_array($value, $selected) ? ' selected="selected"' : '';
-		$html .= '<option value="' . Util::sanitizeHTML($value) . '"' . $select . '>' . Util::sanitizeHTML($label) . '</option>' . "\n";
+		$html .= '<option value="'.Util::sanitizeHTML($value).'"'.$select.'>'.Util::sanitizeHTML($label).'</option>'."\n";
 	}
 	return $html;
 }