From c8795c0b87366ab0dae72adff64652022dccad07 Mon Sep 17 00:00:00 2001
From: Wojtek Zieba <wojtek.zieba@automattic.com>
Date: Fri, 20 Oct 2023 12:25:38 +0200
Subject: [PATCH] refactor: change result type of `ParselyAPIConnection` to
 `Void`

There's no need to return `HttpURLConnection`. This change simplifies implementation.
---
 .../parsely/parselyandroid/ParselyAPIConnection.java   | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/parsely/src/main/java/com/parsely/parselyandroid/ParselyAPIConnection.java b/parsely/src/main/java/com/parsely/parselyandroid/ParselyAPIConnection.java
index b9908192..992fb973 100644
--- a/parsely/src/main/java/com/parsely/parselyandroid/ParselyAPIConnection.java
+++ b/parsely/src/main/java/com/parsely/parselyandroid/ParselyAPIConnection.java
@@ -24,7 +24,7 @@
 import java.net.URL;
 import java.net.HttpURLConnection;
 
-class ParselyAPIConnection extends AsyncTask<String, Exception, HttpURLConnection> {
+class ParselyAPIConnection extends AsyncTask<String, Exception, Void> {
 
     @NonNull
     private final ParselyTracker tracker;
@@ -35,7 +35,7 @@ class ParselyAPIConnection extends AsyncTask<String, Exception, HttpURLConnectio
     }
 
     @Override
-    protected HttpURLConnection doInBackground(String... data) {
+    protected Void doInBackground(String... data) {
         HttpURLConnection connection = null;
         try {
             if (data.length == 1) {  // non-batched (since no post data is included)
@@ -54,12 +54,12 @@ protected HttpURLConnection doInBackground(String... data) {
 
         } catch (Exception ex) {
             this.exception = ex;
-            return null;
         }
-        return connection;
+        return null;
     }
 
-    protected void onPostExecute(HttpURLConnection conn) {
+    @Override
+    protected void onPostExecute(Void result) {
         if (this.exception != null) {
             ParselyTracker.PLog("Pixel request exception");
             ParselyTracker.PLog(this.exception.toString());