Log the content of the editor in case HeadingHandler.handleNewlineInBody has start > end

Author: daniloercoli

aztec/src/main/kotlin/org/wordpress/aztec/AztecExceptionHandler.kt

@@ -33,7 +33,7 @@ class AztecExceptionHandler(private val logHelper: ExceptionHandlerHelper?, priv
                 AppLog.e(AppLog.T.EDITOR, "HTML Content of Aztec Editor before the crash " + visualEditor.toPlainHtml(false))
             } catch (e: Throwable) {
                 AppLog.e(AppLog.T.EDITOR, "HTML Content of Aztec Editor before the crash is unavailable, log the details instead")
-                AztecLog.logEditorContentDetails(visualEditor)
+                AztecLog.logContentDetails(visualEditor)
             }
         }
 

aztec/src/main/kotlin/org/wordpress/aztec/handlers/BlockHandler.kt

@@ -2,9 +2,11 @@ package org.wordpress.aztec.handlers
 
 import android.text.Spannable
 import android.text.Spanned
+import org.wordpress.android.util.AppLog
 import org.wordpress.aztec.Constants
 import org.wordpress.aztec.spans.IAztecBlockSpan
 import org.wordpress.aztec.spans.IAztecNestable
+import org.wordpress.aztec.util.AztecLog
 import org.wordpress.aztec.util.SpanWrapper
 import org.wordpress.aztec.watchers.BlockElementWatcher.TextChangeHandler
 
@@ -115,6 +117,13 @@ abstract class BlockHandler<SpanType : IAztecBlockSpan>(val clazz: Class<SpanTyp
 
     companion object {
         fun set(text: Spannable, block: IAztecBlockSpan, start: Int, end: Int) {
+            if (start > end) {
+                AppLog.w(AppLog.T.EDITOR, "BlockHandler.set static method called with start > end. Start: " + start + " End: " + end)
+                AppLog.w(AppLog.T.EDITOR, "Invoked with block type of " + block.javaClass.canonicalName)
+                AztecLog.logContentDetails(text)
+                return
+            }
+
             val flags = Spanned.SPAN_PARAGRAPH
             if (SpanWrapper.isInvalidParagraph(text, start, end, flags)) return
 

aztec/src/main/kotlin/org/wordpress/aztec/util/Azteclog.kt

@@ -1,5 +1,6 @@
 package org.wordpress.aztec.util
 
+import android.text.Spannable
 import org.json.JSONArray
 import org.json.JSONException
 import org.json.JSONObject
@@ -14,23 +15,27 @@ class AztecLog {
     }
 
     companion object {
-        fun logEditorContentDetails(aztecText: AztecText) {
+        fun logContentDetails(aztecText: AztecText) {
+            AppLog.d(AppLog.T.EDITOR, "Below are the details of the content in the editor:")
+            logContentDetails(aztecText.text)
+        }
+
+        fun logContentDetails(text: Spannable) {
             try {
                 val logContentJSON = JSONObject()
-                logContentJSON.put("content", aztecText.text.toString())
-                logContentJSON.put("length", aztecText.text.length)
+                logContentJSON.put("content", text.toString())
+                logContentJSON.put("length", text.length)
                 val spansJSON = JSONArray()
-                val spans = aztecText.text.getSpans(0, aztecText.text.length, Any::class.java)
+                val spans = text.getSpans(0, text.length, Any::class.java)
                 spans.forEach {
                     val currenSpanJSON = JSONObject()
-                    currenSpanJSON.put("clasz", it.javaClass.name)
-                    currenSpanJSON.put("start", aztecText.text.getSpanStart(it))
-                    currenSpanJSON.put("end", aztecText.text.getSpanEnd(it))
-                    currenSpanJSON.put("flags", aztecText.text.getSpanFlags(it))
+                    currenSpanJSON.put("clazz", it.javaClass.name)
+                    currenSpanJSON.put("start", text.getSpanStart(it))
+                    currenSpanJSON.put("end", text.getSpanEnd(it))
+                    currenSpanJSON.put("flags", text.getSpanFlags(it))
                     spansJSON.put(currenSpanJSON)
                 }
                 logContentJSON.put("spans", spansJSON)
-                AppLog.d(AppLog.T.EDITOR, "Below are the details of the content in the editor:")
                 AppLog.d(AppLog.T.EDITOR, logContentJSON.toString())
             } catch (e: JSONException) {
                 AppLog.e(AppLog.T.EDITOR, "Uhh ohh! There was an error logging the content of the Editor. This should" +