marionettejs/backbone.marionette

Better error messages

Open

#2,795 创建于 2015年10月28日

在 GitHub 查看
 (9 评论) (0 反应) (0 负责人)JavaScript (1,294 fork)batch import
discussionhelp wanted

仓库指标

Star
 (7,085 star)
PR 合并指标
 (平均合并 34分钟) (30 天内合并 1 个 PR)

描述

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.

贡献者指南