search

ufront / ufront-mvc

The core MVC framework that powers ufront
MIT License
17 stars 15 forks source link

AddClientAction should return ViewResult. #57

Closed postite closed 8 years ago

postite commented 8 years ago

hello ! too shy to make a PR but why not :

new AddClientActionResult( originalResult, clientAction, data );
return originalResult;

instead of 

return  new AddClientActionResult( originalResult, clientAction, data );

at https://github.com/ufront/ufront-mvc/blob/master/src/ufront/web/result/AddClientActionResult.hx#L36

actual code breaks chaining

postite commented 8 years ago

forget this .. it won't work.