update

Author: saadshams

Sources/PureMVC/interfaces/IFacade.swift

@@ -64,9 +64,9 @@ public protocol IFacade: INotifier {
     Register an `ICommand` with the `Controller`.
 
     - parameter noteName: the name of the `INotification` to associate the `ICommand` with.
-    - parameter closure: reference that returns `ICommand`
+    - parameter factory: closure that returns `ICommand`
     */
-    func registerCommand(_ notificationName: String, closure: @escaping () -> ICommand)
+    func registerCommand(_ notificationName: String, factory: @escaping () -> ICommand)
 
     /**
     Check if a Command is registered for a given Notification

Sources/PureMVC/patterns/facade/Facade.swift

@@ -47,7 +47,7 @@ open class Facade: IFacade {
     This `IFacade` implementation is a Multiton,
     so you should not call the constructor
     directly, but instead call the static Factory method,
-    passing the unique key for this instance and the closure reference
+    passing the unique key for this instance and the factory closure
     that returns the `IFacade` implementation.
     `Facade.getInstance( multitonKey ) { Facade(key: multitonKey) }`
 
@@ -159,10 +159,10 @@ open class Facade: IFacade {
     Register an `ICommand` with the `Controller` by Notification name.
 
     - parameter notificationName: the name of the `INotification` to associate the `ICommand` with
-    - parameter closure: reference that returns `ICommand`
+    - parameter factory: reference that returns `ICommand`
     */
-    open func registerCommand(_ notificationName: String, closure: @escaping () -> ICommand) {
-        controller.registerCommand(notificationName, factory: closure)
+    open func registerCommand(_ notificationName: String, factory: @escaping () -> ICommand) {
+        controller.registerCommand(notificationName, factory: factory)
     }
 
     /**

Sources/PureMVC/patterns/mediator/Mediator.swift

@@ -31,7 +31,7 @@ open class Mediator: Notifier, IMediator {
     /**
     Constructor.
 
-    - parameter mediatorName: the mediator name
+    - parameter name: the mediator name
     - parameter viewComponent: viewComponent instance
     */
     public init(name: String?=nil, viewComponent: AnyObject?=nil) {

Tests/PureMVCTests/patterns/facade/FacadeTest.swift

@@ -52,7 +52,7 @@ class FacadeTest: XCTestCase {
         // Create the Facade, register the FacadeTestCommand to
         // handle 'FacadeTest' notifications
         let facade = Facade.getInstance("FacadeTestKey2", factory: { key in Facade(key: key) })
-        facade.registerCommand("FacadeTestNote", closure: { FacadeTestCommand()} )
+        facade.registerCommand("FacadeTestNote", factory: { FacadeTestCommand()} )
 
         // Send notification. The Command associated with the event
         // (FacadeTestCommand) will be invoked, and will multiply
@@ -80,7 +80,7 @@ class FacadeTest: XCTestCase {
         // Create the Facade, register the FacadeTestCommand to
         // handle 'FacadeTest' events
         let facade = Facade.getInstance("FacadeTestKey3", factory: {key in Facade(key: key)}) as! Facade
-        facade.registerCommand("FacadeTestNote", closure: {FacadeTestCommand()})
+        facade.registerCommand("FacadeTestNote", factory: {FacadeTestCommand()})
         facade.removeCommand("FacadeTestNote")
 
         // Send notification. The Command associated with the event
@@ -200,7 +200,7 @@ class FacadeTest: XCTestCase {
     func testHasCommand() {
         // register the ControllerTestCommand to handle 'hasCommandTest' notes
         let facade = Facade.getInstance("FacadeTestKey10", factory: { key in Facade(key: key) })
-        facade.registerCommand("facadeHasCommandTest", closure: {FacadeTestCommand()})
+        facade.registerCommand("facadeHasCommandTest", factory: {FacadeTestCommand()})
 
         // test that hasCommand returns true for hasCommandTest notifications
         XCTAssertTrue(facade.hasCommand("facadeHasCommandTest"), "Expecting facade.hasCommand('facadeHasCommandTest') == true")

Tests/PureMVCTests/patterns/facade/ShellFacade.swift

@@ -16,7 +16,7 @@ public class ShellFacade: Facade {
 
     public override func initializeController() {
         super.initializeController()
-        registerCommand(ShellFacade.TestCommand, closure: {FacadeTestCommand2()})
+        registerCommand(ShellFacade.TestCommand, factory: {FacadeTestCommand2()})
     }
 
     public func startup(vo: FacadeTestVO) {

Tests/PureMVCTests/patterns/observer/NotifierTest.swift

@@ -30,7 +30,7 @@ class NotifierTest: XCTestCase {
         XCTAssertTrue(Facade.hasCore("notifierTest"), "Expecting Facade.hasCore('notifierTest') == true")
 
         let vo = FacadeTestVO(input: 5)
-        facade.registerCommand("testCommand", closure: {FacadeTestCommand()})
+        facade.registerCommand("testCommand", factory: {FacadeTestCommand()})
 
         let notifier = Notifier()
         notifier.initializeNotifier("notifierTest")