struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lukasz Lenart (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (WW-4424) When a property doesn't exists, the log warn message always misses information
Date Tue, 23 Dec 2014 21:19:14 GMT

     [ https://issues.apache.org/jira/browse/WW-4424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Lukasz Lenart resolved WW-4424.
-------------------------------
    Resolution: Fixed
      Assignee: Lukasz Lenart

Patch applied, thanks!

> When a property doesn't exists, the log warn message always misses information
> ------------------------------------------------------------------------------
>
>                 Key: WW-4424
>                 URL: https://issues.apache.org/jira/browse/WW-4424
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Value Stack
>    Affects Versions: 2.3.16.3
>            Reporter: Brendan Le Ny
>            Assignee: Lukasz Lenart
>            Priority: Trivial
>              Labels: easyfix, patch
>             Fix For: 2.3.21
>
>         Attachments: WW-4424.patch
>
>
> As a developer, I write long JSPs and sometimes I make mistakes. For example, i can write
the line underneath:
> {code:xml}
> <s:if test="ghamRestitution.moraHalfFoodFree">blah</s:if>
> {code}
> if the the field "moraHalfFoodFree" doesn't exists on bean "ghamRestitution", XWork raise
a warn such as:
> {quote}
> 2014/11/25 11:45:08  WARN (CommonsLogger.java:56) warn Could not find property [null]
> {quote}
> The problem is that the message always say "Count not find property [null]" and it's
very hard for the developer to know where the bug is in a long JSP.
> {code:title=com.opensymphony.xwork2.ognl.OgnlValueStack}
>     private Object handleOgnlException(String expr, boolean throwExceptionOnFailure,
OgnlException e) {
>         Object ret = findInContext(expr);
>         if (ret == null) {
>             if (shouldLogMissingPropertyWarning(e)) {
>                 LOG.warn("Could not find property [" + ((NoSuchPropertyException) e).getName()
+ "]");
>             }
>             if (throwExceptionOnFailure) {
>                 throw new XWorkException(e);
>             }
>         }
>         return ret;
>     }
> {code}
> {quote}
> ((NoSuchPropertyException) e).getName() 
> {quote}
> always returns null, instead we should use expr parameter which give the right information.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message