Ernst Bunders <[EMAIL PROTECTED]> wrote:
> I very much like the new comparator attribute to the listtag. The only
> problem with it is that you can't pass args to it. I made a little hack to
> the ListSorter class (see atachement). It checks if the Comparator class
> has an 'init' method that allows a PageContext object. if so, it passes
> the pageContext to it. the init method can then preconfigure the
> Comparator.
>
> I don't suppose this is the best way to do this or that this hack should
> go into mmbase. But i think something must be done about it.
I don't dislike your idea, though perhaps the implementation is a bit
clumsy ( }catch(Exception e){System.out.println("hier"+e.toString()); }
??)
You're a voted committor, if you'd propose this improvement as a hack, you'd
have my vote. On the other hand the taglib 2 project is still open (and the
comparator attribute was created by it). As the leader of this project I
would also welcome you in the project (beter late than never..), and then
I think you can make the change simply with the aproval of the project (er,
me).
Michiel
--
Michiel Meeuwissen
Mediacentrum 140 H'sum
+31 (0)35 6772979
nl_NL eo_XX en_US
mihxil'
[] ()
