Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread allan
Yes, I would say so. Cristian Oneț wrote: >So all of you are saying that :GBP should be removed from the end of the URL? > >Regards, >Cristian > >2014-08-23 19:16 GMT+03:00 Allan : >> On 23/08/14 15:52, Ian Neal wrote: >>> >>> Hi David, >>> >>> The one I had a problem with is GB00B1XG9607. >>> >

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Cristian Oneț
So all of you are saying that :GBP should be removed from the end of the URL? Regards, Cristian 2014-08-23 19:16 GMT+03:00 Allan : > On 23/08/14 15:52, Ian Neal wrote: >> >> Hi David, >> >> The one I had a problem with is GB00B1XG9607. >> >> >> Probably a configuration problem at the FT end, as i

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Allan
On 23/08/14 15:52, Ian Neal wrote: Hi David, The one I had a problem with is GB00B1XG9607. Probably a configuration problem at the FT end, as it resolves to GB00B1XG9607:GBP but is displayed as a GBX and if you try GB00B1XG9607:GBX it fails to find it. There may be others with this issue, but

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Ian Neal
Hi David, The one I had a problem with is GB00B1XG9607 Probably a configuration problem at the FT end, as it resolves to GB00B1XG9607:GBP but is displayed as a GBX and if you try GB00B1XG9607:GBX it fails to find it. There may be others with this issue, but it is trying to fix that one that

Re: [Kmymoney-devel] What's new page and documentation

2014-08-23 Thread Jack
On 2014.08.23 09:21, Cristian Oneț wrote: 2014-08-21 21:25 GMT+03:00 Jack : > Cristian, > > Yes, as I'm far better at editing than at writing, that sounds good. > Unfortunately, I've done extremely little so far, but at least I've managed > to clean out the problems with my local git copy so I

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Allan
On 23/08/14 14:42, David Houlden wrote: OK, I've had a look at this now. The change as submitted by Thomas does not work for me because of the :GBP on the end. This should be :GBX which is what I have been using for several months now. From my limited testing It does seem that removing :GBP and

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread David Houlden
OK, I've had a look at this now. The change as submitted by Thomas does not work for me because of the :GBP on the end. This should be :GBX which is what I have been using for several months now. From my limited testing It does seem that removing :GBP and having the URL end in %1 also works.

Re: [Kmymoney-devel] Short video about the Randa Meeting

2014-08-23 Thread Cristian Oneț
2014-08-23 16:05 GMT+03:00 Alvaro Soliverez : > Hehe, they used one of the proposed new logos. Just another reason for us to adopt the new artwork :). > > Very good, Cristian! > > On Sat, Aug 23, 2014 at 9:42 AM, Thomas Baumgart wrote: >> Hi all, >> >> check out >> >>http://www.youtube.com/w

Re: [Kmymoney-devel] What's new page and documentation

2014-08-23 Thread Cristian Oneț
2014-08-21 21:25 GMT+03:00 Jack : > Cristian, > > Yes, as I'm far better at editing than at writing, that sounds good. > Unfortunately, I've done extremely little so far, but at least I've managed > to clean out the problems with my local git copy so I can at least do a > clean compile. I've got p

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Ian Neal
Hi Allan, If you do not put a suffix on, then the website automatically puts the correct one on. I have tried with a number of funds which would end in :EUR, :USD and :GBX and they have all correctly been identified. Try and see what happens. Ian On 23/08/14 14:11, Allan wrote: On 23/08/14

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Allan
On 23/08/14 13:47, Ian Neal wrote: Hi Allan, Does it work with just: http://funds.ft.com/uk/Tearsheet/Summary?s=%1 so not using either of: http://funds.ft.com/uk/Tearsheet/Summary?s=%1:GBP or http://funds.ft.com/uk/Tearsheet/Summary?s=%1:GBX Regards, Ian Hi Neal Yes, but... The symbol has

Re: [Kmymoney-devel] Short video about the Randa Meeting

2014-08-23 Thread Alvaro Soliverez
Hehe, they used one of the proposed new logos. Very good, Cristian! On Sat, Aug 23, 2014 at 9:42 AM, Thomas Baumgart wrote: > Hi all, > > check out > >http://www.youtube.com/watch?v=yua6M9jqoEk > > KMyMoney is mentioned as one of the projects as well as our very own Cristian > Oneţ welcomes

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Ian Neal
Hi Allan, Does it work with just: http://funds.ft.com/uk/Tearsheet/Summary?s=%1 so not using either of: http://funds.ft.com/uk/Tearsheet/Summary?s=%1:GBP or http://funds.ft.com/uk/Tearsheet/Summary?s=%1:GBX Regards, Ian On 23/08/14 13:40, Allan wrote: On 23/08/14 13:15, Ian Neal wrote: Hi A

[Kmymoney-devel] Short video about the Randa Meeting

