From 9356230e747a8694901a449a3921c90aea79578d Mon Sep 17 00:00:00 2001
From: meowtec <bertonzh@gmail.com>
Date: Wed, 23 Mar 2022 19:03:58 +0800
Subject: [PATCH] [Fix] `button-has-type`: fix exception for `<button type>`

---
 CHANGELOG.md                       | 2 ++
 lib/rules/button-has-type.js       | 2 +-
 tests/lib/rules/button-has-type.js | 9 +++++++++
 3 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4d6e1d11d8..416ca2d677 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,6 +19,7 @@ This change log adheres to standards from [Keep a CHANGELOG](https://keepachange
 * [`no-unused-state`]: avoid a crash ([#3258][] @WillyLiaoWH @ljharb)
 * [`jsx-no-useless-fragment`]: use proper apostrophe in error message ([#3266][] @develohpanda)
 * `propTypes`: handle imported types/interface in forwardRef generic ([#3280][] @vedadeepta)
+* [`button-has-type`]: fix exception for `<button type>` ([#3255][] @meowtec)
 
 ### Changed
 * [readme] remove global usage and eslint version from readme ([#3254][] @aladdin-add)
@@ -48,6 +49,7 @@ This change log adheres to standards from [Keep a CHANGELOG](https://keepachange
 [#3260]: https://github.jsx-eslintckcr/eslint-plugin-react/pull/3260
 [#3259]: https://githubjsx-eslintickcr/eslint-plugin-react/pull/3259
 [#3258]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3258
+[#3255]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3255
 [#3254]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3254
 [#3251]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3251
 [#3249]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3249
diff --git a/lib/rules/button-has-type.js b/lib/rules/button-has-type.js
index add42d69ca..277f6c2a31 100644
--- a/lib/rules/button-has-type.js
+++ b/lib/rules/button-has-type.js
@@ -126,7 +126,7 @@ module.exports = {
           return;
         }
 
-        if (typeProp.value.type === 'JSXExpressionContainer') {
+        if (typeProp.value && typeProp.value.type === 'JSXExpressionContainer') {
           checkExpression(node, typeProp.value.expression);
           return;
         }
diff --git a/tests/lib/rules/button-has-type.js b/tests/lib/rules/button-has-type.js
index 8c7ab73955..be47c7e51e 100644
--- a/tests/lib/rules/button-has-type.js
+++ b/tests/lib/rules/button-has-type.js
@@ -179,6 +179,15 @@ ruleTester.run('button-has-type', rule, {
         },
       ],
     },
+    {
+      code: '<button type/>',
+      errors: [
+        {
+          messageId: 'invalidValue',
+          data: { value: true },
+        },
+      ],
+    },
     {
       code: '<button type={condition ? "reset" : "button"}/>',
       options: [{ reset: false }],