fix comments

Author: AntoxaAntoxic

src/main/java/org/prebid/server/bidder/akcelo/AkceloBidder.java

@@ -54,32 +54,22 @@ public AkceloBidder(String endpointUrl, JacksonMapper mapper) {
 
     @Override
     public Result<List<HttpRequest<BidRequest>>> makeHttpRequests(BidRequest request) {
+        final List<Imp> imps = request.getImp();
         final List<Imp> modifiedImps = new ArrayList<>();
-        final List<BidderError> errors = new ArrayList<>();
-
-        String siteId = null;
 
+        final ExtImpAkcelo firstExtImp;
         try {
-            final List<Imp> imps = request.getImp();
-            for (int i = 0; i < imps.size(); i++) {
-                final Imp imp = imps.get(i);
-                if (i == 0) {
-                    final ExtImpAkcelo extImp = parseImpExt(imp);
-                    siteId = extImp.getSiteId();
-                }
-                modifiedImps.add(modifyImp(imp));
-            }
+            firstExtImp = parseImpExt(imps.getFirst());
         } catch (PreBidException e) {
-            errors.add(BidderError.badInput(e.getMessage()));
+            return Result.withError(BidderError.badInput(e.getMessage()));
         }
 
-        if (modifiedImps.isEmpty()) {
-            return Result.withErrors(errors);
+        for (final Imp imp : imps) {
+            modifiedImps.add(modifyImp(imp));
         }
 
-        final BidRequest outgoingRequest = modifyRequest(request, modifiedImps, siteId);
-        final HttpRequest<BidRequest> httpRequest = BidderUtil.defaultRequest(outgoingRequest, endpointUrl, mapper);
-        return Result.of(Collections.singletonList(httpRequest), errors);
+        final BidRequest outgoingRequest = modifyRequest(request, modifiedImps, firstExtImp.getSiteId());
+        return Result.withValue(BidderUtil.defaultRequest(outgoingRequest, endpointUrl, mapper));
     }
 
     private ExtImpAkcelo parseImpExt(Imp imp) {
@@ -91,12 +81,9 @@ private ExtImpAkcelo parseImpExt(Imp imp) {
     }
 
     private Imp modifyImp(Imp imp) {
-        return Optional.ofNullable(imp.getExt())
-                .map(prebid -> prebid.get("bidder"))
-                .filter(JsonNode::isObject)
-                .map(bidder -> (ObjectNode) mapper.mapper().createObjectNode().set(BIDDER_NAME, bidder))
-                .map(ext -> imp.toBuilder().ext(ext).build())
-                .orElseThrow(() -> new PreBidException("imp.ext.prebid.bidder can't be parsed"));
+        return imp.toBuilder()
+                .ext(mapper.mapper().createObjectNode().set(BIDDER_NAME, imp.getExt().get("bidder")))
+                .build();
     }
 
     private BidRequest modifyRequest(BidRequest request, List<Imp> imps, String siteId) {

src/test/java/org/prebid/server/bidder/akcelo/AkceloBidderTest.java

@@ -63,21 +63,6 @@ public void makeHttpRequestsShouldReturnErrorWhenImpExtCouldNotBeParsed() {
         assertThat(result.getValue()).isEmpty();
     }
 
-    @Test
-    public void makeHttpRequestsShouldNotReturnErrorWhenSiteIdBeParsedInTheSecondImp() {
-        // given
-        final BidRequest bidRequest = givenBidRequest(
-                imp -> imp.id("imp1"),
-                imp -> imp.id("imp2").ext(mapper.valueToTree(ExtPrebid.of(null, mapper.createArrayNode()))));
-
-        // when
-        final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);
-
-        // then
-        assertThat(result.getErrors()).hasSize(1);
-        assertThat(result.getValue()).hasSize(1);
-    }
-
     @Test
     public void makeHttpRequestsShouldCorrectlyModifyRequest() {
         // given