| Summary: | Options to open the sent bug report and restart the affected application do not work | ||
|---|---|---|---|
| Product: | [Applications] drkonqi | Reporter: | Patrick Silva <bugseforuns> |
| Component: | general | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | sitter |
| Priority: | NOR | ||
| Version First Reported In: | master | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/drkonqi/commit/61e4560e012118471252936c80550d9f5476fc8d | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
| Attachments: | screenshot | ||
|
Description
Patrick Silva
2021-10-15 12:36:49 UTC
Created attachment 142470 [details]
screenshot
The labeless button in the buttonbox is also exciting. I cannot reproduce the url not opening, and indeed I don't see why it would fail. The restart option has the same problem as the other restart option had and simply loses a race condition with the program exit() -.- What I think might make sense is to convert the restart same as on the other page and remove the url open thing (there is the link right above the checkbox already). I see inconsistent behaviors on neon unstable. Drkonqi does not restart Ark nor open the bug report page after the crash reported as bug 446927. Drkonqi opens the bug report page but does not restart Dolphin after the crash reported as bug 446926. Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.23.80 KDE Frameworks Version: 5.89.0 Qt Version: 5.15.3 Graphics Platform: Wayland That's not really surprising. The reason the features don't work is because they are in a race condition on application quit i.e. depending on how quick the quit is 0, 1, or all of the options may work. Git commit 61e4560e012118471252936c80550d9f5476fc8d by Harald Sitter. Committed on 24/04/2022 at 20:17. Pushed by sitter into branch 'master'. grand qml UI rewrite the old qwidget UI sticks around for safer mode (for now) just in case the qml view goes belly up. M +11 -41 src/CMakeLists.txt M +13 -4 src/Messages.sh D +0 -253 src/aboutbugreportingdialog.cpp D +0 -32 src/aboutbugreportingdialog.h D +0 -58 src/applicationdetailsexamples.cpp D +0 -27 src/applicationdetailsexamples.h D +0 -106 src/bugzillaintegration/assistantpage_bugzilla_supported_entities.cpp D +0 -44 src/bugzillaintegration/assistantpage_bugzilla_supported_entities.h D +0 -76 src/bugzillaintegration/assistantpage_bugzilla_version.cpp D +0 -38 src/bugzillaintegration/assistantpage_bugzilla_version.h D +0 -313 src/bugzillaintegration/assistantpage_conclusion.cpp D +0 -40 src/bugzillaintegration/assistantpage_conclusion.h D +0 -130 src/bugzillaintegration/bugzillareportconfirmationdialog.cpp D +0 -31 src/bugzillaintegration/bugzillareportconfirmationdialog.h D +0 -365 src/bugzillaintegration/bugzillareportinformationdialog.cpp D +0 -55 src/bugzillaintegration/bugzillareportinformationdialog.h D +0 -399 src/bugzillaintegration/reportassistantdialog.cpp D +0 -85 src/bugzillaintegration/reportassistantdialog.h D +0 -52 src/bugzillaintegration/reportassistantpage.cpp D +0 -64 src/bugzillaintegration/reportassistantpage.h D +0 -181 src/bugzillaintegration/reportassistantpages_base.cpp D +0 -77 src/bugzillaintegration/reportassistantpages_base.h D +0 -816 src/bugzillaintegration/reportassistantpages_bugzilla.cpp D +0 -147 src/bugzillaintegration/reportassistantpages_bugzilla.h D +0 -549 src/bugzillaintegration/reportassistantpages_bugzilla_duplicates.cpp D +0 -84 src/bugzillaintegration/reportassistantpages_bugzilla_duplicates.h D +0 -52 src/bugzillaintegration/reportinformationdialog.cpp D +0 -24 src/bugzillaintegration/reportinformationdialog.h M +0 -8 src/bugzillaintegration/reportinterface.cpp M +0 -3 src/bugzillaintegration/reportinterface.h D +0 -334 src/bugzillaintegration/ui/assistantpage_bugawareness.ui D +0 -217 src/bugzillaintegration/ui/assistantpage_bugzilla_duplicates.ui D +0 -153 src/bugzillaintegration/ui/assistantpage_bugzilla_duplicates_dialog.ui D +0 -291 src/bugzillaintegration/ui/assistantpage_bugzilla_duplicates_dialog_confirmation.ui D +0 -138 src/bugzillaintegration/ui/assistantpage_bugzilla_information.ui D +0 -162 src/bugzillaintegration/ui/assistantpage_bugzilla_login.ui D +0 -37 src/bugzillaintegration/ui/assistantpage_bugzilla_preview.ui D +0 -127 src/bugzillaintegration/ui/assistantpage_bugzilla_send.ui D +0 -178 src/bugzillaintegration/ui/assistantpage_bugzilla_supported_entities.ui D +0 -163 src/bugzillaintegration/ui/assistantpage_bugzilla_version.ui D +0 -114 src/bugzillaintegration/ui/assistantpage_conclusions.ui D +0 -67 src/bugzillaintegration/ui/assistantpage_conclusions_dialog.ui D +0 -103 src/bugzillaintegration/ui/assistantpage_introduction.ui M +1 -1 src/coredump/gui/CMakeLists.txt D +0 -13 src/coredump/gui/Messages.sh M +1 -0 src/crashedapplication.h M +0 -12 src/debuggerlaunchers.cpp M +0 -12 src/debuggerlaunchers.h M +68 -40 src/drkonqidialog.cpp M +3 -6 src/drkonqidialog.h A +51 -0 src/qml/BacktracePage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +81 -0 src/qml/BugzillaPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +99 -0 src/qml/ContextPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +234 -0 src/qml/DeveloperPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +57 -0 src/qml/DuplicatesLoadingPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +134 -0 src/qml/DuplicatesPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +14 -0 src/qml/FooterActionBar.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +20 -0 src/qml/Globals.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +115 -0 src/qml/LoginPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +122 -0 src/qml/MainPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +73 -0 src/qml/PerfectDuplicatePage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +43 -0 src/qml/PreviewPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +80 -0 src/qml/RatingItem.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +147 -0 src/qml/ReportPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +99 -0 src/qml/SendingPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +28 -0 src/qml/SentPage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +54 -0 src/qml/WelcomePage.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +40 -0 src/qml/main.qml [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +27 -0 src/qml/qml.qrc A +3 -0 src/qml/qmldir A +10 -0 src/qmlextensions/commentmodel.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +19 -0 src/qmlextensions/commentmodel.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +114 -0 src/qmlextensions/credentialstore.cpp [License: GPL(v2.0+)] A +49 -0 src/qmlextensions/credentialstore.h [License: GPL(v2.0+)] A +45 -0 src/qmlextensions/doctore.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +34 -0 src/qmlextensions/doctore.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +118 -0 src/qmlextensions/duplicatemodel.cpp [License: GPL(v2.0+)] A +57 -0 src/qmlextensions/duplicatemodel.h [License: GPL(v2.0+)] A +115 -0 src/qmlextensions/platformmodel.cpp [License: GPL(v2.0+)] A +40 -0 src/qmlextensions/platformmodel.h [License: GPL(v2.0+)] A +37 -0 src/qmlextensions/qobjectlistmodel.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +25 -0 src/qmlextensions/qobjectlistmodel.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +68 -0 src/qmlextensions/reproducibilitymodel.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +29 -0 src/qmlextensions/reproducibilitymodel.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] M +3 -1 src/systeminformation.cpp M +6 -1 src/systeminformation.h https://invent.kde.org/plasma/drkonqi/commit/61e4560e012118471252936c80550d9f5476fc8d |