History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: LOOM-143
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Ignacio Coloma
Reporter: Ignacio Coloma
Votes: 0
Watchers: 0
Operations

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

Add URI parameters automatically with paged tags (warning: may break existing behavior)

Created: 28/Jan/09 05:51 PM   Updated: 28/Jan/09 05:55 PM
Component/s: None
Affects Version/s: 1.0
Fix Version/s: 1.1


 Description  « Hide
Actually, when using paged tags any required parameter must be added by hand, reconstructing the source URL:

@Path("/{foo}/{bar}")
public Resolution myEvent() {
...
}

<l:pagedList param-foo="${action.foo}" param-bar="${action.bar}">
...
</l:pagedList>

In its current implementation, the paged tag will complain if any required parameter is missing. Instead, they should be added automatically with the current value found in the request object. If there is any need to NOT add these values automatically, an attribute addUriParams=false should be used.

NOTE: THIS IS A CHANGE IN BEHAVIOR. It is extremely unlikely that this breaks any existing code, but existing tags may have already specified the value and end up with the same parameter/value may be added twice. Other than that, it should not have any downside.

 All   Comments   Change History      Sort Order:
There are no comments yet on this issue.