Merge pull request #5 from learn-symfony/master

sync with main repository

Author: Eugene Matvejev

.scrutinizer.yml

@@ -0,0 +1,20 @@
+checks:
+    php: true
+
+coding_style:
+    php: {  }
+
+filter:
+    excluded_paths: ["bin/*", "tests/*", "vendor/*"]
+
+build:
+    environment:
+        php:
+            version: "7.0.4"
+#    tests:
+#        override:
+#            -
+#                command: "php bin/phpunit -c phpunit.xml --colors=always --verbose --coverage-clover=coverage.xml"
+#                coverage:
+#                    file: "coverage.xml"
+#                    format: "php-clover"

README.md

@@ -1,3 +1,5 @@
+[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/learn-symfony/css-compiler/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/learn-symfony/css-compiler/?branch=master)
+
 [![Latest Stable Version](https://poser.pugx.org/eugene-matvejev/css-compiler/version)](https://packagist.org/packages/eugene-matvejev/css-compiler)
 [![Total Downloads](https://poser.pugx.org/eugene-matvejev/css-compiler/downloads)](https://packagist.org/packages/eugene-matvejev/css-compiler)
 [![License](https://poser.pugx.org/eugene-matvejev/css-compiler/license)](https://packagist.org/packages/eugene-matvejev/css-compiler)

composer.json

@@ -23,7 +23,8 @@
     "require": {
         "php": ">= 5.5.9",
         "leafo/lessphp": "^0.5",
-        "leafo/scssphp-compass": "dev-master"
+        "leafo/scssphp": "@dev",
+        "leafo/scssphp-compass": "@dev"
     },
     "require-dev": {
         "composer/composer": "^1.1",

src/Container/File.php

@@ -10,6 +10,16 @@ class File
     const TYPE_SASS    = 'sass';
     const TYPE_COMPASS = 'compass';
     const TYPE_LESS    = 'less';
+    const TYPE_UNKNOWN = 'unknown';
+    /**
+     * @var string[]
+     */
+    private static $extensions = [
+        self::TYPE_SCSS,
+        self::TYPE_SASS,
+        self::TYPE_COMPASS,
+        self::TYPE_LESS,
+    ];
     /**
      * @var string
      */
@@ -157,21 +167,11 @@ public function setType($type)
      */
     private function detectSourceTypeFromPath($path)
     {
-        switch (true) {
-            case 0 !== preg_match('/^.*\.' . static::TYPE_SCSS . '/', $path):
-                $this->type = static::TYPE_SCSS;
-                break;
-            case 0 !== preg_match('/^.*\.' . static::TYPE_SASS . '/', $path):
-                $this->type = static::TYPE_SASS;
-                break;
-            case 0 !== preg_match('/^.*\.' . static::TYPE_COMPASS . '/', $path):
-                $this->type = static::TYPE_COMPASS;
-                break;
-            case 0 !== preg_match('/^.*\.' . static::TYPE_LESS . '/', $path):
-                $this->type = static::TYPE_LESS;
-                break;
-            default:
-                $this->type = 'unknown';
+        $extension = strtolower(pathinfo($path, PATHINFO_EXTENSION));
+        if (in_array($extension, static::$extensions)) {
+            $this->type = $extension;
+        } else {
+            $this->type = static::TYPE_UNKNOWN;
         }
     }