javaserverfaces
  1. javaserverfaces
  2. JAVASERVERFACES-2729

Composite action method prints PropertyNotFoundException in the log

    Details

      Description

      This issue is related to the fix applied for http://java.net/jira/browse/JAVASERVERFACES-1806.

      Look at the applied changeset - http://java.net/jira/secure/attachment/50275/changebundle-1806-trunk.txt

      If you look closely - @@ -187,11 +195,19 @@ there was a line:
      else

      { throw ele; so if the fallback worked, the exception got swollen However in the current implementation the exception stays thrown even if the fallback worked and even if the action method is sucessfully executed I get exception in the log. Suggested solution is to return the check that swallows the exception if the fallback is successfully executed like this: ContextualCompositeMethodExpression line 191 }

      catch (ELException ele) {
      if (source != null) {
      // special handling when an ELException handling. This is necessary
      // when there are multiple levels of composite component nesting.
      // When this happens, we need to evaluate the source expression
      // to find and invoke the MethodExpression at the next highest
      // nesting level. Is there a cleaner way to detect this case?
      try {
      Object fallback = source.getValue(elContext);
      if (fallback != null && fallback instanceof MethodExpression)

      { return ((MethodExpression) fallback).invoke(elContext, objects); }

      else

      { throw ele; }

      } catch(Exception ex) {
      if (LOGGER.isLoggable(Level.WARNING)) {
      LOGGER.log(Level.WARNING,
      "jsf.facelets.el.method.expression.invoke.error:"+ex.toString(),
      new Object[]

      { source.getExpressionString() }

      );
      }
      throw ele;
      }
      } else

      { throw ele; }

        Issue Links

          Activity

          There are no comments yet on this issue.

            People

            • Assignee:
              Manfred Riem
              Reporter:
              amitev
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: