[Kmymoney-devel] playground/office/kmymoney/kmymoney2/views

[Cristian Oneţ]

În data de Duminică 25 Octombrie 2009 15:39:35 Alvaro Soliverez a scris:
> On Sun, Oct 25, 2009 at 5:30 AM, Cristian Oneţ 
<onet.cristian at gmail.com>wrote:
> > În data de Duminică 25 Octombrie 2009 10:27:10 Cristian Oneț a scris:
> > > SVN commit 1039980 by conet:
> > >
> > > Fixed a crash in the Forecast chart. Also made the chart use the
> > > widgets space better.
> > >
> > >  M  +3 -2      kforecastview.cpp
> > >
> > >
> > > --- trunk/playground/office/kmymoney/kmymoney2/views/kforecastview.cpp
> > >  #1039979:1039980 @@ -658,7 +658,8 @@
> > >    reportCfg.setIncludingSchedules( false );
> > >    reportCfg.addAccountGroup(MyMoneyAccount::Asset);
> > >    reportCfg.addAccountGroup(MyMoneyAccount::Liability);
> > > -  reportCfg.setColumnsAreDays( true );
> > > +  // FIXME: this causes a crash
> > > +  //reportCfg.setColumnsAreDays( true );
> > >    reportCfg.setConvertCurrency( true );
> > >    reportCfg.setIncludingForecast( true );
> >
> > 
> > reportCfg.setDateFilter(QDate::currentDate(),QDate::currentDate().addDays
> >(
> >
> > > m_forecastDays->value())); @@ -667,7 +668,7 @@
> > >    table.drawChart(*m_forecastChart);
> > >
> > >    // Adjust the size
> > > -  m_forecastChart->resize(m_tab->width()-30, m_tab->height()-60);
> > > +  m_forecastChart->resize(m_tab->width() - 10, m_tab->height());
> > >    m_forecastChart->update();
> > >  }
> >
> > Alvaro, I thought you might want to take a look at this when you have
> > time
> >
> > :).
> 
> Sure. Any idea why this might be failing?
> There are other reports using this setting and they work just fine.
> 
As I saw some vector is indexed out of bounds in KDChart 
(KDChartCartesianDiagramDataCompressor_p.cpp in the invalidate function I 
think) but I don't know the cause of that. That's why I fixed it that way :).

-- 
Regards,

Cristian Oneţ
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/kmymoney-devel/attachments/20091025/7ad109de/attachment.sig