Merge pull request #430 from wordpress-mobile/feature/crash-log

khaykov · web-flow · commit 0a0b8579e360 · 2017-07-24T21:00:02.000+09:00
Make the exception log actually useful
diff --git a/aztec/src/main/kotlin/org/wordpress/aztec/AztecText.kt b/aztec/src/main/kotlin/org/wordpress/aztec/AztecText.kt
@@ -976,7 +976,7 @@ class AztecText : AppCompatAutoCompleteTextView, TextWatcher, UnknownHtmlSpan.On
                     editable.replace(min, max, builder)
                 } catch (e: RuntimeException) {
                     // try to get more context for this crash: https://github.com/wordpress-mobile/AztecEditor-Android/issues/424
-                    throw RuntimeException("min: $min; max: $max; text: ${editable.toString()}; pasted: ${builder.toString()}", e)
+                    throw RuntimeException("### MIN: $min, MAX: $max\n---\n### TEXT:${toHtml()}\n---\n### PASTED:${parser.toHtml(builder)}", e)
                 }
                 enableTextChangedListener()
 

Original file line number	Diff line number	Diff line change
`@@ -976,7 +976,7 @@ class AztecText : AppCompatAutoCompleteTextView, TextWatcher, UnknownHtmlSpan.On`
`976`	`976`	`editable.replace(min, max, builder)`
`977`	`977`	`} catch (e: RuntimeException) {`
`978`	`978`	`// try to get more context for this crash: https://github.com/wordpress-mobile/AztecEditor-Android/issues/424`
`979`		`- throw RuntimeException("min: $min; max: $max; text: ${editable.toString()}; pasted: ${builder.toString()}", e)`
	`979`	`+ throw RuntimeException("### MIN: $min, MAX: $max\n---\n### TEXT:${toHtml()}\n---\n### PASTED:${parser.toHtml(builder)}", e)`
`980`	`980`	`}`
`981`	`981`	`enableTextChangedListener()`
`982`	`982`