https://bugs.kde.org/show_bug.cgi?id=312127
allan <agande...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
CC| |agande...@gmail.com
Resolution|DUPLICATE |---
Ever confirmed|0 |1
--- Comment #3 from allan <agande...@gmail.com> ---
As far as I can see, m_onlineFactor is enabled at
KOnlineUpdateWizardPage::slotCheckPage(const QString& txt), which can be called
from KNewInvestmentWizard::KNewInvestmentWizard(QWidget *parent)
m_onlineUpdatePage->slotCheckPage(QString()) or from
KNewInvestmentWizard::init2(void). the latter only getting called when editing
an existing security, or when an existing symbol has been entered. So, it
seems the first call is the relevant one.
However, this call to slotCheckPage() is made with txt = QString(), so the
widget will not get enabled. QString() could be replaced with a non-empty
string, but there seems little point in that. Instead, why have the check for
a string in m_onlineFactor->setEnabled(!txt.isEmpty())? Why not change it to
setEnabled(void))? Why have the enabling conditional in a constructor?
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel
