-
-
Notifications
You must be signed in to change notification settings - Fork 5.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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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 extension Commits ------- 51cfb23 Updated references to PHP functions bc439ff Fixed spelling 1d03b0e Added documentation for missing ctype extension
No description provided.