[jbosstools-issues] [JBoss JIRA] Updated: (JBIDE-1280) JBoss Tools JSP Editor does not process some problems in vpe templates in a user friendly way

[Max Andersen (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBIDE-1280?page=all ]

Max Andersen updated JBIDE-1280:
--------------------------------

    Priority: Critical  (was: Major)

Sounds like two issues here:

1) why does template code not handle broken userinput gracefully

2) why do we have exceptions that subclass error?

both should be fixed.

> JBoss Tools JSP Editor does not process some problems in vpe templates in a user friendly way
> ---------------------------------------------------------------------------------------------
>
>                 Key: JBIDE-1280
>                 URL: http://jira.jboss.com/jira/browse/JBIDE-1280
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: Visual Page Editor
>    Affects Versions: 2.0.0.Beta4
>            Reporter: Viacheslav Kabanovich
>         Assigned To: Sergey Vasilyev
>            Priority: Critical
>             Fix For: 2.1
>
>
> 1. provide an incorrect vpe expression in some vpe template.
> (I used <span title="{tagstring()|tagstring()}" /> instead of <span title="{tagstring()}" /> 
> for standard jsf tag templates in vpe-templates-jsf.xml).
> 2. prepare jsp file that uses the corrupted template.
> 3. open jsp file with JBoss Tools JSP Editor.
> Result: blank visual editor and empty Eclipse log. 
> 4. Select Preview tab
> Result: tab is empty and log contains Error
> org.jboss.tools.vpe.editor.template.expression.VpeExpressionError: Error of type conversion
> 	at org.jboss.tools.vpe.editor.template.expression.VpeValue.or(VpeValue.java:94)
> 	at org.jboss.tools.vpe.editor.template.expression.VpeOrOperation.exec(VpeOrOperation.java:26)
> 	at org.jboss.tools.vpe.editor.template.VpeAttributeCreator.create(VpeAttributeCreator.java:54)
> The Error is caught by Workbench.
> The question is: why most problems in vpe expressions throw VpeExpressionBuilderException 
> and some problems throw VpeExpressionError (which extends java.lang.Error)?
> Comment to java.lang.Error says that 
> "An Error is a subclass of Throwable that indicates serious problems 
> that a reasonable application should not try to catch"
> As shown above, in cases when Studio does not catch VpeExpressionError, user has no idea what is wrong, 
> and in cases when Eclipse catches it, it would be more proper to use something like 
> VpeExpressionExecException extending Exception rather than Error.

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