[CLOSED] add check to all public constructors to ensure they are called with 'new'

[core-ai-bot]

Issue by joelrbrandt
Friday Jan 06, 2012 at 23:30 GMT
Originally opened as adobe/brackets#48

---

All public constructors should verify that they were called with 'new'. If they were not, they should throw an Error.

It should look something like this:

function Waffle(diameter) {
    if (!(this instanceof Waffle)) { // ERROR: constructor called without 'new'
        var e = new Error("Waffle constructor must be called with 'new'");
        e.detail = { waffleSizeDesired: diameter };
        throw e;
    }
    this.diameter = diameter;
    this.tastes = "yummy";
}

[core-ai-bot]

Comment by jlondon
Monday Mar 05, 2012 at 20:10 GMT

---

This should part of user story for extensibility research