Merge pull request #10068 from Mark-Simulacrum/hotfix-codeinspection

MiguelCastillo · MiguelCastillo · commit 075e05ca2228 · 2014-12-02T23:51:23.000-05:00
Fixes the exception thrown when the provider list for a given language is undefined.
diff --git a/src/language/CodeInspection.js b/src/language/CodeInspection.js
@@ -161,7 +161,8 @@ define(function (require, exports, module) {
      * @return ?{Array.<{name:string, scanFileAsync:?function(string, string):!{$.Promise}, scanFile:?function(string, string):?{errors:!Array, aborted:boolean}}>} provider
      */
     function getProvidersForPath(filePath) {
-        return _providers[LanguageManager.getLanguageForPath(filePath).getId()].slice(0) || [];
+        var providers = _providers[LanguageManager.getLanguageForPath(filePath).getId()];
+        return (providers && providers.slice(0)) || [];
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,8 @@ define(function (require, exports, module) {`
`161`	`161`	`* @return ?{Array.<{name:string, scanFileAsync:?function(string, string):!{$.Promise}, scanFile:?function(string, string):?{errors:!Array, aborted:boolean}}>} provider`
`162`	`162`	`*/`
`163`	`163`	`function getProvidersForPath(filePath) {`
`164`		`- return _providers[LanguageManager.getLanguageForPath(filePath).getId()].slice(0) \|\| [];`
	`164`	`+ var providers = _providers[LanguageManager.getLanguageForPath(filePath).getId()];`
	`165`	`+ return (providers && providers.slice(0)) \|\| [];`
`165`	`166`	`}`
`166`	`167`
`167`	`168`	`/**`