Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

using the idea of the web_func to inject params to function #16

Merged
merged 8 commits into from
Dec 15, 2011

Conversation

zhuangxm
Copy link
Owner

@zhuangxm zhuangxm commented Dec 5, 2011

sometimes we need data of the request as parameters to be injected into function , for example: client-ip, here using the idea of the web-func provided in the branch context-export to implement.

@zhuangxm
Copy link
Owner Author

zhuangxm commented Dec 7, 2011

Using multimethod to generalize handle command options.
rename the option :params-checks to :params-check

@zhuangxm
Copy link
Owner Author

zhuangxm commented Dec 7, 2011

Ready for merging

@zhuangxm
Copy link
Owner Author

zhuangxm commented Dec 9, 2011

对options 的处理作了调整,允许扩展option,options 改成了vector, 由调用方保证次序. options 统一作为一项放在 cmd 的 :options 中.

@zhuangxm
Copy link
Owner Author

changed:
1 render-method-request return a new method-request, does not return nil;
2 change the text of unknown option.

robertluo added a commit that referenced this pull request Dec 15, 2011
using the idea of the web_func to inject params to function
@robertluo robertluo merged commit 4418c9e into master Dec 15, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants