[PEPr] Comment on Console::Console_Wizard
| From: | Clay Loveless | Date: | Sun, 31 Jul 2005 20:33:27 +0000 |
| Subject: | [PEPr] Comment on Console::Console_Wizard | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-39064@lists.php.net to get a copy of this message | ||
Clay Loveless (http://pear.php.net/user/clay) has commented on the proposal for
Console::Console_Wizard.
Comment:
Thanks, Bertrand. Regarding your points:
-camelCase instance variables
I have looked through the CS guidelines, and I am familiar with the
camelCase rule for method names. However, there is no rule that I could
find regarding variable names. So, I've written variables_like_this and
methodsLikeThis to make it very easy to tell them apart at a glance.
- Index types settable on per-question basis
This is a good idea, and I just realized I forgot to put it in alpha2. I
will add this feature for alpha3.
- option/config arrays for constructor, singleton() and addQuestion()
Great idea, done!
- previous answer to setup next question
Unless I misunderstand you, this is already possible with the
setNextQuestion methods. Both the Question and Answer classes have this
method, allowing for a default next question, and the setting of a
specific next question based on a specific answer.
- xml wizard definition
Good idea, I'm planning on this for a future release. I'd like to make the
format compatible with the XML format used in defining post-install scripts
in PEAR 1.4.0.
- localizable text
Done!
- addGroupRule empty
This is still on my to-do list. Working on a good way to handle this, and
will include it in the next alpha release.
Thanks again, Bertrand!
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=273
--
Sent by PEPr, the automatic proposal system at http://pear.php.net