bug fix for thread check

Author: pknotfound

OGParser/src/main/java/com/kedia/ogparser/OpenGraphParser.kt

@@ -26,14 +26,14 @@ class OpenGraphParser(
     private val OG_SITE_NAME = "og:site_name"
     private val OG_TYPE = "og:type"
 
-    private val openGraphResult = OpenGraphResult()
+    private var openGraphResult: OpenGraphResult? = null
 
     fun parse(url: String) {
         this.url = url
 
-        GlobalScope.launch {
+        GlobalScope.launch(Dispatchers.Main) {
             val result = fetchContent()
-            listener.onPostResponse(result)
+            result?.let { listener.onPostResponse(it) }
         }
 
     }
@@ -55,29 +55,30 @@ class OpenGraphParser(
             val doc = response.parse()
 
             val ogTags = doc.select(DOC_SELECT_QUERY)
+            openGraphResult = OpenGraphResult()
             when {
                 ogTags.size > 0 ->
                     ogTags.forEachIndexed { index, _ ->
                         val tag = ogTags[index]
                         val text = tag.attr(PROPERTY)
                         when (text) {
                             OG_IMAGE -> {
-                                openGraphResult.image = (tag.attr(OPEN_GRAPH_KEY))
+                                openGraphResult!!.image = (tag.attr(OPEN_GRAPH_KEY))
                             }
                             OG_DESCRIPTION -> {
-                                openGraphResult.description = (tag.attr(OPEN_GRAPH_KEY))
+                                openGraphResult!!.description = (tag.attr(OPEN_GRAPH_KEY))
                             }
                             OG_URL -> {
-                                openGraphResult.url = (tag.attr(OPEN_GRAPH_KEY))
+                                openGraphResult!!.url = (tag.attr(OPEN_GRAPH_KEY))
                             }
                             OG_TITLE -> {
-                                openGraphResult.title = (tag.attr(OPEN_GRAPH_KEY))
+                                openGraphResult!!.title = (tag.attr(OPEN_GRAPH_KEY))
                             }
                             OG_SITE_NAME -> {
-                                openGraphResult.siteName = (tag.attr(OPEN_GRAPH_KEY))
+                                openGraphResult!!.siteName = (tag.attr(OPEN_GRAPH_KEY))
                             }
                             OG_TYPE -> {
-                                openGraphResult.type = (tag.attr(OPEN_GRAPH_KEY))
+                                openGraphResult!!.type = (tag.attr(OPEN_GRAPH_KEY))
                             }
                         }
                     }

app/src/main/java/com/kedia/opengraphpreview/MainActivity.kt

@@ -2,6 +2,7 @@ package com.kedia.opengraphpreview
 
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
+import android.os.Looper
 import android.util.Log
 import com.kedia.ogparser.OpenGraphCallback
 import com.kedia.ogparser.OpenGraphParser
@@ -17,10 +18,10 @@ class MainActivity : AppCompatActivity(), OpenGraphCallback {
     }
 
     override fun onPostResponse(openGraphResult: OpenGraphResult) {
-        Log.d("TAG", "onPostResponse: called $openGraphResult")
+        Log.d("TAG!!!!", "onPostResponse: called $openGraphResult")
     }
 
     override fun onError(error: String) {
-        Log.d("TAG", "$error")
+        Log.e("TAG!!!!", "$error")
     }
 }

app/src/main/res/layout/activity_main.xml

@@ -7,6 +7,7 @@
     tools:context=".MainActivity">
 
     <TextView
+        android:id="@+id/tview"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:text="Hello World!"