When you go through the Wizard (Choosing Stocks) and get to the online update screen and choose an online data source the 'Factor' field stays greyed out. To enter a Factor (I am entering UK Shares), you need to save the record then go back in under edit to change the Factor. Reproducible: Always Steps to Reproduce: 1. Start New Investment Wizard 2. Choose Stocks, and on-line update 3. Try and change 'Factor' Actual Results: Factor is greyed out Expected Results: Able to enter factor without having to go back in.
This is a duplicate of bug 281459
*** This bug has been marked as a duplicate of bug 281459 ***
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?
As a probably better alternative, more in keeping with the intended use, and, assuming it might have been the intention to enable the m_onlineFactor widget once an update source has been selected, I've introduced another call to m_onlineUpdatePage->slotCheckPage(), triggered by a connect when the source combo is activated. This seems to work as I expected.
Fixed in duplicate - Bug 281459 New: Can't select Factor in New Investment Wizard (Sorry, but I did not see your comment on a duplicate of the bug. I just fixed that similar to your proposed patch but a bit differently. So you can simply drop this review request here. - Thomas)