On Oct 18, 2006, at 7:30 AM, Jacopo Cappellato (JIRA) wrote:
[ http://issues.apache.org/jira/browse/OFBIZ-364?
page=comments#action_12443180 ]
Jacopo Cappellato commented on OFBIZ-364:
-----------------------------------------
Leon Wrote:
It's working without issues for me, although I don't know if
putting a magical parameter in the context
is the right way to invoke this ability. Maybe a form widget
attribute would be better?
This is a good question... and I'd love to get a comment from David
Jones about it since I'm not totally sure about the best way of
doing this.
That is a good point. An attribute on the form definition element
would be better, and can be a constant or parameterized as needed
(even if it would almost always be parameterized...). Something like
the current name would be great, though we can probably leave out the
"Form" since that is redundant, so we'd just have "override-list-size".
Of course, if the intent is just to get a partial list from the
database and display it with the form widget there are ways of doing
that without all of this manual stuff... I haven't looked at this
specific scenario enough to know if all the manual stuff is really
required, but in general it's nice to keep the code as simple as
possible and let it take care of the pagination and getting partial
results from the database using the built in stuff.
Whatever the case is, this additional flexibility of being able to
put the list together manually does sound like a nice feature (I
guess I just haven't run into a need for it yet).
-David
