Bug 443303 - Kirigami AboutPage bug report action causes drkonqi to be unable to report crashes
Summary: Kirigami AboutPage bug report action causes drkonqi to be unable to report cr...
Status: RESOLVED DUPLICATE of bug 444554
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.86.0
Platform: Neon Linux
: VHI normal
Target Milestone: Not decided
Assignee: Marco Martin
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2021-10-04 13:27 UTC by Patrick Silva
Modified: 2021-12-16 02:12 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
screenshot (54.89 KB, image/png)
2021-10-04 13:27 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2021-10-04 13:27:37 UTC
Created attachment 142141 [details]
screenshot

SUMMARY
Every time I try to report a Discover crash, I get the result seen in the
attached screenshot.

EXPECTED RESULT
drkonqi should report Discover crash

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.23.80
KDE Frameworks Version: 5.87.0
Qt Version: 5.15.3
Graphics Platform: Wayland
Comment 1 Harald Sitter 2021-10-05 09:06:35 UTC
https://invent.kde.org/plasma/discover/-/commit/7674dae2c3b7c52f4552300b416616f0ce647759

bugaddress mustn't be set (when using bugzilla), or must be submit@bugs.kde.org
Comment 2 Nate Graham 2021-10-05 14:53:13 UTC
Ugh, this is messy. We set bugaddress in Discover to support the new Kirigami "report a bug action" that appears by default in the About page. I guess it's a classic "multiple sources of truth" problem, since that action in Discover pulls its bug reporting URL from the app itself, whereas DrKonqi pulls its data from a separate repo/location.

Could we make DrKonqi introspect the value of bugaddress in the app to see if it matches the value that's set elsewhere, maybe? Or make the Kirigami action able to see the URL from the other place, if not overridden in the app?
Comment 3 Felipe Kinoshita 2021-10-05 18:43:58 UTC
After commenting the line that adds the bug address URL to the aboutData DrKonqi works as expected.

bug reported for testing through DrKonqi after commenting that line: #443362
Comment 4 Felipe Kinoshita 2021-10-05 18:44:33 UTC
This seems more like a DrKonqi bug than a Discover one to me...
Comment 5 Harald Sitter 2021-10-15 10:39:08 UTC
(In reply to Nate Graham from comment #2)
> Ugh, this is messy. We set bugaddress in Discover to support the new
> Kirigami "report a bug action" that appears by default in the About page. I
> guess it's a classic "multiple sources of truth" problem, since that action
> in Discover pulls its bug reporting URL from the app itself, whereas DrKonqi
> pulls its data from a separate repo/location.
> 
> Could we make DrKonqi introspect the value of bugaddress in the app to see
> if it matches the value that's set elsewhere, maybe?

No, the app has crashed, we can't ask it anything ;)

I haven't given this any thought but it seems to me kirigami needs to assemble KAboutData::bugAddress, ::productName and ::componentName to the actual bugs.kde.org address (if submit@bugs.kde.org was used as address). That is how this works in kbugreportdialog and drkonqi ultimately.
Comment 6 Nate Graham 2021-10-15 16:04:45 UTC
If DrKonqi can't ask the app for anything after it crashed, how can it get confused by the app setting its own bug address internally?
Comment 7 Harald Sitter 2021-10-15 17:37:24 UTC
It gets the data passed along by kcrash which excavates it from kaboutdata.
Comment 8 Nate Graham 2021-10-15 21:28:09 UTC
Gotcha.

Felipe, would you be able to investigate possible fixes? Thanks!
Comment 9 Nate Graham 2021-12-16 02:12:28 UTC

*** This bug has been marked as a duplicate of bug 444554 ***