checkParamNames: skip dotted params (as it was initially)

jscs-dev · Dec 1, 2014 · b311704 · b311704
1 parent 5d24286
commit b311704
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/lib/rules/validate-jsdoc/check-param-names.js b/lib/rules/validate-jsdoc/check-param-names.js
@@ -26,6 +26,7 @@ function validateCheckParamNames(node, err) {
         }
     });
     var outOfOrder = {};
+    var skipped = 0;
 
     node.jsdoc.iterateByType(['param', 'arg', 'argument'],
         /**
@@ -39,12 +40,17 @@ function validateCheckParamNames(node, err) {
                 return err('missing param name', tag.loc);
             }
 
-            var param = node.params[i];
+            var param = node.params[i - skipped];
             if (!param) {
                 // skip if no param
                 return;
             }
 
+            if (tag.value.indexOf('.') !== -1) {
+                skipped++;
+                return;
+            }
+
             // checking name
             var msg;
             if (tag.name.value !== param.name && !outOfOrder[tag.name.value]) {

diff --git a/test/lib/rules/validate-jsdoc/check-param-names.js b/test/lib/rules/validate-jsdoc/check-param-names.js
@@ -256,6 +256,20 @@ describe('lib/rules/validate-jsdoc/check-param-names', function() {
                      */
                     function methodThree(required, optional) {}
                 }
+            }, {
+                it: 'should not report dotted param names',
+                code: function() {
+                    /**
+                     * Declares modifier
+                     * @param {Object} mod
+                     * @param {String} mod.modName
+                     * @param {String|Boolean|Array} [mod.modVal]
+                     * @param {Object} props
+                     * @param {Object} [staticProps]
+                     * @returns {Function}
+                     */
+                    function yeah(mod, props, staticProps) {}
+                }
             }
             /* jscs: enable */
             /* jshint ignore:end */