Issue Details (XML | Word | Printable)

Key: PULSE-245
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Martin Dietz
Reporter: Daniel Dietz
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
pulse

Results of Controller are being overridden if Controller provides @AnyAction-annotated web-method and a @Action-annotated web-method is being called

Created: 2011-01-17 16:46 UTC   Updated: 2011-07-06 16:42 UTC
Component/s: None
Affects Version/s: 0.7, 0.8
Fix Version/s: 0.8

Time Tracking:
Not Specified


 Description  « Hide
Problem:
If the Controller provides an @AnyAction-annotated web-method and a @Action-annotation is actually being called the expected result may be overridden depending on the alwaysRun-value of the current Controller-configuration in the bundle.xml.

Current state:

@AnyAction-annotated web-method exists and a @Action-annotated web-method is being called:

IF: @Action-annotated produces a stop-Event:

1. alwaysRun undefined:
ONLY the result of the @Action-annotated web-method will be returned
2. alwaysRun="pre"
ONLY the result of the @Action-annotated web-method will be returned
3. alwaysRun="post"
ONLY the result of the @Action-annotated web-method will be returned

ELSE:

1. alwaysRun undefined:
ONLY the result of the @Action-annotated web-method will be returned
2. alwaysRun="pre"
ONLY the result of the @Action-annotated web-method will be returned
3. alwaysRun="post"
ONLY the result of the @AnyAction-annotated web-method will be returned (if the Controller does have multiple @AnyAction-annotated web-methods only the result of the last executed web-method is being returned); the result of the @Action-annotated web-method will be overridden

Solution:
Provide an additional wrapper within the result of the Controller which wraps the the result for the actual web-method which was executed...

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Martin Dietz added a comment - 2011-05-18 15:39 UTC
adding fix