mirror of
https://github.com/TeamNewPipe/NewPipe
synced 2026-02-22 09:59:44 +00:00
Partially revert: ErrorActivity: Kotlin-fy buildMarkdown method
Partially reverts c3dbed54e5
Fix the bug where collapse container for several crashlogs is created
when only one crash log present
This commit is contained in:
@@ -228,24 +228,26 @@ class ErrorActivity : AppCompatActivity() {
|
||||
|
||||
// Collapse all logs to a single paragraph when there are more than one
|
||||
// to keep the GitHub issue clean.
|
||||
if (errorInfo.stackTraces.isNotEmpty()) {
|
||||
if (errorInfo.stackTraces.size > 1) {
|
||||
append("<details><summary><b>Exceptions (")
|
||||
append(errorInfo.stackTraces.size)
|
||||
append(")</b></summary><p>\n")
|
||||
}
|
||||
|
||||
// add the logs
|
||||
errorInfo.stackTraces.forEachIndexed { index, stacktrace ->
|
||||
append("<details><summary><b>Crash log ")
|
||||
if (errorInfo.stackTraces.isNotEmpty()) {
|
||||
append(index + 1)
|
||||
}
|
||||
append("</b>")
|
||||
append("</summary><p>\n")
|
||||
append("\n```\n${stacktrace}\n```\n")
|
||||
append("</details>\n")
|
||||
// add the logs
|
||||
errorInfo.stackTraces.forEachIndexed { index, stacktrace ->
|
||||
append("<details><summary><b>Crash log ")
|
||||
if (errorInfo.stackTraces.size > 1) {
|
||||
append(index + 1)
|
||||
}
|
||||
append("</b>")
|
||||
append("</summary><p>\n")
|
||||
append("\n```\n${stacktrace}\n```\n")
|
||||
append("</details>\n")
|
||||
}
|
||||
|
||||
// make sure to close everything
|
||||
// make sure to close everything
|
||||
if (errorInfo.stackTraces.size > 1) {
|
||||
append("</p></details>\n")
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user