Merge pull request #7688 from nextcloud/stable12-fix-scss-webroot-and-url-rewrite
[Stable12] fix scss webroot and url rewrite
This commit is contained in:
commit
28d77efe80
|
@ -171,7 +171,7 @@ class ThemingController extends Controller {
|
|||
$this->themingDefaults->set($setting, $value);
|
||||
|
||||
// reprocess server scss for preview
|
||||
$cssCached = $this->scssCacher->process(\OC::$SERVERROOT, '/core/css/server.scss', 'core');
|
||||
$cssCached = $this->scssCacher->process(\OC::$SERVERROOT, 'core/css/server.scss', 'core');
|
||||
|
||||
return new DataResponse(
|
||||
[
|
||||
|
|
|
@ -104,7 +104,7 @@ class CSSResourceLocator extends ResourceLocator {
|
|||
if($this->scssCacher !== null) {
|
||||
if($this->scssCacher->process($root, $file, $app)) {
|
||||
|
||||
$this->append($root, $this->scssCacher->getCachedSCSS($app, $file), false, true, true);
|
||||
$this->append($root, $this->scssCacher->getCachedSCSS($app, $file), \OC::$WEBROOT, true, true);
|
||||
return true;
|
||||
} else {
|
||||
$this->logger->warning('Failed to compile and/or save '.$root.'/'.$file, ['app' => 'core']);
|
||||
|
@ -141,7 +141,7 @@ class CSSResourceLocator extends ResourceLocator {
|
|||
}
|
||||
}
|
||||
|
||||
$this->resources[] = array($webRoot? : '/', $webRoot, $file);
|
||||
$this->resources[] = array($webRoot? : \OC::$WEBROOT, $webRoot, $file);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -2,6 +2,13 @@
|
|||
/**
|
||||
* @copyright Copyright (c) 2016, John Molakvoæ (skjnldsv@protonmail.com)
|
||||
*
|
||||
* @author John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
||||
* @author Julius Haertl <jus@bitgrid.net>
|
||||
* @author Julius Härtl <jus@bitgrid.net>
|
||||
* @author Lukas Reschke <lukas@statuscode.ch>
|
||||
* @author Morris Jobke <hey@morrisjobke.de>
|
||||
* @author Roeland Jago Douma <roeland@famdouma.nl>
|
||||
*
|
||||
* @license GNU AGPL version 3 or any later version
|
||||
*
|
||||
* This program is free software: you can redistribute it and/or modify
|
||||
|
@ -95,8 +102,7 @@ class SCSSCacher {
|
|||
$fileNameCSS = $this->prependBaseurlPrefix(str_replace('.scss', '.css', $fileNameSCSS));
|
||||
|
||||
$path = implode('/', $path);
|
||||
|
||||
$webDir = substr($path, strlen($this->serverRoot)+1);
|
||||
$webDir = $this->getWebDir($path, $app, $this->serverRoot, \OC::$WEBROOT);
|
||||
|
||||
try {
|
||||
$folder = $this->appData->getFolder($app);
|
||||
|
@ -105,10 +111,10 @@ class SCSSCacher {
|
|||
$folder = $this->appData->newFolder($app);
|
||||
}
|
||||
|
||||
|
||||
if(!$this->variablesChanged() && $this->isCached($fileNameCSS, $folder)) {
|
||||
return true;
|
||||
}
|
||||
|
||||
return $this->cache($path, $fileNameCSS, $fileNameSCSS, $folder, $webDir);
|
||||
}
|
||||
|
||||
|
@ -181,7 +187,7 @@ class SCSSCacher {
|
|||
$scss = new Compiler();
|
||||
$scss->setImportPaths([
|
||||
$path,
|
||||
\OC::$SERVERROOT . '/core/css/',
|
||||
$this->serverRoot . '/core/css/',
|
||||
]);
|
||||
// Continue after throw
|
||||
$scss->setIgnoreErrors(true);
|
||||
|
@ -276,12 +282,7 @@ class SCSSCacher {
|
|||
*/
|
||||
private function rebaseUrls($css, $webDir) {
|
||||
$re = '/url\([\'"]([\.\w?=\/-]*)[\'"]\)/x';
|
||||
// OC\Route\Router:75
|
||||
if(($this->config->getSystemValue('htaccess.IgnoreFrontController', false) === true || getenv('front_controller_active') === 'true')) {
|
||||
$subst = 'url(\'../../'.$webDir.'/$1\')';
|
||||
} else {
|
||||
$subst = 'url(\'../../../'.$webDir.'/$1\')';
|
||||
}
|
||||
$subst = 'url(\''.$webDir.'/$1\')';
|
||||
return preg_replace($re, $subst, $css);
|
||||
}
|
||||
|
||||
|
@ -307,4 +308,23 @@ class SCSSCacher {
|
|||
private function prependBaseurlPrefix($cssFile) {
|
||||
return md5($this->urlGenerator->getBaseUrl()) . '-' . $cssFile;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get WebDir root
|
||||
* @param string $path the css file path
|
||||
* @param string $appName the app name
|
||||
* @param string $serverRoot the server root path
|
||||
* @param string $webRoot the nextcloud installation root path
|
||||
* @return string the webDir
|
||||
*/
|
||||
private function getWebDir($path, $appName, $serverRoot, $webRoot) {
|
||||
// Detect if path is within server root AND if path is within an app path
|
||||
if ( strpos($path, $serverRoot) === false && $appWebPath = \OC_App::getAppWebPath($appName)) {
|
||||
// Get the file path within the app directory
|
||||
$appDirectoryPath = explode($appName, $path)[1];
|
||||
// Remove the webroot
|
||||
return str_replace($webRoot, '', $appWebPath.$appDirectoryPath);
|
||||
}
|
||||
return $webRoot.substr($path, strlen($serverRoot));
|
||||
}
|
||||
}
|
||||
|
|
|
@ -12,6 +12,17 @@ use OC\Files\View;
|
|||
use OC_Helper;
|
||||
|
||||
class LegacyHelperTest extends \Test\TestCase {
|
||||
/** @var string */
|
||||
private $originalWebRoot;
|
||||
|
||||
public function setUp() {
|
||||
$this->originalWebRoot = \OC::$WEBROOT;
|
||||
}
|
||||
|
||||
public function tearDown() {
|
||||
// Reset webRoot
|
||||
\OC::$WEBROOT = $this->originalWebRoot;
|
||||
}
|
||||
|
||||
/**
|
||||
* @dataProvider humanFileSizeProvider
|
||||
|
|
|
@ -204,7 +204,7 @@ class CSSResourceLocatorTest extends \Test\TestCase {
|
|||
$webRoot = $resource[1];
|
||||
$file = $resource[2];
|
||||
|
||||
$expectedRoot = '/';
|
||||
$expectedRoot = '';
|
||||
$expectedWebRoot = '';
|
||||
$expectedFile = 'test-file';
|
||||
|
||||
|
|
|
@ -352,19 +352,10 @@ class SCSSCacherTest extends \Test\TestCase {
|
|||
}
|
||||
|
||||
public function testRebaseUrls() {
|
||||
$webDir = 'apps/files/css';
|
||||
$webDir = '/apps/files/css';
|
||||
$css = '#id { background-image: url(\'../img/image.jpg\'); }';
|
||||
$actual = self::invokePrivate($this->scssCacher, 'rebaseUrls', [$css, $webDir]);
|
||||
$expected = '#id { background-image: url(\'../../../apps/files/css/../img/image.jpg\'); }';
|
||||
$this->assertEquals($expected, $actual);
|
||||
}
|
||||
|
||||
public function testRebaseUrlsIgnoreFrontendController() {
|
||||
$this->config->expects($this->once())->method('getSystemValue')->with('htaccess.IgnoreFrontController', false)->willReturn(true);
|
||||
$webDir = 'apps/files/css';
|
||||
$css = '#id { background-image: url(\'../img/image.jpg\'); }';
|
||||
$actual = self::invokePrivate($this->scssCacher, 'rebaseUrls', [$css, $webDir]);
|
||||
$expected = '#id { background-image: url(\'../../apps/files/css/../img/image.jpg\'); }';
|
||||
$expected = '#id { background-image: url(\'/apps/files/css/../img/image.jpg\'); }';
|
||||
$this->assertEquals($expected, $actual);
|
||||
}
|
||||
|
||||
|
@ -393,4 +384,55 @@ class SCSSCacherTest extends \Test\TestCase {
|
|||
$this->assertEquals(substr($result, 1), $actual);
|
||||
}
|
||||
|
||||
private function randomString() {
|
||||
return sha1(uniqid(mt_rand(), true));
|
||||
}
|
||||
|
||||
private function rrmdir($directory) {
|
||||
$files = array_diff(scandir($directory), array('.','..'));
|
||||
foreach ($files as $file) {
|
||||
if (is_dir($directory . '/' . $file)) {
|
||||
$this->rrmdir($directory . '/' . $file);
|
||||
} else {
|
||||
unlink($directory . '/' . $file);
|
||||
}
|
||||
}
|
||||
return rmdir($directory);
|
||||
}
|
||||
|
||||
public function dataGetWebDir() {
|
||||
return [
|
||||
// Root installation
|
||||
['/http/core/css', 'core', '', '/http', '/core/css'],
|
||||
['/http/apps/scss/css', 'scss', '', '/http', '/apps/scss/css'],
|
||||
['/srv/apps2/scss/css', 'scss', '', '/http', '/apps2/scss/css'],
|
||||
// Sub directory install
|
||||
['/http/nextcloud/core/css', 'core', '/nextcloud', '/http/nextcloud', '/nextcloud/core/css'],
|
||||
['/http/nextcloud/apps/scss/css', 'scss', '/nextcloud', '/http/nextcloud', '/nextcloud/apps/scss/css'],
|
||||
['/srv/apps2/scss/css', 'scss', '/nextcloud', '/http/nextcloud', '/apps2/scss/css']
|
||||
];
|
||||
}
|
||||
|
||||
/**
|
||||
* @param $path
|
||||
* @param $appName
|
||||
* @param $webRoot
|
||||
* @param $serverRoot
|
||||
* @dataProvider dataGetWebDir
|
||||
*/
|
||||
public function testgetWebDir($path, $appName, $webRoot, $serverRoot, $correctWebDir) {
|
||||
$tmpDir = sys_get_temp_dir().'/'.$this->randomString();
|
||||
// Adding fake apps folder and create fake app install
|
||||
\OC::$APPSROOTS[] = [
|
||||
'path' => $tmpDir.'/srv/apps2',
|
||||
'url' => '/apps2',
|
||||
'writable' => false
|
||||
];
|
||||
mkdir($tmpDir.$path, 0777, true);
|
||||
$actual = self::invokePrivate($this->scssCacher, 'getWebDir', [$tmpDir.$path, $appName, $tmpDir.$serverRoot, $webRoot]);
|
||||
$this->assertEquals($correctWebDir, $actual);
|
||||
array_pop(\OC::$APPSROOTS);
|
||||
$this->rrmdir($tmpDir.$path);
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
@ -27,6 +27,8 @@ class UrlGeneratorTest extends \Test\TestCase {
|
|||
private $request;
|
||||
/** @var IURLGenerator */
|
||||
private $urlGenerator;
|
||||
/** @var string */
|
||||
private $originalWebRoot;
|
||||
|
||||
public function setUp() {
|
||||
parent::setUp();
|
||||
|
@ -38,6 +40,12 @@ class UrlGeneratorTest extends \Test\TestCase {
|
|||
$this->cacheFactory,
|
||||
$this->request
|
||||
);
|
||||
$this->originalWebRoot = \OC::$WEBROOT;
|
||||
}
|
||||
|
||||
public function tearDown() {
|
||||
// Reset webRoot
|
||||
\OC::$WEBROOT = $this->originalWebRoot;
|
||||
}
|
||||
|
||||
private function mockBaseUrl() {
|
||||
|
@ -47,7 +55,6 @@ class UrlGeneratorTest extends \Test\TestCase {
|
|||
$this->request->expects($this->once())
|
||||
->method('getServerHost')
|
||||
->willReturn('localhost');
|
||||
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -156,4 +163,3 @@ class UrlGeneratorTest extends \Test\TestCase {
|
|||
}
|
||||
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue