Guild Role Update - Added mentionable to old role

[motoenduroboy]

-Added mentionable to oldRole on event GUILD_ROLE_UPDATE

[Brayzure]

Just some general commentary, but why don't we just pass the old role as-is? Why do we create another vanilla object, and copy properties over?

[motoenduroboy]

Probably a good idea unless @abalabahaha has a reason behind it?

[abalabahaha]

update() updates the roles in place so you would be getting two of the same object...

var oldRole = {name: "foo"};

var newRole = oldRole; // Both pointing to the same object
(function update(role, data) { // Over-simplified update()
    role.name = data.name;
})(newRole, {name: "bar"});

emit("roleUpdate", newRole, oldRole); // ("roleUpdate", {name: "bar"}, {name: "bar"})