beehive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Carlin Rogers (JIRA)" <...@beehive.apache.org>
Subject [jira] Updated: (BEEHIVE-1090) NetUI page flow compiler's WebappPathType.checkRelativePath() needs to handle paths to JPF that begin with "./"
Date Fri, 31 Mar 2006 17:56:46 GMT
     [ http://issues.apache.org/jira/browse/BEEHIVE-1090?page=all ]

Carlin Rogers updated BEEHIVE-1090:
-----------------------------------

    Attachment: j1090-repro.zip

Zip file with test to repro warnings that the JPF in the forward path could not be found...

> NetUI page flow compiler's WebappPathType.checkRelativePath() needs to handle paths to
JPF that begin with "./"
> ---------------------------------------------------------------------------------------------------------------
>
>          Key: BEEHIVE-1090
>          URL: http://issues.apache.org/jira/browse/BEEHIVE-1090
>      Project: Beehive
>         Type: Bug
>   Components: NetUI
>     Versions: 1.0.1
>     Reporter: Carlin Rogers
>     Assignee: Carlin Rogers
>     Priority: Minor
>      Fix For: v.next
>  Attachments: j1090-repro.zip
>
> Add an action such as the following to a page flow controller:
>     @Jpf.Action(forwards = {
>         @Jpf.Forward(name = "success",
>                      path = "./subdir/Controller.jpf")
>     })
>     protected Forward AddAction() { ... }
> As the jpf compiler does its thing, it translates external pageflow paths into type names
and then requests those types. Here it is incorrectly translating "./subdir/Controller.jpf"
to a type name "..subdir.Controller"
> which may cause an APT implementation to barf when we try to get a TypeDeclaration from
this name. With the Sun APT, we get a null type without an error, but then this causes us
to write a warning that we cannot find the page flow for the given path.
> The problem is in org.apache.beehive.netui.compiler.grammar.WebappPathType.checkRelativePath().
It does some stuff to look for "../" at the beginning of a path but does not handle "./".
> I'll add a zip file for the repro. The fix is coming shortly.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


Mime
View raw message