2014-08-23 Thread Thomas Baumgart
Hi all, check out http://www.youtube.com/watch?v=yua6M9jqoEk KMyMoney is mentioned as one of the projects as well as our very own Cristian Oneţ welcomes the world as #5 in Romanian (I suppose). Great job Cristian!! -- Regards Thomas Baumgart GPG-FP: E55E D592 F45F 116B 8429 4F99 9C5

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Allan
On 23/08/14 13:15, Ian Neal wrote: Hi Allan, Does it work any better with no symbol at the end, i.e.: http://funds.ft.com/uk/Tearsheet/Summary?s=%1 At least one of my investments only works with configured that way. Ian Hi Neal It works with the ':GBX' suffix after the url, or after the s

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Thomas Baumgart
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119911/ --- (Updated Aug. 23, 2014, 12:25 p.m.) Status -- This change has been m

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Ian Neal
Hi Allan, Does it work any better with no symbol at the end, i.e.: http://funds.ft.com/uk/Tearsheet/Summary?s=%1 At least one of my investments only works with configured that way. Ian On 23/08/14 12:11, Allan wrote: On 23/08/14 12:05, David Houlden wrote: Hi all, It was me that sent this

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Allan
On 23/08/14 12:05, David Houlden wrote: Hi all, It was me that sent this change to Thomas. I just checked and I have subsequently made another change to the URL. My version now has GBX instead of GBP. If you can give me a while I will check Thomas's patch to verify it works for me as I am using

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread David Houlden
Hi all, It was me that sent this change to Thomas. I just checked and I have subsequently made another change to the URL. My version now has GBX instead of GBP. If you can give me a while I will check Thomas's patch to verify it works for me as I am using that source for prices. Bit busy jus

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Cristian Oneț
> On Aug. 23, 2014, 10:43 a.m., Cristian Oneț wrote: > > Ship It! > > Cristian Oneț wrote: > Although the currency symbol could be made configurable in the URL. For > fund in euro, like http://funds.ft.com/uk/Tearsheet/Summary?s=ES0102562032:EUR > > Thomas Baumgart wrote: > Right, but

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Thomas Baumgart
> On Aug. 23, 2014, 12:43 p.m., Cristian Oneț wrote: > > Ship It! > > Cristian Oneț wrote: > Although the currency symbol could be made configurable in the URL. For > fund in euro, like http://funds.ft.com/uk/Tearsheet/Summary?s=ES0102562032:EUR Right, but we only have the symbol of the se

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Cristian Oneț
> On Aug. 23, 2014, 10:43 a.m., Cristian Oneț wrote: > > Ship It! Although the currency symbol could be made configurable in the URL. For fund in euro, like http://funds.ft.com/uk/Tearsheet/Summary?s=ES0102562032:EUR - Cristian --- Thi

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Cristian Oneț
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119911/#review65087 --- Ship it! Ship It! - Cristian Oneț On Aug. 23, 2014, 10:40

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Thomas Baumgart
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119911/ --- (Updated Aug. 23, 2014, 12:40 p.m.) Review request for KMymoney. Change

Re: [Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Cristian Oneț
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119911/#review65086 --- kmymoney/converter/webpricequote.cpp

[Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

2014-08-23 Thread Thomas Baumgart
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119911/ --- Review request for KMymoney. Repository: kmymoney Description ---

Re: [Kmymoney-devel] Some build warnings on OSX/10.9 on branch master

2014-08-23 Thread Cristian Oneț
Thanks for reporting these, I'll take a look at them. Regards, Cristian 2014-08-23 12:32 GMT+03:00 Marko Käning : > Hi devs, > > I’ve seen the below warnings using clang on OSX building (successfully) > current git master. > > Greets, > Marko > > --- > > /opt/local/var/macports/build/_opt_mports

Re: [Kmymoney-devel] Building git master fails on OSX 10.6 Snow Leopard

2014-08-23 Thread Cristian Oneț
Hi, It should be fixed now. Regards, Cristian 2014-08-23 12:45 GMT+03:00 Marko Käning : > Hi devs, > > looks like OSX 10.6’s gcc doesn’t know option "-Wlogical-op” ! > > Would you fix that upstream using cmake or shall I simply patch it away for > Snow Leopard on MacPorts? > > Greets, > Marko

[Kmymoney-devel] Building git master fails on OSX 10.6 Snow Leopard

2014-08-23 Thread Marko Käning
Hi devs, looks like OSX 10.6’s gcc doesn’t know option "-Wlogical-op” ! Would you fix that upstream using cmake or shall I simply patch it away for Snow Leopard on MacPorts? Greets, Marko P.S.: All builds for OSX >= 10.7 work fine. --- cd /opt/local/var/macports/build/_opt_mports_dport

[Kmymoney-devel] Some build warnings on OSX/10.9 on branch master

2014-08-23 Thread Marko Käning
Hi devs, I’ve seen the below warnings using clang on OSX building (successfully) current git master. Greets, Marko --- /opt/local/var/macports/build/_opt_mports_dports_kde_kmymoney4-devel/kmymoney4-devel/work/kmymoney/kmymoney/widgets/fixedcolumntreeview.cpp:43:24: warning: private field 'm_f