Issue Details (XML | Word | Printable)

Key: DISPL-157
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Reporter: Paul Woodward
Votes: 0
Watchers: 4
Operations

If you were logged in you would be able to see more operations.
DisplayTag

Allow displaytag in a form

Created: 17/Dec/04 01:49 PM   Updated: 22/Oct/09 09:15 AM   Resolved: 16/Dec/07 03:02 PM
Return to search
Component/s: None
Affects Version/s: 1.1
Fix Version/s: 1.2

Time Tracking:
Original Estimate: 2 hours
Original Estimate - 2 hours
Remaining Estimate: 2 hours
Remaining Estimate - 2 hours
Time Spent: Not Specified
Time Spent - Not Specified

Application server: Any


 Description  « Hide
Basically I would like to be able to make the sort/page links be javascript calls to a function. This is so that I can incorporate the displaytag in existing forms/framework I am using without losing information that is kept on the page.

I can almost achieve this by:
1. using a custom RequestHelperFactory to create custom RequestHelper that creates a special type of Href - a JavascriptHref that takes the displaytag parameter map and passes it to a function to package up in my existing form.
2. the same RequestHelper to retrieve the displaytag parameters from the request object where they are being and pass them back to TableTag when requested.

The only downfall of this plan is gereating the javascript link.
getSortingHref in TableTag (and the similar functions) do:
Href href = new Href(this.basehref);
And hence I lose the functionality of my special JavascriptHref class.

If it was:
Href href = this.basehref.clone();
or similar my plan would work perfectly.

As far as I can tell, this would be a minimal impact change, just a clone method in Href and the use of it in TableTag.

Would this be possible?

Apart from this tiny issue I have, it's a truly excellent project.


Emilio Dolce added a comment - 15/Dec/05 03:59 PM
Possible solution can be in the DISPL-267

fabrizio giustina made changes - 29/Dec/05 09:32 AM
Field Original Value New Value
Fix Version/s After 1.1 [ 12190 ]
fabrizio giustina made changes - 15/Oct/07 05:10 PM
Fix Version/s TBD [ 12190 ]
Summary Allow displaytag in a form - possible solution Allow displaytag in a form
Fix Version/s 1.1.2 [ 13662 ]
Affects Version/s 1.1 [ 11225 ]
fabrizio giustina added a comment - 16/Dec/07 03:02 PM
implemented in 1.2. DIsplaytag can now be configured to submit a form when clicking on a pagination or sorting link

fabrizio giustina made changes - 16/Dec/07 03:02 PM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
fabrizio giustina made changes - 16/Dec/07 04:07 PM
Status Resolved [ 5 ] Closed [ 6 ]
Mike Calmus added a comment - 26/Dec/07 02:23 PM
Part of this most recent patch could cause issues:

            if (value != null & value.getClass().isArray())

should be rewritten as:

            if (value != null && value.getClass().isArray())

If value is null, a NullPointerException will be thrown

Esat Postaci added a comment - 25/Jun/08 02:40 AM
Where is the 1.2 version :)

Still testing since 6 months.

OMG...

Severyn Vital added a comment - 22/Oct/09 09:15 AM
why form attribute was not added in displaytag-el.tld but only in displaytag.tld?