Some more Remote Handler code removed.

MindScriptAct · MindScriptAct · commit 6795d51ab991 · 2012-07-19T15:46:23.000+02:00
diff --git a/mvcExpress-frameWork.as3proj b/mvcExpress-frameWork.as3proj
@@ -78,7 +78,7 @@
   </library>
   <!-- Class files to compile (other referenced classes will automatically be included) -->
   <compileTargets>
-    <compile path="test\MvcExpressTestRunner.as" />
+    <compile path="sampleProjects\com\mindScriptAct\mvcExpressSpeedTest\MvcExpressSpeedTest.as" />
   </compileTargets>
   <!-- Paths to exclude from the Project Explorer tree -->
   <hiddenPaths>
diff --git a/sampleProjects/com/mindScriptAct/modules/counter/CounterModule.as b/sampleProjects/com/mindScriptAct/modules/counter/CounterModule.as
diff --git a/sampleProjects/com/mindScriptAct/modules/counter/view/CounterModuleMediator.as b/sampleProjects/com/mindScriptAct/modules/counter/view/CounterModuleMediator.as
diff --git a/src/org/mvcexpress/MvcExpress.as b/src/org/mvcexpress/MvcExpress.as
@@ -10,11 +10,11 @@ public class MvcExpress {
 	public static const WEBSITE_URL:String = "http://mvcexpress.org";
 	
 	/** Current framework major version */
-	public static const MAJOR_VERSION:uint = 0;
+	public static const MAJOR_VERSION:uint = 1;
 	/** Current framework minor version */
-	public static const MINOR_VERSION:uint = 9;
+	public static const MINOR_VERSION:uint = 0;
 	/** Current framework revision version */
-	public static const REVISION:uint = 9;
+	public static const REVISION:uint = 0;
 	
 	/** Current framework version */
 	public static function get VERSION():String {
@@ -46,7 +46,7 @@ public class MvcExpress {
 	
 	/**
 	 * Flag to force sendMessageToAll function throw an error in debug mode.
-	 * For people who don't what this feature in.
+	 * (For people who don't what this feature in.)
 	 */
 	static public var disableSendToAllFeature:Boolean = false;
 
diff --git a/src/org/mvcexpress/core/CommandMap.as b/src/org/mvcexpress/core/CommandMap.as
@@ -130,13 +130,9 @@ public class CommandMap {
 	}
 	
 	/** function to be called by messenger on needed message type sent */
-	private function handleCommandExecute(messageType:String, params:Object, remoteModule:String):void {
+	private function handleCommandExecute(messageType:String, params:Object):void {
 		var commandList:Vector.<Class>;
-		if (remoteModule) {
-			commandList = classRegistry[messageType + ModuleManager.MESSAGE_MODULE_SEPARATOR + remoteModule];
-		} else {
-			commandList = classRegistry[messageType];
-		}
+		commandList = classRegistry[messageType];
 		
 		if (commandList) {
 			for (var i:int = 0; i < commandList.length; i++) {
diff --git a/src/org/mvcexpress/core/FlexMediatorMap.as b/src/org/mvcexpress/core/FlexMediatorMap.as
@@ -13,6 +13,7 @@ import org.mvcexpress.mvc.Mediator;
  * @author Raimundas Banevicius (http://www.mindscriptact.com/)
  */
 public class FlexMediatorMap extends MediatorMap {
+	
 	private var uiComponentClass:Class;
 	
 	/* CONSTRUCTOR */
diff --git a/src/org/mvcexpress/core/ModuleManager.as b/src/org/mvcexpress/core/ModuleManager.as
@@ -12,7 +12,6 @@ import org.mvcexpress.MvcExpress;
  * @author Raimundas Banevicius (http://www.mindscriptact.com/)
  */
 public class ModuleManager {
-	static public const MESSAGE_MODULE_SEPARATOR:String = "_§_";
 	
 	/* messenger counter, increased with every new created module */
 	static private var _moduleId:int;
diff --git a/src/org/mvcexpress/core/interfaces/IProxyMap.as b/src/org/mvcexpress/core/interfaces/IProxyMap.as
@@ -7,6 +7,5 @@ import org.mvcexpress.mvc.Proxy;
  */
 public interface IProxyMap {
 	function getProxy(injectClass:Class, name:String = ""):Proxy;
-
 }
 }
diff --git a/src/org/mvcexpress/core/messenger/HandlerVO.as b/src/org/mvcexpress/core/messenger/HandlerVO.as
@@ -20,9 +20,6 @@ public class HandlerVO {
 	
 	/** FOR INTERNAL USE ONLY. Variable to store class there handler came from. (for debugging) */
 	public var handlerClassName:String;
-	
-	/** FOR INTERNAL USE ONLY. if set - indicates remote module that added this handler . */
-	public var remoteModule:String;
 
 }
 }
diff --git a/src/org/mvcexpress/core/messenger/Messenger.as b/src/org/mvcexpress/core/messenger/Messenger.as
@@ -131,7 +131,7 @@ public class Messenger {
 					
 					// check if handling function handles commands.
 					if (handlerVo.isExecutable) {
-						handlerVo.handler(type, params, handlerVo.remoteModule);
+						handlerVo.handler(type, params);
 					} else {
 						CONFIG::debug {
 							// FOR DEBUG viewing only(mouse over over variables while in debugger mode.)
@@ -204,18 +204,10 @@ public class Messenger {
 			var messageHandlers:String = "";
 			for (var i:int = 0; i < msgList.length; i++) {
 				var handlerVo:HandlerVO = msgList[i];
-				if (handlerVo.remoteModule) {
-					if (handlerVo.isExecutable) {
-						messageHandlers += "[EXECUTES {" + handlerVo.remoteModule + "} :" + ModuleManager.listModuleMessageCommands(handlerVo.remoteModule, key) + "], ";
-					} else {
-						messageHandlers += "[{" + handlerVo.remoteModule + "}:" + handlerVo.handlerClassName + "], ";
-					}
+				if (handlerVo.isExecutable) {
+					messageHandlers += "[EXECUTES:" + commandMap.listMessageCommands(key) + "], ";
 				} else {
-					if (handlerVo.isExecutable) {
-						messageHandlers += "[EXECUTES:" + commandMap.listMessageCommands(key) + "], ";
-					} else {
-						messageHandlers += "[" + handlerVo.handlerClassName + "], ";
-					}
+					messageHandlers += "[" + handlerVo.handlerClassName + "], ";
 				}
 			}
 			
diff --git a/src/org/mvcexpress/modules/ModuleMovieClip.as b/src/org/mvcexpress/modules/ModuleMovieClip.as
@@ -112,6 +112,15 @@ public class ModuleMovieClip extends MovieClip {
 		moduleBase.sendMessage(type, params);
 	}
 	
+	/**
+	 * Sends message to all existing modules.
+	 * @param	type				message type to find needed handlers
+	 * @param	params				parameter object that will be sent to all handler and execute functions as single parameter.
+	 */
+	protected function sendMessageToAll(type:String, params:Object = null):void {
+		moduleBase.sendMessageToAll(type, params);
+	}
+	
 	//----------------------------------
 	//     Debug
 	//----------------------------------
diff --git a/src/org/mvcexpress/modules/ModuleSprite.as b/src/org/mvcexpress/modules/ModuleSprite.as
@@ -112,6 +112,15 @@ public class ModuleSprite extends Sprite {
 		moduleBase.sendMessage(type, params);
 	}
 	
+	/**
+	 * Sends message to all existing modules.
+	 * @param	type				message type to find needed handlers
+	 * @param	params				parameter object that will be sent to all handler and execute functions as single parameter.
+	 */
+	protected function sendMessageToAll(type:String, params:Object = null):void {
+		moduleBase.sendMessageToAll(type, params);
+	}
+	
 	//----------------------------------
 	//     Debug
 	//----------------------------------
diff --git a/src/org/mvcexpress/mvc/Mediator.as b/src/org/mvcexpress/mvc/Mediator.as
@@ -34,10 +34,6 @@ public class Mediator {
 	/** @private */
 	pureLegsCore var messenger:Messenger;
 	
-	/** @private */
-	CONFIG::debug
-	static pureLegsCore var canConstruct:Boolean;
-	
 	// Allows Mediator to be constructed. (removed from release build to save some performance.)
 	/** @private */
 	pureLegsCore var pendingInjections:int = 0;
@@ -51,6 +47,10 @@ public class Mediator {
 	/** contains array of added event listeners, stored by event listening function as a key. For event useCapture = true*/
 	private var eventListenerCaptureRegistry:Dictionary = new Dictionary(); /* or Dictionary by Function */
 	
+	/** @private */
+	CONFIG::debug
+	static pureLegsCore var canConstruct:Boolean;
+	
 	/** CONSTRUCTOR */
 	public function Mediator() {
 		CONFIG::debug {
diff --git a/src/org/mvcexpress/utils/checkClassStringConstants.as b/src/org/mvcexpress/utils/checkClassStringConstants.as
@@ -41,7 +41,7 @@ public function checkClassStringConstants(... args:Array):void {
 				}
 			}
 		} else {
-			throw Error("Please send Class names to checkClassStringConstants() only(not object).");
+			throw Error("Please send Class names to checkClassStringConstants() only(not object or strings).");
 		}
 	}
 }
diff --git a/src/org/mvcexpress/utils/checkClassSuperclass.as b/src/org/mvcexpress/utils/checkClassSuperclass.as
@@ -3,7 +3,8 @@ import flash.utils.getDefinitionByName;
 import flash.utils.getQualifiedSuperclassName;
 
 /**
- * Checks if provided class is extended from provided class name
+ * Checks if provided class is extended from provided class name.
+ * Will return false if it is of same type as superClassName, or other type that has nothing to do with superClassName.
  * @author Raimundas Banevicius (http://www.mindscriptact.com/)
  */
 

Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,5 @@ import org.mvcexpress.mvc.Proxy;`
`7`	`7`	`*/`
`8`	`8`	`public interface IProxyMap {`
`9`	`9`	`function getProxy(injectClass:Class, name:String = ""):Proxy;`
`10`		`-`
`11`	`10`	`}`
`12`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,9 +20,6 @@ public class HandlerVO {`
`20`	`20`
`21`	`21`	`/** FOR INTERNAL USE ONLY. Variable to store class there handler came from. (for debugging) */`
`22`	`22`	`public var handlerClassName:String;`
`23`		`-`
`24`		`- /** FOR INTERNAL USE ONLY. if set - indicates remote module that added this handler . */`
`25`		`- public var remoteModule:String;`
`26`	`23`
`27`	`24`	`}`
`28`	`25`	`}`