?
avatar franzpeter
franzpeter
28 Feb 2018

Steps to reproduce the issue

Create a new article with a title containing a comma for example as title: My title, long title and click on SAVE button. Joomla will create an alias like my-title,-long-title. Question is: is comma allowed in alias?

Expected result

comma filtered out while auto create alias

Actual result

with comma

System information (as much as possible)

Joomla 3.8.5

Additional comments

I do not know if that is intended or fine to have comma in alias.

avatar franzpeter franzpeter - open - 28 Feb 2018
avatar joomla-cms-bot joomla-cms-bot - labeled - 28 Feb 2018
avatar brianteeman
brianteeman - comment - 28 Feb 2018

I tested the creation of an alias in both an article and a menu item
The title I used was new, old, and brave and in both cases the alias that was generated was new-old-and-brave

avatar Bakual
Bakual - comment - 28 Feb 2018

Depends if "Unicode Alias" are enabled or not. If enabled, the alias may contain a coma.
I guess that is ok since comas aren't invalid URL characters to my knowledge.

avatar franzpeter
franzpeter - comment - 28 Feb 2018

It seems to be a server issue. I use php 7.2.2 with Plesk and php is running as FPM application served by nginx. Did test on localhost without Nginx and using php 7.2.1 cgi-fcgi, issue does not occur. Did switch the live server to use php with FastCGI served by Apache, same issue. So maybe is php 7.2.2


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19803.

avatar brianteeman
brianteeman - comment - 28 Feb 2018

@Bakual - good catch yes it does require unicode alias to be on for this to occur

avatar mbabker
mbabker - comment - 28 Feb 2018

Personally I don't see the issue with it so long as commas are valid characters in URI slugs, but if anyone does want to change the behavior send a PR changing this method.

avatar franzpeter
franzpeter - comment - 28 Feb 2018

Yes, if switching off unicode alias it does not occur. But if commas are valid characters in slugs in case of activated unicode alias I think the issue could get closed. Thanks.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19803.

avatar brianteeman
brianteeman - comment - 28 Feb 2018

a comma is a reserved character as a sub-delimiter http://www.ietf.org/rfc/rfc3986.txt

avatar Bakual
Bakual - comment - 1 Mar 2018

Afaik, comas are allowed (not reserved) in the path part of an URI. Which is the case here. It's up to the application if it is used as delimiter or just a generic char.

avatar joomla-cms-bot joomla-cms-bot - change - 1 Mar 2018
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 1 Mar 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 1 Mar 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-03-01 06:23:53
Closed_By franz-wohlkoenig
avatar joomla-cms-bot
joomla-cms-bot - comment - 1 Mar 2018
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 Mar 2018

Issue closed as stated by @franzpeter (Comment)

Add a Comment

Login with GitHub to post a comment