8000 Doesn't work with Require.js · Issue #72 · f/delorean · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Doesn't work with Require.js #72
Open
@bradleyayers

Description

@bradleyayers

The bower distributed delorean does not work with Require.js

First problem is require('events').EventEmitter:

module.exports = requirements = {
  // DeLorean uses **Node.js native EventEmitter** for event emittion
  EventEmitter: require('events').EventEmitter,
  // and **es6-promise** for Deferred object management.
  Promise: require('es6-promise').Promise
};

But the module is defined as:

module.exports = EventEmitter

So I think in the injector it should just be require('event') right?

Second problem is in the injector:

// This library needs to work for Browserify and also standalone.
// If DeLorean is defined, it means it's called from the browser, not
// the browserify.
if (typeof DeLorean !== 'undefined') {
  for (var requirement in requirements) {
    DeLorean.Flux.define(requirement, requirements[requirement]);
  }
}

But when used in the browser with Require.js, DeLorean is undefined because:

if (typeof define === 'function' && define.amd) {
  define([], function () {
    return DeLorean;
  });
} else {
  window.DeLorean = DeLorean;
}

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0