marionettejs/backbone.marionette

Better error messages

Open

#2 795 ouverte le 28 oct. 2015

Voir sur GitHub
 (9 commentaires) (0 réactions) (0 assignés)JavaScript (1 294 forks)batch import
discussionhelp wanted

Métriques du dépôt

Stars
 (7 085 stars)
Métriques de merge PR
 (Merge moyen 34m) (1 PR mergée en 30 j)

Description

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.

Guide contributeur