documentation and variable name improvements. Thanks @rreusser

Author: bpostlethwaite

src/plots/plots.js

@@ -1044,16 +1044,16 @@ plots.supplyTransformDefaults = function(traceIn, traceOut, layout) {
          * and adds the _module to any output transforms.
          * If transforms exist another pass is run so that any generated traces also
          * go through supply defaults. This has the effect of rerunning
-         * supplyTransforms. If the transform does not have a `transform` function
-         * it could not have generated any new traces and the second stage is
-         * unnecessary. We detect this case with the following variables.
+         * supplyTransformDefaults. If the transform does not have a `transform`
+         * function it could not have generated any new traces and the second stage
+         * is unnecessary. We detect this case with the following variables.
          */
-        var isSecondStage = transformIn._module && transformIn._module === _module,
-            doSecondStage = _module && typeof _module.transform === 'function';
+        var isFirstStage = !(transformIn._module && transformIn._module === _module),
+            doLaterStages = _module && typeof _module.transform === 'function';
 
         if(!_module) Lib.warn('Unrecognized transform type ' + type + '.');
 
-        if(_module && _module.supplyDefaults && (!isSecondStage || doSecondStage)) {
+        if(_module && _module.supplyDefaults && (isFirstStage || doLaterStages)) {
             transformOut = _module.supplyDefaults(transformIn, traceOut, layout, traceIn);
             transformOut.type = type;
             transformOut._module = _module;