Fixed according to comments and added tests for new cases

Author: zaggino

src/file/FileUtils.js

@@ -332,17 +332,16 @@ define(function (require, exports, module) {
             parts.shift();
         }
 
-        // test all other parts of the baseName
-        while (parts.length > 1) {
-            var ext = parts.join(".");
-            if (LanguageManager.getLanguageForExtension(ext)) {
-                return ext;
+        var extension = [parts.pop()], // last part is always an extension
+            i = parts.length;
+        while (i--) {
+            if (LanguageManager.getLanguageForExtension(parts[i])) {
+                extension.unshift(parts[i]);
             } else {
-                parts.shift();
+                break;
             }
         }
-
-        return parts[0] || "";
+        return extension.join(".");
     }
 
     /**

test/spec/FileUtils-test.js

@@ -158,6 +158,12 @@ define(function (require, exports, module) {
                 expect(FileUtils.getSmartFileExtension("C:/foo/bar/.baz/jaz.scss.erb")).toBe("scss.erb");
                 expect(FileUtils.getSmartFileExtension("foo/bar/baz/.jaz.js.erb")).toBe("js.erb");
             });
+
+            it("should return the extension combined from other known extensions", function () {
+                expect(FileUtils.getSmartFileExtension("foo.bar.php.js")).toBe("php.js");
+                expect(FileUtils.getSmartFileExtension("foo.bar.php.html.js")).toBe("php.html.js");
+                expect(FileUtils.getSmartFileExtension("foo.bar.php.scss.erb")).toBe("php.scss.erb");
+            });
         });
     });
 });