marionettejs/backbone.marionette

Better error messages

Open

#2795 aperta il 28 ott 2015

Vedi su GitHub
 (9 commenti) (0 reazioni) (0 assegnatari)JavaScript (1294 fork)batch import
discussionhelp wanted

Metriche repository

Star
 (7085 star)
Metriche merge PR
 (Merge medio 34m) (1 PR mergiata in 30 g)

Descrizione

When I'm using Marionette, I'll occasionally make errors such as accessing a region that doesn't exist e.g.

var MyView = Marionette.LayoutView.extend({
  regions: {
    main: '.main-hook'
  },

  onRender: function() {
    this.showChildView('layout', someView());
  }
});

var view = new MyView();
view.getChildView('layout');

The exception I get comes from inside Marionette as a Cannot read property currentView of undefined or this.getRegion(..) is undefined. While I can chase the stacktrace back to find out the exact issue, it would be much better if I could get the following error:

LayoutError: 'layout' is not a region on MyView. The available regions are: 'main'

This is just to illustrate my point - if we have the information on why something isn't going to work we should tell the developer to save them time.

Guida contributor