MenuItemSprite's automatic init logic for sprite is incorrect

[pandamicro]

This is the ctor function:

ctor: function (normalSprite, selectedSprite, three, four, five) {
    cc.MenuItem.prototype.ctor.call(this);
    this._normalImage = null;
    this._selectedImage = null;
    this._disabledImage = null;

    if (selectedSprite !== undefined) {
        normalSprite = normalSprite;
        selectedSprite = selectedSprite;
        var disabledImage, target, callback;
        //when you send 4 arguments, five is undefined
        if (five !== undefined) {
            disabledImage = three;
            callback = four;
            target = five;
        } else if (four !== undefined && cc.isFunction(four)) {
            disabledImage = three;
            callback = four;
        } else if (four !== undefined && cc.isFunction(three)) {
            target = four;
            callback = three;
            disabledImage = new cc.Sprite(selectedSprite);
        } else if (three === undefined) {
            disabledImage = new cc.Sprite(selectedSprite);
        }
        this.initWithNormalSprite(normalSprite, selectedSprite, disabledImage, callback, target);
    }
},

It use a sprite to initialize another sprite which will fail

[pandamicro]

closed via cocos2d/cocos2d-html5#2481