[Kmymoney-devel] Payees view

Cristian Oneţ onet.cristian at gmail.com
Tue Mar 2 11:25:15 CET 2010


On Tue, Mar 2, 2010 at 11:53 AM, Alvaro Soliverez <asoliverez at gmail.com> wrote:
> Hello all,
> I've just commited a major change to the payees view, to port it to a
> Qt4 widget.
> There are probably some issues still to be solved, so please keep your
> eyes open.
> As usual, feedback is welcome!
I'll keep an eye for any issues :).
So now that we've done this for a few lists I see that the way the
code was written and then ported requires a lot of dynamic_casts in
order to get to the specific item which stores the business logic
specific data (for instance an id). I think the code would be much
cleaner without these routine casts so I propose the following simple
steps to get rid of them:
1. Don't use an implementation specific item.
2. Store the data that was previously a member of the implementation
specific item in the QTree(List)WidgetItem using the setData and data
functions and a user role definition for each implementation specific
data
Any thoughts?

Regards,
Cristian


More information about the KMyMoney-devel mailing list