feat: addPropertyMap

Author: tyankatsu0105

lib/rules/no-use-computed-property-like-method.js

@@ -168,6 +168,22 @@ const getValueType = ({ property, propertyMap }) => {
   }
 }
 
+/**
+ * @param {Set<string>} groups
+ * @param {ObjectExpression} vueNodeMap
+ * @param {PropertyMap} propertyMap
+ */
+const addPropertyMap = (groups, vueNodeMap, propertyMap) => {
+  const properties = utils.iterateProperties(vueNodeMap, groups)
+  for (const property of properties) {
+    propertyMap[property.name] = {
+      ...propertyMap[property.name],
+      ...property,
+      valueType: getValueType({ property, propertyMap })
+    }
+  }
+}
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -208,24 +224,10 @@ module.exports = {
         }
       },
 
-      /**
-       * Re-check propertyMap's valueType.
-       * Because there is a possibility that propertyMap's member reference other propertyMap's member.
-       */
-      'MemberExpression[object.type="ThisExpression"]'(node) {
-        const properties = utils.iterateProperties(vueNodeMap, groups)
-
-        for (const property of properties) {
-          propertyMap[property.name] = {
-            ...propertyMap[property.name],
-            ...property,
-            valueType: getValueType({ property, propertyMap })
-          }
-        }
-      },
-
       /** @param {ThisExpression} node */
       'CallExpression > MemberExpression > ThisExpression'(node) {
+        addPropertyMap(groups, vueNodeMap, propertyMap)
+
         if (node.parent.type !== 'MemberExpression') return
         if (node.parent.property.type !== 'Identifier') return