On Tue, Nov 24, 2009 at 07:47, Fritz Zaucker <[email protected]> wrote:

> Hi Qooxdoo gods,
>
> the simple table model has a method setSortMethods(). However, it is
> missing
> the matching getSortMethods() method.
>
> As the sort methods are stored in a private variable (__sortMethods), it is
> not possible to access the sort methods in a class extending
> qx.ui.table.model.Simple. Currently the contributed Smart table model is
> not
> fully functional for this reason.
>
> Would it make sense to
>
> a) make __sortMethods protected instead of private
>
> or
>
> b) add a getSortMethods() method?
>
> I'd be willing to provide a patch for either of these variants, please
> advice which you'd be willing to include into the code base ...
>

Option (b) sounds appropriate. I agree there should be a way to retrieve
what has been set, although I've never had a need for such thing. It's all
done programatically (by the developer) in nearly every case, so the
developer should know what has been set. In fact, because of that, I'd
almost prefer that the getSortMethods() method be added as a mixin. I
suppose it's small enough that it could be directly included in the Simple
model, but enough small things add up to something big so I'm always
thinking about how much use something will get before directly adding it to
a class.

Derrell
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to