Skip to content
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

Non-english tags don't work after submitting #89

Open
KotM opened this issue Mar 27, 2017 · 5 comments
Open

Non-english tags don't work after submitting #89

KotM opened this issue Mar 27, 2017 · 5 comments

Comments

@KotM
Copy link

KotM commented Mar 27, 2017

I've modified regex to support Cyrillic symbols: /^[\- a-zа-я0-9]{3,30}$/i.

When I'm trying to specify russian tags (e.g. тест), it seems to be working: no errors on UI. But on topic submit the following errors occurred:

  1. Board locale suddenly switched to English;
  2. The following error displayed: The following tags are invalid: тест.

phpBB 3.2.0, table collation is utf8_bin.

@RobertHeim
Copy link
Owner

Did you try this on a clean installation of 3.2.0 or do you have other extensions installed?

@RobertHeim
Copy link
Owner

and which version of the extension did you use?

@KotM
Copy link
Author

KotM commented Mar 27, 2017

I have much extensions installed right now. I can try to check it later on clean installation, tomorrow evening. I have latest extension version installed from GitHub (downloaded 3-5 hours earlier).

@KotM
Copy link
Author

KotM commented Mar 28, 2017

The same isue on clean board installation. I'm pretty sure it is related to php operations with utf8 and other codepages. Some addons sometime has an issue with storing input Cyrillic data as "?????" values in database. I don't know how to fix it, but sure it is known issue which should have much suggestions in kb.

@wcz-txp
Copy link

wcz-txp commented Apr 3, 2017

Hi, I had the same problem on a 3.1 installation with Cyrillics and German umlauts.
/^[\- üäößa-z0-9а-я]{3,30}$/iu
adding u helped me out, but to have all unicode letters \p{L} should be used, however I couldn't get it to work.

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

No branches or pull requests

3 participants