[jira] Commented: (TAPESTRY-1457) Setting empty activation context

["Howard M. Lewis Ship (JIRA)" <de...@tapestry.apache.org>]

    [ https://issues.apache.org/jira/browse/TAPESTRY-1457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12496701 ] 

Howard M. Lewis Ship commented on TAPESTRY-1457:
------------------------------------------------

Ah, the joys of JDK 5 varargs.  I really liked the idea of being able to pass Object... for the context.  The problem is that there's no way to distinguish explicit null from omitting the varargs parameters (it just shows up as an empty array).

I may have to convert the page context parameters to Object[] so that we can differential null from explict no context.

> Setting empty activation context
> --------------------------------
>
>                 Key: TAPESTRY-1457
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1457
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: Core Components
>    Affects Versions: 5.0.3
>            Reporter: Peter
>
> It isn't possible to set an empty context using 'contex' parameter of PageLink. It always treats null and zero length array/list in the same way - asks the page about its activation context via onPassivate. I think that only null should trigger onPassivate.
> Example
> <a t:type="PageLink" page="test" context="context">page link</a>
> public Object[] getContext()
> {
> 	if (test == 0)
> 		return new Object[] {"1", "2"};	// context will be ../test/1/2
> 	else
> 		if (test == 1)
> 			return new Object[0];	// onPassivate will be asked but context should be empty IMO
> 		else
> 			return null;	// onPassivate will be asked, ok
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org