Fix gzip files for Safari

* Safari support gzip only if the filename does not
  end on .gz - so this renames them to .gzip

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

8 files changed, 18 insertions, 18 deletions

diff --git a/core/Controller/CssController.php b/core/Controller/CssController.php
index b467d386f98..57197e9999e 100644
--- a/core/Controller/CssController.php
+++ b/core/Controller/CssController.php
@@ -95,7 +95,7 @@ class CssController extends Controller {
 		if ($encoding !== null && strpos($encoding, 'gzip') !== false) {
 			try {
 				$gzip = true;
-				return $folder->getFile($fileName . '.gz');
+				return $folder->getFile($fileName . '.gzip'); # Safari doesn't like .gz
 			} catch (NotFoundException $e) {
 				// continue
 			}
diff --git a/core/Controller/JsController.php b/core/Controller/JsController.php
index 0b50abc158a..c7c9dea9101 100644
--- a/core/Controller/JsController.php
+++ b/core/Controller/JsController.php
@@ -96,7 +96,7 @@ class JsController extends Controller {
 		if ($encoding !== null && strpos($encoding, 'gzip') !== false) {
 			try {
 				$gzip = true;
-				return $folder->getFile($fileName . '.gz');
+				return $folder->getFile($fileName . '.gzip'); # Safari doesn't like .gz
 			} catch (NotFoundException $e) {
 				// continue
 			}
diff --git a/lib/private/Template/JSCombiner.php b/lib/private/Template/JSCombiner.php
index 0f30fb915f7..77c46a5e45e 100644
--- a/lib/private/Template/JSCombiner.php
+++ b/lib/private/Template/JSCombiner.php
@@ -155,9 +155,9 @@ class JSCombiner {
 		}
 
 		try {
-			$gzipFile = $folder->getFile($fileName . '.gz');
+			$gzipFile = $folder->getFile($fileName . '.gzip'); # Safari doesn't like .gz
 		} catch (NotFoundException $e) {
-			$gzipFile = $folder->newFile($fileName . '.gz');
+			$gzipFile = $folder->newFile($fileName . '.gzip'); # Safari doesn't like .gz
 		}
 
 		try {
diff --git a/lib/private/Template/SCSSCacher.php b/lib/private/Template/SCSSCacher.php
index df2e0232508..6e09f41a215 100644
--- a/lib/private/Template/SCSSCacher.php
+++ b/lib/private/Template/SCSSCacher.php
@@ -188,9 +188,9 @@ class SCSSCacher {
 
 		// Gzip file
 		try {
-			$gzipFile = $folder->getFile($fileNameCSS . '.gz');
+			$gzipFile = $folder->getFile($fileNameCSS . '.gzip'); # Safari doesn't like .gz
 		} catch (NotFoundException $e) {
-			$gzipFile = $folder->newFile($fileNameCSS . '.gz');
+			$gzipFile = $folder->newFile($fileNameCSS . '.gzip'); # Safari doesn't like .gz
 		}
 
 		try {
diff --git a/tests/Core/Controller/CssControllerTest.php b/tests/Core/Controller/CssControllerTest.php
index 7fa358e056e..30bbc12809e 100644
--- a/tests/Core/Controller/CssControllerTest.php
+++ b/tests/Core/Controller/CssControllerTest.php
@@ -121,7 +121,7 @@ class CssControllerTest extends TestCase {
 			->willReturn($folder);
 
 		$folder->method('getFile')
-			->with('file.css.gz')
+			->with('file.css.gzip')
 			->willReturn($gzipFile);
 
 		$this->request->method('getHeader')
diff --git a/tests/Core/Controller/JsControllerTest.php b/tests/Core/Controller/JsControllerTest.php
index 8f48a7c3390..8456ba8b6e3 100644
--- a/tests/Core/Controller/JsControllerTest.php
+++ b/tests/Core/Controller/JsControllerTest.php
@@ -120,7 +120,7 @@ class JsControllerTest extends TestCase {
 			->willReturn($folder);
 
 		$folder->method('getFile')
-			->with('file.js.gz')
+			->with('file.js.gzip')
 			->willReturn($gzipFile);
 
 		$this->request->method('getHeader')
diff --git a/tests/lib/Template/JSCombinerTest.php b/tests/lib/Template/JSCombinerTest.php
index 1e6234a062c..89bb13338c8 100644
--- a/tests/lib/Template/JSCombinerTest.php
+++ b/tests/lib/Template/JSCombinerTest.php
@@ -111,7 +111,7 @@ class JSCombinerTest extends \Test\TestCase {
 					return $file;
 				} else if ($path === 'combine.js.deps') {
 					throw new NotFoundException();
-				} else if ($path === 'combine.js.gz') {
+				} else if ($path === 'combine.js.gzip') {
 					return $gzfile;
 				}
 				$this->fail();
@@ -148,7 +148,7 @@ class JSCombinerTest extends \Test\TestCase {
 					return $file;
 				} else if ($path === 'combine.js.deps') {
 					throw new NotFoundException();
-				} else if ($path === 'combine.js.gz') {
+				} else if ($path === 'combine.js.gzip') {
 					return $gzfile;
 				}
 				$this->fail();
@@ -302,7 +302,7 @@ class JSCombinerTest extends \Test\TestCase {
 					return $file;
 				} else if ($filename === 'combine.js.deps') {
 					return $depsFile;
-				} else if ($filename === 'combine.js.gz') {
+				} else if ($filename === 'combine.js.gzip') {
 					return $gzFile;
 				}
 				$this->fail();
@@ -333,7 +333,7 @@ class JSCombinerTest extends \Test\TestCase {
 					return $file;
 				} else if ($filename === 'combine.js.deps') {
 					return $depsFile;
-				} else if ($filename === 'combine.js.gz') {
+				} else if ($filename === 'combine.js.gzip') {
 					return $gzFile;
 				}
 				$this->fail();
@@ -401,7 +401,7 @@ var b = \'world\';
 					return $file;
 				} else if ($filename === 'combine.js.deps') {
 					return $depsFile;
-				} else if ($filename === 'combine.js.gz') {
+				} else if ($filename === 'combine.js.gzip') {
 					return $gzFile;
 				}
 				$this->fail();
diff --git a/tests/lib/Template/SCSSCacherTest.php b/tests/lib/Template/SCSSCacherTest.php
index 887fa1ed605..24865b9ae45 100644
--- a/tests/lib/Template/SCSSCacherTest.php
+++ b/tests/lib/Template/SCSSCacherTest.php
@@ -81,7 +81,7 @@ class SCSSCacherTest extends \Test\TestCase {
 					return $file;
 				} else if ($path === 'styles.css.deps') {
 					throw new NotFoundException();
-				} else if ($path === 'styles.css.gz') {
+				} else if ($path === 'styles.css.gzip') {
 					return $gzfile;
 				} else {
 					$this->fail();
@@ -110,7 +110,7 @@ class SCSSCacherTest extends \Test\TestCase {
 					return $file;
 				} else if ($path === 'styles.css.deps') {
 					throw new NotFoundException();
-				} else if ($path === 'styles.css.gz') {
+				} else if ($path === 'styles.css.gzip') {
 					return $gzfile;
 				}else {
 					$this->fail();
@@ -228,7 +228,7 @@ class SCSSCacherTest extends \Test\TestCase {
 				return $file;
 			} else if ($fileName === 'styles.css.deps') {
 				return $depsFile;
-			} else if ($fileName === 'styles.css.gz') {
+			} else if ($fileName === 'styles.css.gzip') {
 				return $gzipFile;
 			}
 			throw new \Exception();
@@ -258,7 +258,7 @@ class SCSSCacherTest extends \Test\TestCase {
 				return $file;
 			} else if ($fileName === 'styles.css.deps') {
 				return $depsFile;
-			} else if ($fileName === 'styles.css.gz') {
+			} else if ($fileName === 'styles.css.gzip') {
 				return $gzipFile;
 			}
 			throw new \Exception();
@@ -288,7 +288,7 @@ class SCSSCacherTest extends \Test\TestCase {
 				return $file;
 			} else if ($fileName === 'styles-success.css.deps') {
 				return $depsFile;
-			} else if ($fileName === 'styles-success.css.gz') {
+			} else if ($fileName === 'styles-success.css.gzip') {
 				return $gzipFile;
 			}
 			throw new \Exception();