Bug 408451 - Financial Times online price update fails for LSE equities
Summary: Financial Times online price update fails for LSE equities
Status: REPORTED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-08 11:13 UTC by David Mason
Modified: 2020-06-11 21:26 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch to make Financial Times date regexp more flexible (1.09 KB, patch)
2019-06-08 11:13 UTC, David Mason
Details

Note You need to log in before you can comment on or make changes to this bug.
Description David Mason 2019-06-08 11:13:45 UTC
Created attachment 120690 [details]
Patch to make Financial Times date regexp more flexible

SUMMARY

I've noticed that online price update fails for LSE (London Stock Exchange) equities when using the "Financial Times" source.

The date regexp for this data source searches for the text: "Data delayed at least 15 minutes". However, LSE equities have a delay of 20 minutes, so the match fails: https://markets.ft.com/data/equities/tearsheet/summary?s=IHG:LSE

Making the date regexp more flexible allows price update to work as expected.

I've attached a patch that updates the default date regexp.

STEPS TO REPRODUCE
1. Add an LSE equity, such as IHG:LSE
2. Set the online source to Financial Times
3. Update the price

OBSERVED RESULT

Price found: '123.40' (123.4)
Unable to update price for xxx (no price or no date)

EXPECTED RESULT

Price found: '123.40' (123.4)
Date found: 'Fri Jun 7 2019'
Price for xxx updated (ID E000001)
Comment 1 Ralf Habacker 2019-06-22 23:22:59 UTC
Comment on attachment 120690 [details]
Patch to make Financial Times date regexp more flexible

Thanks for this patch, which has been added to the related alkimia online source located at https://www.pling.com/p/1311072/.
Comment 2 Jack 2020-06-11 21:26:07 UTC
Should this change still be applied to the main repository, or is it no longer needed.  It looks like the entry for FT was last updated in 2017.