Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Added documentation for missing ctype extension#3601
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| * `upper <http://php.net/function.ctype-upper.php>`_ | ||
| * `xdigit <http://php.net/function.ctype-xdigit.php>`_ | ||
| Make sure that proper `locale <http://php.net/function.setlocale.php>`_ is set before using one of those. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Make sure that the proper...
... before using one of these.
@slavafomin 👍 I didn't know about this feature :). I've just added a few notes, and can you also move the links to the bottom of the page. Your syntax for external links is correct, we just have a standard of putting all the URLs at the bottom of each file. Thanks! |
| * `punct <http://php.net/function.ctype-punct.php>`_ | ||
| * `space <http://php.net/function.ctype-space.php>`_ | ||
| * `upper <http://php.net/function.ctype-upper.php>`_ | ||
| * `xdigit <http://php.net/function.ctype-xdigit.php>`_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
instead of the links, you should use the:phpfunction: role:
*:phpfunction:`ctype_alnum`*:phpfunction:`ctype_alpha`* ...
Or if you don't want to use the function name:
*:phpfunction:`alnum <ctype_alnum>`*:phpfunction:`alpha <ctype_alpha>`* ...
Fixed spelling.
Updated references to PHP functions using syntax suggested by@wouterj (`:phpfunction:` role).
@weaverryan,@wouterj thanks for suggestions! I've introduced necessary changes. I hope my syntax is correct. |
It looks great. Thanks so much for adding these details and making prompt tweakss ! Cheers Slava! |
…omin)This PR was submitted for the 2.4 branch but it was merged into the 2.3 branch instead (closes#3601).Discussion----------Added documentation for missing ctype extensionCommits-------51cfb23 Updated references to PHP functionsbc439ff Fixed spelling1d03b0e Added documentation for missing ctype extension
No description provided.