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

Problem with Cyrillic(UTF-8) #14884

Closed
Aleksst95 opened this issue Feb 29, 2020 · 6 comments
Closed

Problem with Cyrillic(UTF-8) #14884

Aleksst95 opened this issue Feb 29, 2020 · 6 comments
Labels
bug A bug report duplicate Duplicate issue. The duplicate issue is referenced in the comments

Comments

@Aleksst95
Copy link

Aleksst95 commented Feb 29, 2020

Describe the bug

I have a route: $api->add('/is_unique_term/{term}', ['action' => 'isUniqueTerm']);
This is my action for the route: public function isUniqueTermAction(string $term)
When i try to send cyrillic characters(UTF-8) in the term part Phalcon removes my characters and heads to other route.
Example of URI: /is_unique_term/фалкон.
Instead of my route is described above Phalcon heads to /is_unique_term (if this exists of course).

@ruudboon
Copy link
Member

@Aleksst95 Can you add the versions (php + phalcon) you are using and some example cyrillic characters?

@ruudboon ruudboon added bug A bug report status: unverified Unverified labels Feb 29, 2020
@Aleksst95
Copy link
Author

Yes, I already updated my issue(I accidentally clicked "submit new issue" before when it was not ready).
Phalcon version: 3.4.4
I'm not sure about version php, because I don't have access to a server with the app now.
But this should be 7.2 or 7.3(it is more likely).

@ruudboon
Copy link
Member

@Aleksst95 With normal alpha numeric characters this works fine?

@Aleksst95
Copy link
Author

Yes. It will work even if I add just one latin character. For example: /is_unique_term/qфалкон or /is_unique_term/фалконq.
But Phalcon removes utf-8 characters anyway. I'll get just 'q' in $term variable.

@ruudboon
Copy link
Member

@Aleksst95 Thnx for reporting. I will check if the happens in Phalcon 4 as well.

@sergeyklay sergeyklay self-assigned this May 2, 2020
@sergeyklay sergeyklay removed their assignment Jul 29, 2020
@zsilbi zsilbi added 4.1.0 duplicate Duplicate issue. The duplicate issue is referenced in the comments and removed status: unverified Unverified labels Oct 20, 2020
@zsilbi
Copy link
Member

zsilbi commented Oct 20, 2020

Duplicate: #15102
Fixed in #15162

@zsilbi zsilbi closed this as completed Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug report duplicate Duplicate issue. The duplicate issue is referenced in the comments
Projects
None yet
Development

No branches or pull requests

4 participants