* Fixing removeEvent and fireEvent to behave like dom events. See htt…

…p://mooshell.net/9CX5f/ for more information. This also removes the *ugly* (sorry Scott) way to use unshift for Array.include and makes it possible to use eventsOf everywhere again!

Source/Class/Mixins.js

@@ -23,16 +23,17 @@ var eventsOf = function(object, type){
 };
 
 var removeEventsOfType = function(object, type){
-	var events = eventsOf(object, type);
-	for (var i = events.length; i--; ) object.removeEvent(type, events[i]);
+	eventsOf(object, type).each(function(fn){
+		object.removeEvent(type, fn);
+	});
 };
 
 this.Events = new Class({
 
 	$events: {},
 
 	addEvent: function(type, fn){
-		eventsOf(this, type).include(fn, true);
+		eventsOf(this, type).include(fn);
 		return this;
 	},
 
@@ -43,8 +44,9 @@ this.Events = new Class({
 
 	fireEvent: function(type, args){
 		args = Array.from(args);
-		var events = eventsOf(this, type);
-		for (var i = events.length; i--; ) events[i].apply(this, args);
+		eventsOf(this, type).each(function(fn){
+			fn.apply(this, args);
+		}, this);
 		return this;
 	},
 
@@ -54,7 +56,12 @@ this.Events = new Class({
 	}.overload(Function.overloadList),
 
 	removeEvent: function(type, fn){
-		if (!fn.$protected) eventsOf(this, type).erase(fn);
+		if (!fn.$protected){
+			var events = eventsOf(this, type);
+			var index = events.indexOf(fn);
+			if (index != -1) delete events[index];
+		}
+
 		return this;
 	},
 

Source/Types/Array.js

@@ -80,8 +80,8 @@ Array.implement({
 		return (this.length) ? this[Number.random(0, this.length - 1)] : null;
 	},
 
-	include: function(item, unshift){
-		if (!this.contains(item)) this[unshift ? 'unshift' : 'push'](item);
+	include: function(item){
+		if (!this.contains(item)) this.push(item);
 		return this;
 	},