Patch map.fire to format arguments passed to it.

Ref #766

Author: tristen

README.md

@@ -84,7 +84,7 @@ Install dependencies, build the source files and crank up a server via:
 ```
 git clone [email protected]:mapbox/mapbox-gl-draw.git
 npm install
-npm start & open http://localhost:9966/debug/?access_token=<token>
+npm start & open http://localhost:9967/debug/?access_token=<token>
 ```
 
 ### Testing

package.json

@@ -56,8 +56,8 @@
     "eslint": "^4.2.0",
     "eslint-config-mourner": "^2.0.1",
     "express": "^4.13.4",
-    "mapbox-gl": "0.41.0",
-    "mapbox-gl-js-mock": "^0.28.0",
+    "mapbox-gl": "0.45.0-beta.1",
+    "mapbox-gl-js-mock": "^1.0.2",
     "mock-browser": "^0.92.10",
     "nyc": "^11.0.2",
     "opener": "^1.4.1",

src/setup.js

@@ -2,6 +2,7 @@ const events = require('./events');
 const Store = require('./store');
 const ui = require('./ui');
 const Constants = require('./constants');
+const xtend = require('xtend');
 
 module.exports = function(ctx) {
 
@@ -35,6 +36,20 @@ module.exports = function(ctx) {
       ctx.events.addEventListeners();
     },
     onAdd: function(map) {
+
+      // Monkey patch to resolve breaking change to `fire` introduced by
+      // mapbox-gl-js. See mapbox/mapbox-gl-draw/issues/766.
+      const _fire = map.fire;
+      map.fire = function(type, event) {
+        let args = arguments;
+
+        if (_fire.length === 1 && arguments.length !== 1) {
+          args = [xtend({}, { type: type }, event)];
+        }
+
+        return _fire.apply(map, args);
+      };
+
       ctx.map = map;
       ctx.events = events(ctx);
       ctx.ui = ui(ctx);