Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed problem with needless seperator at begin/end of string #252

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

davidbehler commented Aug 21, 2011

If you use url_title on strings with special characters at the begin/end, you end up with a needless seperator.

url_title('%&/ fu bar', 'dash', TRUE); // would become -fu-bar
url_title('fu bar %&/', 'dash', TRUE); // would become fu-bar-
Contributor

philsturgeon commented Aug 21, 2011

I merged this already.

Contributor

gaker commented Aug 21, 2011

Preparing unit tests in the feature/unit-tests branch for this as well :D

Dentxinho pushed a commit to Dentxinho/CodeIgniter that referenced this pull request Sep 28, 2012

Merge pull request #252 from tagawa/sql-edits
Enlarged IP address DB columns to hold IPv6 addresses.

Dentxinho pushed a commit to Dentxinho/CodeIgniter that referenced this pull request Sep 28, 2012

Reverted IP address field length changes from pull request #252 since…
… this is not needed when converting to binary. Thanks @Militis!

Dentxinho pushed a commit to Dentxinho/CodeIgniter that referenced this pull request Sep 28, 2012

Reverted IP address field length changes from pull request #252 since…
… this is not needed when converting to binary. Thanks @Militis!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment