The fix is to replace the parameter declarations in NavBar.java with these:

        @Parameter
        private String tab = "";

        @Parameter
        private String subTab = "";

and in Layout.java with these:

        @Parameter
        private String tab = "";

        @Parameter
        private String subTab = "";

        @Parameter
        private String title = "";

I'll release 5.4.15 with the fix shortly.

BTW, I've learnt something. By my reading of the Parameter javadoc 
(http://tapestry.apache.org/tapestry5.2-dev/apidocs/org/apache/tapestry5/annotations/Parameter.html),
 I thought these 3 declarations should be equivalent:

        @Parameter
        private String tab;

        @Parameter(value = "")
        private String tab;

        @Parameter
        private String tab = "";

but they are not: if the parameter is not bound, then in the first two its 
value will be null, and in the third one it will be "".

I've raised https://issues.apache.org/jira/browse/TAP5-1668 .

I'll have a look at your debugging questions soon.


On 24/09/2011, at 7:52 AM, George Ludwig wrote:

> Geoff,
> 
> One more thing, I changed NavBar.java to add a check if the tab is null:
> 
> public boolean isSecurityTabActive() {
>               if(tab==null)
>                       return false;
>               return tab.equals("Security");
>       }
> 
> Now the app appears to work, but I sure wish I understood better how
> the tab param should have been passed in the first place!
> 
> -George
> 
> On Fri, Sep 23, 2011 at 2:21 PM, George Ludwig <georgelud...@gmail.com> wrote:
>> Geoff,
>> 
>> I'm running jumpstart 5.4.14 in Eclipse 3.7 on OSX 10.5.8.
>> 
>> However, as Jonathan mentioned if I ignore the error, the debugger
>> takes me to the correct line in NavBar.java  But all I can see is that
>> the tab parameter is null. :)
>> 
>> I can't see the source in any of the tapestry jars, so that's about as
>> far as I can go in the debugger. Is there a way to use ant to download
>> the sources and make Eclipse aware of them? Is it even worth the
>> trouble?
>> 
>> It's been a long time since I used Ant...
>> 
>> It just occurred to me to try this on my machine at home, which is a
>> Vista box. Will let you know if it changes anything.
>> 
>> Best,
>> 
>> George
>> 
>> 
>> 
>> On Fri, Sep 23, 2011 at 2:02 AM, Geoff Callender
>> <geoff.callender.jumpst...@gmail.com> wrote:
>>> What version of JumpStart are you running and what OS are you on? I'll try 
>>> to have a look in the next 24 hours at what's gone awry.
>>> 
>>> On 23/09/2011, at 12:31 PM, Jonathan Barker wrote:
>>> 
>>>> I hit this as well today.
>>>> 
>>>> Strangely, I said to ignore it, and after the usual prompting for a 
>>>> location
>>>> to find source files, debugging worked just fine.
>>>> 
>>>> 
>>>> 
>>>> On Thu, Sep 22, 2011 at 6:37 PM, georgeludwig 
>>>> <georgelud...@gmail.com>wrote:
>>>> 
>>>>> I have jumpstart running in Eclipse as a java app, and it works with the
>>>>> exception of the login. When I log in as admin, I get a null pointer
>>>>> exception:
>>>>> jumpstart.web.components.theapp.NavBar.isSecurityTabActive(NavBar.java:18)
>>>>> 
>>>>> I set a break point in eclipse on line 18 in NavBar.java, and when I run 
>>>>> it
>>>>> in Eclipse, Eclipse tells me that it cannot install a a break point 
>>>>> because
>>>>> the class files were compiled without line numbers.
>>>>> 
>>>>> So I enabled the line numbers in the Eclipse environment, but that didn't
>>>>> help.
>>>>> 
>>>>> I'm unfamiliar with many of the tools used in this project, and can't
>>>>> figure
>>>>> out how the collapser gets invoked during app launch, much less how to
>>>>> ensure line numbers are included!
>>>>> 
>>>>> Any idea how can I enable this?
>>>>> 
>>>>> --
>>>>> View this message in context:
>>>>> http://tapestry.1045711.n5.nabble.com/debugging-jumpstart-in-Eclipse-missing-line-numbers-tp4831779p4831779.html
>>>>> Sent from the Tapestry - User mailing list archive at Nabble.com.
>>>>> 
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
>>>>> For additional commands, e-mail: users-h...@tapestry.apache.org
>>>>> 
>>>>> 
>>>> 
>>>> 
>>>> --
>>>> Jonathan Barker
>>>> ITStrategic
>>> 
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
>>> For additional commands, e-mail: users-h...@tapestry.apache.org
>>> 
>>> 
>> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
> 

Reply via email to