Skip to content

Commit

Permalink
Load babel/polyfill correctly
Browse files Browse the repository at this point in the history
We need to load this since the babel runtime can’t detect when an
instance method is being used that needs to be polyfilled. So we have
to require the polyfill once, and move the `babel` dependency out of
devDependencies and into production. Closes #11
  • Loading branch information
ethanresnick committed Apr 10, 2015
1 parent 2461e9a commit a3fb68f
Show file tree
Hide file tree
Showing 6 changed files with 3 additions and 9 deletions.
2 changes: 0 additions & 2 deletions build/src/adapters/Mongoose/MongooseAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,6 @@ var LinkObject = _interopRequire(require("../../types/LinkObject"));

var APIError = _interopRequire(require("../../types/APIError"));

var polyfill = _interopRequire(require("babel/polyfill"));

var MongooseAdapter = (function () {
function MongooseAdapter(models, inflector, idGenerator) {
_classCallCheck(this, MongooseAdapter);
Expand Down
4 changes: 0 additions & 4 deletions build/src/steps/do-query/do-get.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,6 @@ var APIError = _interopRequire(require("../../types/APIError"));

var arrayContains = require("../../util/arrays").arrayContains;

var polyfill = _interopRequire(require("babel/polyfill"));

//needed for string instance methods.

module.exports = function (requestContext, responseContext, registry) {
var type = requestContext.type;
var adapter = registry.adapter(type);
Expand Down
2 changes: 2 additions & 0 deletions index.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
require('babel/polyfill');

module.exports = {
adapters: {
Mongoose: require('./build/src/adapters/Mongoose/MongooseAdapter')
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
},
"main": "index.js",
"dependencies": {
"babel": "4.7.x",
"babel-runtime": "4.7.x",
"express": "4.x.x",
"co": "4.5.x",
Expand All @@ -26,7 +27,6 @@
"negotiator": "0.5.x"
},
"devDependencies": {
"babel": "4.7.x",
"babel-eslint": "2.x.x",
"eslint": "^0.x.x",
"chai": "~1.9.0",
Expand Down
1 change: 0 additions & 1 deletion src/adapters/Mongoose/MongooseAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import Collection from "../../types/Collection";
import Linkage from "../../types/Linkage";
import LinkObject from "../../types/LinkObject";
import APIError from "../../types/APIError";
import polyfill from "babel/polyfill";

export default class MongooseAdapter {
constructor(models, inflector, idGenerator) {
Expand Down
1 change: 0 additions & 1 deletion src/steps/do-query/do-get.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import APIError from "../../types/APIError";
import {arrayContains} from "../../util/arrays";
import polyfill from "babel/polyfill"; //needed for string instance methods.

export default function(requestContext, responseContext, registry) {
let type = requestContext.type;
Expand Down

0 comments on commit a3fb68f

Please sign in to comment.