Hi!
I have had problems lately with the fact that the DirectLink component
uses service parameters with all the same parameter names
(ServiceConstants.PARAMETER, that is, "sp"). This is in fact a problem
for me, as I have a servlet filter which wraps any request and puts all
the query string values into a Map (so if query string parameters have
the same name, only one gets in the Map). Right now, I cannot afford to
change this (probably problematic) behaviour.
The contrib:Table component uses for instance 2 service parameters
(sp=value1&sp=value2) for its sorting and paging features. Somehow, I
needed to have different query string parameter names for each service
parameters. So I had the courage to do a *really ugly* patch in
LinkFactoryImpl, which so far meets my needs (sorting and paging now
works fine with my contrib:Table). Here is what it looks like:
protected void squeezeServiceParameters(Map parameters)
{
Object[] serviceParameters = (Object[])
parameters.get(ServiceConstants.PARAMETER);
if (serviceParameters == null)
return;
String[] squeezed = squeeze(serviceParameters);
//** BEGIN patch **
if(squeezed.length == 2){
parameters.put(ServiceConstants.PARAMETER + "1", squeezed[0]);
parameters.put(ServiceConstants.PARAMETER + "2", squeezed[1]);
parameters.remove(ServiceConstants.PARAMETER);
}else
//** END patch **
parameters.put(ServiceConstants.PARAMETER, squeezed);
}
public Object[] extractListenerParameters(IRequestCycle cycle)
{
String[] squeezed =
cycle.getParameters(ServiceConstants.PARAMETER);
//** BEGIN patch **
String s1 = cycle.getParameter(ServiceConstants.PARAMETER + "1");
if(s1 != null){
String s2 = cycle.getParameter(ServiceConstants.PARAMETER + "2");
squeezed = new String[]{s1, s2};
}
//** END patch **
if (Tapestry.size(squeezed) == 0)
return EMPTY;
try
{
return _dataSqueezer.unsqueeze(squeezed);
}
catch (Exception ex)
{
throw new ApplicationRuntimeException(ex);
}
}
If you would have a better solution for me, which would enable different
query string parameter names for service parameters (instead of reusing
"sp"), I'm open to your suggestions! Otherwise, would you see how my
patch could break some other code elsewhere?
/Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
