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

Already on GitHub? Sign in to your account

fix for Main::getParams(), #167 #178

Merged
merged 1 commit into from Jun 29, 2012

Conversation

Projects
None yet
2 participants
Owner

clayhinson commented Jun 29, 2012

moved the contents of getParams() into a new method setParams(), which is only called once in the constructor.
now getParams() is a simple getter rather than a mixed getter/setter. also removed mixing of $this->request and $this->params

@clayhinson clayhinson getParams now only returns the parsed params. fixes #167
moved the contents of getParams() into a new method setParams(), which is only called once in the constructor.
now getParams() is a simple getter rather than a mixed getter/setter.  also removed mixing of $this->request and $this->params
49e21ad

@helgi helgi added a commit that referenced this pull request Jun 29, 2012

@helgi helgi Merge pull request #178 from clayhinson/master
fix for Main::getParams(), #167
83ce5d9

@helgi helgi merged commit 83ce5d9 into frapi:master Jun 29, 2012

@MunGell MunGell pushed a commit to MunGell/frapi that referenced this pull request Mar 6, 2015

@helgi helgi Merge pull request #178 from clayhinson/master
fix for Main::getParams(), #167
5adad03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment