Bug 437332

Summary: We should default to yahoo for any new securities
Product: [Applications] kmymoney Reporter: Dawid Wróbel <me>
Component: importerAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 5.1.1   
Target Milestone: ---   
Platform: Other   
OS: Other   
Latest Commit: Version Fixed In: 5.1.2
Sentry Crash Report:

Description Dawid Wróbel 2021-05-18 23:59:24 UTC
SUMMARY

When importing a statement with new securities, they get created with Stooq source configured by default, which most of the cases will not work due proprietary symbol prefixes/suffixes which Stooq uses (e.g. AMD is AMD.US)

We should default to Yahoo Finance instead, now that it's been stable for over a year. We may also default to Yahoo Finance for any new currencies added. This may actually be necessary  if there's no distinction in code between the type of security at the time it gets created – and from a quick glimpse at the code, it seems to be the case.

STEPS TO REPRODUCE
1. Import a statement with new, previously unknown securities

OBSERVED RESULT
1. Notice that Stooq is selected as a default source for online quotes
2. Notice that Stooq will most likely *not* work with the imported security's vanilla symbol.


EXPECTED RESULT
1. KMM should default to Yahoo Finance, increasing the chances that the online quotes would work out of the box for any new security.
Comment 1 Dawid Wróbel 2021-06-13 06:56:59 UTC
Git commit d3f7eef2030b524f8e8258ccddb4cbb4722099fb by Dawid Wróbel.
Committed on 13/06/2021 at 06:56.
Pushed by wrobelda into branch '5.1'.

Make Yahoo default quote source for new securities
FIXED-IN: 5.1.2

M  +1    -1    kmymoney/converter/mymoneystatementreader.cpp

https://invent.kde.org/office/kmymoney/commit/d3f7eef2030b524f8e8258ccddb4cbb4722099fb
Comment 2 Dawid Wróbel 2021-06-13 06:57:55 UTC
Git commit 4091fff43905cf463b48211a898e5d49585cd6ee by Dawid Wrobel, on behalf of Dawid Wróbel.
Committed on 13/06/2021 at 06:57.
Pushed by wrobelda into branch 'master'.

Make Yahoo default quote source for new securities
FIXED-IN: 5.1.2


(cherry picked from commit d3f7eef2030b524f8e8258ccddb4cbb4722099fb)

M  +1    -1    kmymoney/converter/mymoneystatementreader.cpp

https://invent.kde.org/office/kmymoney/commit/4091fff43905cf463b48211a898e5d49585cd6ee