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.

Sign up for GitHub

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

Jump to bottom

Emoji in entry titles throw a DB errors #2513

Closed
leevigraham opened this issue Mar 1, 2018 · 1 comment
Closed

Emoji in entry titles throw a DB errors #2513

leevigraham opened this issue Mar 1, 2018 · 1 comment
Assignees
@MakeilaLundy @Radabaugh
Labels
bug severity:minor Cosmetic issues or bugs with simple workarounds

Comments

@leevigraham
Copy link
Contributor

Steps to reproduce

  1. Create an entry
  2. Add an emoji to the title 🕷🐨

Additional info

  • Craft version: rc-13
  • PHP version: 7.1
  • Database driver & version: MariaDB 10.2.13
@leevigraham leevigraham mentioned this issue Mar 1, 2018
Closed
@brandonkelly
Copy link
Member

We only support emoji in titles in PostgreSQL, but i suppose we could at least check for it and set a validation error.

@brandonkelly brandonkelly added bug severity:minor Cosmetic issues or bugs with simple workarounds labels Mar 1, 2018
@bramus bramus mentioned this issue Mar 4, 2018
Closed
MakeilaLundy added a commit that referenced this issue Mar 7, 2018
@MakeilaLundy
Referenced #2513
902f9b7
brandonkelly added a commit that referenced this issue Mar 8, 2018
@brandonkelly
Fix #2513 for all element types
2e05ca4
@khalwat khalwat mentioned this issue Apr 16, 2018
Closed
@snyk-bot snyk-bot mentioned this issue Jul 29, 2020
Closed
@engram-design engram-design mentioned this issue Jun 9, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MakeilaLundy MakeilaLundy

@Radabaugh Radabaugh

Labels
bug severity:minor Cosmetic issues or bugs with simple workarounds
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@leevigraham @brandonkelly @MakeilaLundy @Radabaugh