Recently migrated to TP

[@rjen]

The current function replaces spaces with dashes: for backward compatibility it would be nice to keep that

[tino]

Ok, I've added a pattern on the input field and correctly decoded the entity on display.

https://github.com/Tinyportal/TinyPortal/pull/979/files

We can repeat the pattern on any other instance we want to limit.

[@rjen]

Yep that will do it. Thanks

Now we still have the issue with the nam field

[tino]

Now we still have the issue with the nam field

What issue? That should be fixed if you mean the displaying with quotes.

[@rjen]

When was that fixed?

[tino]

When was that fixed?

In my commit just now, with the input fix.

[@rjen]

okay, let me check... I only saw the code on my phone...

[@rjen]

NO quite...

when I create an articel with this a title and short name

Experts Say 'Millennials' Are Killing The Pickle Industry

It still breaks..

I cannot access the article, because this URL is generated...

https://test.fjr-club.nl/index.php?page=Experts-Say-%27Millennials%27-Are-Killing-The-Pickle-Industry

[@rjen]

its the space you included that does not work... let's see if I can fix it...

this work...

pattern="[A-Za-z0-9 ]+"

[@rjen]

had some issues with allowing spaces and - characters. It seems the following is better: just excluding the use of quotes...

<input type="text" id="tp_article_shortname" name="tp_article_shortname" value="'.$mg['shortname'].'" size=20 pattern="[^\'\x22]+" >