Use .some() instead of .find()

jseminck · jseminck · commit 322afa4c933b · 2017-07-13T07:34:49.000+03:00
A hasXXX function should return a boolean instead of an object.
I think node 4 supports .some(). - if not, will revert.
diff --git a/lib/rules/jsx-no-target-blank.js b/lib/rules/jsx-no-target-blank.js
@@ -15,7 +15,7 @@ function isTargetBlank(attr) {
 }
 
 function hasExternalLink(element) {
-  return element.attributes.find(function (attr) {
+  return element.attributes.some(function (attr) {
     return attr.name &&
       attr.name.name === 'href' &&
       attr.value.type === 'Literal' &&

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ function isTargetBlank(attr) {`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`function hasExternalLink(element) {`
`18`		`- return element.attributes.find(function (attr) {`
	`18`	`+ return element.attributes.some(function (attr) {`
`19`	`19`	`return attr.name &&`
`20`	`20`	`attr.name.name === 'href' &&`
`21`	`21`	`attr.value.type === 'Literal' &&`