Skip to content

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

Closed
wants to merge 3 commits into from

Conversation

slavafomin
Copy link

No description provided.

* `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.
Copy link
Member

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.

@weaverryan
Copy link
Member

@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>`_
Copy link
Member

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>`
* ...

Slava Fomin added 2 commits March 11, 2014 13:36
Fixed spelling.
Updated references to PHP functions using syntax suggested by @wouterj (`:phpfunction:` role).
@slavafomin
Copy link
Author

@weaverryan, @wouterj thanks for suggestions! I've introduced necessary changes. I hope my syntax is correct.

@weaverryan
Copy link
Member

It looks great. Thanks so much for adding these details and making prompt tweakss ! Cheers Slava!

weaverryan added a commit that referenced this pull request Mar 13, 2014
…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
@weaverryan weaverryan closed this Mar 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants