Merge pull request #3364 from nextcloud/sass-logging

Lower log levels of resource locator

lib/private/Template/CSSResourceLocator.php

@@ -91,11 +91,11 @@ class CSSResourceLocator extends ResourceLocator {
 					$this->append($root, $this->scssCacher->getCachedSCSS($app, $file), false);
 					return true;
 				} else {
-					$this->logger->error('Failed to compile and/or save '.$root.'/'.$file, ['app' => 'core']);
+					$this->logger->warning('Failed to compile and/or save '.$root.'/'.$file, ['app' => 'core']);
 					return false;
 				}
 			} else {
-				$this->logger->error('Scss is disabled for '.$root.'/'.$file.', ignoring', ['app' => 'core']);
+				$this->logger->debug('Scss is disabled for '.$root.'/'.$file.', ignoring', ['app' => 'core']);
 				return true;
 			}
 		}

lib/private/Template/ResourceLocator.php

@@ -75,7 +75,7 @@ abstract class ResourceLocator {
 				$this->doFind($resource);
 			} catch (ResourceNotFoundException $e) {
 				$resourceApp = substr($resource, 0, strpos($resource, '/'));
-				$this->logger->error('Could not find resource file "' . $e->getResourcePath() . '"', ['app' => $resourceApp]);
+				$this->logger->debug('Could not find resource file "' . $e->getResourcePath() . '"', ['app' => $resourceApp]);
 			}
 		}
 		if (!empty($this->theme)) {
@@ -84,7 +84,7 @@ abstract class ResourceLocator {
 					$this->doFindTheme($resource);
 				} catch (ResourceNotFoundException $e) {
 					$resourceApp = substr($resource, 0, strpos($resource, '/'));
-					$this->logger->error('Could not find resource file "' . $e->getResourcePath() . '"', ['app' => $resourceApp]);
+					$this->logger->debug('Could not find resource file "' . $e->getResourcePath() . '"', ['app' => $resourceApp]);
 				}
 			}
 		}

tests/lib/Template/ResourceLocatorTest.php

@@ -69,7 +69,7 @@ class ResourceLocatorTest extends \Test\TestCase {
 			->with('foo')
 			->will($this->throwException(new ResourceNotFoundException('foo', 'map')));
 		$this->logger->expects($this->exactly(2))
-			->method('error')
+			->method('debug')
 			->with($this->stringContains('map/foo'));
 		/** @var \OC\Template\ResourceLocator $locator */
 		$locator->find(array('foo'));