Merge pull request #96 from guilhermef/master

Included a RavenCallback global that allows loaded callbacks

Author: mattrobenolt

plugins/console.js

@@ -5,14 +5,23 @@ var originalConsole = console,
     logLevels = ['debug', 'info', 'warn', 'error'],
     level;
 
-while(level = logLevels.pop()) {
-    console[level] = function () {
+var logForGivenLevel = function(level) {
+    return function () {
         var args = [].slice.call(arguments);
         Raven.captureMessage('' + args, {level: level, logger: 'console'});
 
         // this fails for some browsers. :(
-        originalConsole[level] && originalConsole[level].apply(null, args);
+        if (originalConsole[level]) {
+             originalConsole[level].apply(null, args);
+        }
     };
+};
+
+
+level = logLevels.pop();
+while(level) {
+    console[level] = logForGivenLevel(level);
+    level = logLevels.pop();
 }
 // export
 window.console = console;

src/raven.js

@@ -30,6 +30,19 @@ TK.remoteFetching = false;
 var Raven = {
     VERSION: '@VERSION',
 
+    /*
+     * Allow Raven to be configured as soon as it is loaded
+     * It uses a global RavenConfig = {dsn: '...', config: {}}
+     *
+     * @return undefined
+     */
+    afterLoad: function() {
+        var globalConfig = window.RavenConfig;
+        if (globalConfig) {
+            this.config(globalConfig.dsn, globalConfig.config).install();
+        }
+    },
+
     /*
      * Allow multiple versions of Raven to be installed.
      * Strip Raven from the global context and returns the instance.
@@ -496,3 +509,5 @@ function joinRegExp(patterns) {
     while (i--) sources[i] = patterns[i].source;
     return new RegExp(sources.join('|'), 'i');
 }
+
+Raven.afterLoad();

test/raven.test.js

@@ -731,6 +731,29 @@ describe('Raven (public API)', function() {
     });
   });
 
+  describe('callback function', function() {
+    it('should callback a function if it is global', function() {
+      window.RavenConfig = {
+        dsn: "http://[email protected]:80/2",
+        config: {some: 'config'}
+      };
+
+      this.sinon.stub(window, 'isSetup').returns(false);
+      this.sinon.stub(TK.report, 'subscribe');
+
+      Raven.afterLoad();
+
+      assert.equal(globalKey, 'random');
+      assert.equal(globalServer, 'http://some.other.server:80/api/2/store/');
+      assert.deepEqual(globalOptions.ignoreErrors, ['Script error.'], 'it should install "Script error." by default');
+      assert.equal(globalOptions.some, 'config');
+      assert.equal(globalProject, 2);
+
+      assert.isTrue(window.isSetup.calledOnce);
+      assert.isFalse(TK.report.subscribe.calledOnce);
+    });
+  });
+
   describe('.config', function() {
     it('should work with a DSN', function() {
       assert.equal(Raven, Raven.config(SENTRY_DSN, {foo: 'bar'}), 'it should return Raven');