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

Fix DB.inc.php/Emoji support #167

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Amniotic3
Copy link

If the note contains emoji emojis, then an error will be reported:.

Invalid character in note

Source line 1834 of /var/www/zotero/model/Item.inc.php

After confirming that the database encoding is set correctly, look for errors in Zotero_DB ::query(), after testing, it is determined that the encoding is set incorrectly in line 252 of /var/www/zotero/include/DB.inc.php, and the error is eliminated by changing utf8 to utf8mb4.

If the note contains emoji emojis, then an error will be reported:.

Invalid character in note

Source line 1834 of /var/www/zotero/model/Item.inc.php

After confirming that the database encoding is set correctly, look for errors in Zotero_DB ::query(), after testing, it is determined that the encoding is set incorrectly in line 252 of /var/www/zotero/include/DB.inc.php, and the error is eliminated by changing utf8 to utf8mb4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant