[Kmymoney-devel] Historic forecast seems to be broken

Thomas Baumgart thb at net-bembel.de
Sun Jun 27 16:01:33 CEST 2010


Hi,

on Sunday 27 June 2010 Alvaro Soliverez wrote:



> On Sun, Jun 27, 2010 at 10:14 AM, Thomas Baumgart <thb at net-bembel.de> wrote:
> > Hi,
> >
> > on Sunday 27 June 2010 Alvaro Soliverez wrote:
> >> The forecast is working fine, but the settings are not being refreshed.
> >> If you change the forecast method and restart the application, it
> >> works correctly.
> >>
> >> For some reason, it is not refreshing the settings after changing the
> >> settings. I confirmed it adding a few debugs in mymoneyforecast.
> >>
> >> This line returns always the same value until I restart.
> >>
> >> KMyMoneyGlobalSettings::forecastMethod()
> >>
> >> So, what else changed that causes this behaviour?
> >
> > I think I know what the problem is. The code relies on the fact, that the
> > MyMoneyForecast ctor sets the current value for the method. That is what
> > I have touched. I scanned for it's usage, but apparently, I did not hit
> > all of them.
> 
> No,  KMyMoneyGlobalSettings::forecastMethod() is returning the
> outdated value. I tried it before your changes.

I fixed it in SVN. Please check if it works for you. I did not see a problem 
with the settings object here. See my last commit for reference what needed to 
be done.

-- 

Regards

Thomas Baumgart

GPG-FP: E55E D592 F45F 116B 8429   4F99 9C59 DB40 B75D D3BA
-------------------------------------------------------------
"I'd like to buy Windows." "Are you crazy?" "Is this part of the licence
agreement?" "Not really, but it might ease the usage ..."
-------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 224 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/kmymoney-devel/attachments/20100627/16701ba5/attachment.sig 


More information about the KMyMoney-devel mailing list