Fixes after review

HlebHalkouski · HlebHalkouski · commit 0cf1f9e4d6bd · 2017-07-28T15:16:04.000+03:00
diff --git a/src/main/java/io/appium/java_client/pagefactory/bys/builder/ByAll.java b/src/main/java/io/appium/java_client/pagefactory/bys/builder/ByAll.java
@@ -45,8 +45,7 @@ public WebElement findElement(SearchContext context) {
         return bys.stream()
                 .map(by -> getSearchingFunction(by).apply(context))
                 .filter(Optional::isPresent)
-                .findFirst()
-                .orElseThrow(() -> new NoSuchElementException("Cannot locate an element using " + toString()))
-                .orElse(null);
+                .findFirst().map(Optional::get)
+                .orElseThrow(() -> new NoSuchElementException("Cannot locate an element using " + toString()));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -45,8 +45,7 @@ public WebElement findElement(SearchContext context) {`
`45`	`45`	`return bys.stream()`
`46`	`46`	`.map(by -> getSearchingFunction(by).apply(context))`
`47`	`47`	`.filter(Optional::isPresent)`
`48`		`- .findFirst()`
`49`		`- .orElseThrow(() -> new NoSuchElementException("Cannot locate an element using " + toString()))`
`50`		`- .orElse(null);`
	`48`	`+ .findFirst().map(Optional::get)`
	`49`	`+ .orElseThrow(() -> new NoSuchElementException("Cannot locate an element using " + toString()));`
`51`	`50`	`}`
`52`	`51`	`}`