Skip to content

Commit f03adf9

Browse files
committed
Angular plugin should match interface for Raven.afterLoad
1 parent e865dc8 commit f03adf9

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

plugins/angular.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,7 @@ function ngRavenExceptionHandler(RavenConfig, $delegate) {
2222
if (!RavenConfig)
2323
throw new Error('RavenConfig must be set before using this');
2424

25-
// Pop off DSN
26-
var DSN = RavenConfig.DSN;
27-
delete RavenConfig.DSN;
28-
29-
Raven.config(DSN, RavenConfig).install();
25+
Raven.config(RavenConfig.dsn, RavenConfig.config).install();
3026
return function angularExceptionHandler(ex, cause) {
3127
$delegate(ex, cause);
3228
Raven.captureException(ex, {extra: {cause: cause}});

src/raven.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,9 @@ var Raven = {
3737
* @return undefined
3838
*/
3939
afterLoad: function() {
40-
var globalConfig = window.RavenConfig;
41-
if (globalConfig) {
42-
this.config(globalConfig.dsn, globalConfig.config).install();
40+
var RavenConfig = window.RavenConfig;
41+
if (RavenConfig) {
42+
Raven.config(RavenConfig.dsn, RavenConfig.config).install();
4343
}
4444
},
4545

0 commit comments

Comments
 (0)