Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 2.0
    • Component/s: Uncategorized
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      200
    • Status Whiteboard:
      Hide

      EGEasy5

      Show
      EGEasy5

      Description

      Section 7.3 describes the method signature requirements for an action. In 1.2
      we changed the return value from String to Object.

      From what I have seen, if the method in question returns void, everything will
      still work. This is because MethodExpression.invoke() returns null for void
      methods.

      So, I don't think we need to exclude void as a return type for actions.

        Issue Links

          Activity

          Hide
          rogerk added a comment -

          Status Whiteboard

          Show
          rogerk added a comment - Status Whiteboard
          Hide
          rogerk added a comment -

          Can you give me a specific use cse for this?
          The onlu use case I can think of is if you want to perform the action but
          don't navigate anywhere after the action (the outcome will be null).
          But you can do this with the same "Object return type" method signature..

          Show
          rogerk added a comment - Can you give me a specific use cse for this? The onlu use case I can think of is if you want to perform the action but don't navigate anywhere after the action (the outcome will be null). But you can do this with the same "Object return type" method signature..
          Hide
          pmuir added a comment -

          If we add conditional support to navigation rules (<if/> or <conditional/>) as
          recently discussed this is very useful

          Show
          pmuir added a comment - If we add conditional support to navigation rules (<if/> or <conditional/>) as recently discussed this is very useful
          Hide
          Ed Burns added a comment -

          target 2.0

          Show
          Ed Burns added a comment - target 2.0
          Hide
          Ed Burns added a comment -

          take ownership

          Show
          Ed Burns added a comment - take ownership
          Hide
          Ed Burns added a comment -

          fixed.

          Show
          Ed Burns added a comment - fixed.
          Hide
          Ed Burns added a comment -

          Prepare to delete "spec" subcomponent.

          Show
          Ed Burns added a comment - Prepare to delete "spec" subcomponent.
          Hide
          Manfred Riem added a comment -

          Closing resolved issue out

          Show
          Manfred Riem added a comment - Closing resolved issue out

            People

            • Assignee:
              Ed Burns
              Reporter:
              ssilvert
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: