Check for required value is now validates emptiness of next argument

[dlussky]

Hi. There was an issue with checking required value:

1. Write in your php script $GetOptionKit->add('t|test:', 'required value');
2. Run it as follows:
   ./script --test
3. There is no exception on parsing args.
4. There, I fixed it!)

P.S. If I've misunderstood concept of 'option requires a value.', I'm sorry.

[c9s]

Hi @dlussky Thanks for sending this pull request!

[dlussky]

You are welcome!
Can you please make a new version out of this, so I can use strict version number in my composer.json?

Thank you for this lib, by the way. I like the flexibility)

[c9s]

Hi @dlussky

No problem. Version 1.2.4 was released, please check.

Happy Hacking
Pedro

[dlussky]

Thanks!)
But I need one more favor)) Could you please also update your package here? https://packagist.org/packages/c9s/GetOptionKit
I would be completely happy))

[c9s]

Hi! It's updated now. :)
On Jul 18, 2013, at 8:51 PM, Kirill Dlussky notifications@github.com wrote:

Thanks!)
But I need one more favor)) Could you please also update your package here? https://packagist.org/packages/c9s/GetOptionKit
I would be completely happy))

—
Reply to this email directly or view it on GitHub.

[dlussky]

Awesome! Thank you very much!)

[c9s]

No problem, my pleasure.

Kirill Dlussky 於 2013年7月18日星期四寫道：

Awesome! Thank you very much!)

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-21180919
.

Best Regards,

Yo-An Lin