re-order params of update function in counter.js and elmish.js for clarity see: #44 (comment)

Author: nelsonic

Diff for: elmish.md

@@ -373,7 +373,7 @@ function mount(model, update, view, root_element_id) {
   var root = document.getElementById(root_element_id); // root DOM element
   function signal(action) {                     // signal function takes action
     return function callback() {                // and returns callback
-      var updatedModel = update(model, action); // update model for the action
+      var updatedModel = update(action, model); // update model for the action
       empty(root);                              // clear root el before rerender
       view(signal, updatedModel, root);         // subsequent re-rendering
     };
@@ -1701,7 +1701,7 @@ function mount(model, update, view, root_element_id) {
   var root = document.getElementById(root_element_id); // root DOM element
   function signal(action) {                     // signal function takes action
     return function callback() {                // and returns callback
-      var updatedModel = update(model, action); // update model for the action
+      var updatedModel = update(action, model); // update model for the action
       empty(root);                              // clear root el before rerender
       view(signal, updatedModel, root);         // subsequent re-rendering
     };

Diff for: examples/counter-reset/counter.js

@@ -3,7 +3,7 @@ var Inc = 'inc';                     // increment the counter
 var Dec = 'dec';                     // decrement the counter
 var Res = 'reset';                   // reset counter: git.io/v9KJk
 
-function update(model, action) {     // Update function takes the current state
+function update (action, model) {     // Update function takes the current state
   switch(action) {                   // and an action (String) runs a switch
     case Inc: return model + 1;      // add 1 to the model
     case Dec: return model - 1;      // subtract 1 from model
@@ -27,7 +27,7 @@ function mount(model, update, view, root_element_id) {
   var root = document.getElementById(root_element_id); // root DOM element
   function signal(action) {          // signal function takes action
     return function callback() {     // and returns callback
-      model = update(model, action); // update model according to action
+      model = update(action, model); // update model according to action
       view(signal, model, root);     // subsequent re-rendering
     };
   };

Diff for: test/counter-reset.test.js

@@ -21,20 +21,20 @@ test('Mount app expect state to be Zero', function (t) {
   t.end()
 });
 
-test('Test Update update(0) returns 0 (current state)', function (t) {
-  var result = update(0);
+test('Test Update update("", 0) returns 0 (current state)', function (t) {
+  var result = update('', 0);
   t.equal(result, 0, "Initial state: 0, No Action, Final state: 0");
   t.end();
 });
 
-test('Test Update increment: update(1, "inc") returns 2', function (t) {
-  var result = update(1, "inc");
+test('Test Update increment: update("inc", 1) returns 2', function (t) {
+  var result = update("inc", 1);
   t.equal(result, 2, "Initial state: 1, Increment once, Final state: 2");
   t.end();
 });
 
-test('Test Update decrement: update(3, "dec") returns 2', function (t) {
-  var result = update(1, "dec");
+test('Test Update decrement: update("dec", 1) returns 0', function (t) {
+  var result = update("dec", 1);
   t.equal(result, 0, "Initial state: 1, Decrement once, Final state: 0");
   t.end();
 });
@@ -81,7 +81,7 @@ test('Click reset button resets state to 0', function (t) {
 // Reset Functionality
 
 test('Test reset counter when model/state is 6 returns 0', function(t) {
-  var result = update(6, "reset");
+  var result = update("reset", 6);
   t.equal(result, 0, "counter reset to 0 (Zero)");
   t.end()
 });