Form MemberExpression property values.

Author: Ethan Cohen

src/util/getAttributeValue.js

@@ -8,7 +8,7 @@ const getValue = value => {
   } else if (value.type === 'Identifier') {
     return value.name === "" ? undefined : value.name;
   } else if (value.type === 'JSXExpressionContainer') {
-    const expression = value.expression;
+    const { expression } = value;
 
     switch (expression.type) {
       case 'Literal':
@@ -21,7 +21,13 @@ const getValue = value => {
       case 'FunctionExpression':
         return () => void 0;
       case 'LogicalExpression':
-        return getValue(expression.left) && getValue(expression.right);
+        const { operator, left, right } = expression;
+        const leftVal = getValue(left);
+        const rightVal = getValue(right);
+
+        return operator == '&&' ? leftVal && rightVal : leftVal || rightVal;
+      case 'MemberExpression':
+        return `${getValue(expression.object)}.${expression.property}`;
       default:
         return undefined;
     }

tests/src/rules/img-uses-alt.js

@@ -56,6 +56,8 @@ ruleTester.run('img-uses-alt', rule, {
     { code: '<IMG />', parserOptions },
     { code: '<UX.Layout>test</UX.Layout>', parserOptions },
     { code: '<img alt={alt || "Alt text" } />', parserOptions },
+    { code: '<img alt={photo.caption} />;', parserOptions },
+    { code: '<img alt=" " />', parserOptions }, // For decorative images.
 
     // CUSTOM ELEMENT TESTS FOR STRING OPTION
     { code: '<Avatar alt="foo" />;', options: string, parserOptions },