Bug 408451

Summary: Financial Times online price update fails for LSE equities
Product: [Applications] kmymoney Reporter: David Mason <djm>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: REPORTED ---    
Severity: minor CC: ralf.habacker
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Patch to make Financial Times date regexp more flexible

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.