Fix typos and readability. Remove debug leftovers.

README.md

@@ -54,7 +54,7 @@ or by [downloading the latest release](https://github.com/ericelliott/stampit/re
 
  * Create factory functions (called stamps) which stamp out new objects. All of the new objects inherit all of the prescribed behavior.
 
- * Assign properties by passing references object to the stamp (factory function).
+ * Assign properties by passing a references object to the stamp (factory function).
 
  * Compose stamps together to create new stamps.
 
@@ -396,15 +396,15 @@ components that are passed in or composed.
  * @param  {Object} [options.refs] A map of property names and values to be mixed into each new object.
  * @param  {Object} [options.init] A closure (function) used to create private data and privileged methods.
  * @param  {Object} [options.props] An object to be deeply cloned into each newly stamped object.
- * @return {Function(refs)} factory A factory to produce objects.
- * @return {Function(refs)} factory.create Just like calling the factory function.
+ * @return {Function} factory A factory to produce objects.
+ * @return {Function} factory.create Just like calling the factory function.
  * @return {Object} factory.fixed An object map containing the stamp metadata.
- * @return {Function(methods)} factory.methods Add methods to the stamp. Chainable.
- * @return {Function(refs)} factory.refs Add references to the stamp. Chainable.
- * @return {Function(Function(context))} factory.init Add a closure which called on object instantiation. Chainable.
- * @return {Function(props)} factory.props Add deeply cloned properties to the produced objects. Chainable.
- * @return {Function(stamps} factory.compose Add stamp to stamp. Chainable.
- * @return {Function(statics)} factory.static Add properties to the factory object. Chainable.
+ * @return {Function} factory.methods Add methods to the stamp. Chainable.
+ * @return {Function} factory.refs Add references to the stamp. Chainable.
+ * @return {Function} factory.init Add a closure which called on object instantiation. Chainable.
+ * @return {Function} factory.props Add deeply cloned properties to the produced objects. Chainable.
+ * @return {Function} factory.compose Add stamp to stamp. Chainable.
+ * @return {Function} factory.static Add properties to the factory object. Chainable.
 
 
 ## The stamp object ##
@@ -469,7 +469,7 @@ MyStamp.create().originalStamp === MyStamp; // true
 ### stamp.props() ###
 
 Take n objects and deep merge them safely to the properties. Creates new stamp.
-Note: the merge algorythm will not change any existing `refs` data of a resulting object instance.
+Note: the merge algorithm will not change any existing `refs` data of a resulting object instance.
 * @return {Object} stamp  The new stamp based on the original `this` stamp.
 
 
@@ -488,7 +488,7 @@ Alias to `stamp([properties] [,arg2] [,arg3...])`.
 Just like calling `stamp()`, `stamp.create()` invokes the stamp
 and returns a new object instance. The first argument is an object
 containing properties you wish to set on the new objects.
-The properties are copied by reference using standard mixin/extend/assign algorythm.
+The properties are copied by reference using standard mixin/extend/assign algorithm.
 
 The remaining arguments are passed to all `.init()`
 functions. **WARNING** Avoid using two different `.init()`

mixer.js

@@ -27,27 +27,23 @@ var mixer = function (opts) {
     var loop = opts.chain ? forIn : forOwn;
     var i = 0,
       n = arguments.length,
-      obj;
-    target = opts.getTarget ? opts.getTarget(target) : target;
+      obj,
+      filter = opts.filter,
+      getValue = opts.getValue,
+      mergeValue = function mergeValue(val, key) {
+        if (filter && !filter(val, target[key])) {
+          return;
+        }
 
-    var mergeValue = function mergeValue(val, key) {
-      if (opts.filter && !opts.filter(val, this[key])) {
-        return;
-      }
+        target[key] = getValue ? getValue(val, target[key]) : val;
+      };
 
-      this[key] = opts.getValue ? opts.getValue(val, this[key]) : val;
-      if (this[key] === 'should not be merged') {
-        console.log('got ya');
-      }
-    };
+    target = opts.getTarget ? opts.getTarget(target) : target;
 
     while (++i < n) {
       obj = arguments[i];
       if (obj) {
-        loop(
-          obj,
-          mergeValue,
-          target);
+        loop(obj, mergeValue);
       }
     }
     return target;
@@ -108,7 +104,7 @@ var mergeUnique = mixer({
 });
 
 function mergeUniqueSourceToTarget(srcVal, targetVal) {
-  if ((isObject(srcVal) && isObject(targetVal))) {
+  if (isObject(srcVal) && isObject(targetVal)) {
     // inception, deep merge objects
     return mergeUnique(targetVal, srcVal);
   } else {
@@ -136,7 +132,7 @@ module.exports.mergeChainNonFunctions = mixer({
 /**
  * merge unique properties of objects including prototype chain properties.
  */
-module.exports.mergeChainNonFunctions = mixer({
+module.exports.mergeUniqueChainNonFunctions = mixer({
   filter: function (val) { return !isFunction(val); },
   getTarget: deepClone,
   getValue: mergeUniqueSourceToTarget,