WPIntell

Source evidence

Keep old smilies

Emoji Settings · support · 2019-02-01T23:41:00+00:00

mixedsentiment
mediumseverity
0.84relevance
3replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

7 / 32 rows with source links

21.9% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

25 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Ammaletu resolved
Hi! I’m trying to get rid of the s.w.org call, that’s why I tried out your plugin. I don’t care about emojis, but want to keep my old smilies as images. Now the function “wp430_support” disables the smilies when I disable emojis. As I understand it, these are two distinct features, and when I comment your function out, everything seems to work as intended. Can you give some more information what the function’s docblock means by “This Prevents unreadable character output caused by ‘use_smilies’ option”? Thanks a lot! Greetings, Johannes Hi Johannes, Thanks for reaching out, and my apologies for such a late reply! In the 1.0.5 update, it reads (I should’ve used a spellchecker…): Fixed: New WordPress installations (4.3 and up) don’t have the option to turn off smileys. This leads to incorrect character encoding of smiley abbrev[i]ations, like 🙂 and :D . Therefor[e] the whole function to encode characters will be removed if emojis are set to “disabled”. Fixed: Old WordPress installations with WordPress 4.3 and up will automatically set the smileys to off if Emoji support is disabled for the same reason as above. This only has effect after updating the page for the first time. To summarize, it means that there’s a discrepancy between old and new WordPress installations, and that function accounts for it as a catch-all; so to alleviate confusion and general annoyance: People who use this plugin often don’t want smileys at all. Nevertheless, you can override the [ cw_emoji_settings(), 'wp430_support' ] callback like so ( code ref ): // Adds a sanitation filter to the 'enable_emoji' setting; runs right after it's done sanitizing (priority 11). add_filter( 'sanitize_option_enable_emoji', function( $options ) { // Reenable the option after Emoji Settings disables it. if ( isset( $_POST['use_smilies'] ) && '1' === $_POST['use_smilies'] ) update_option( 'use_smilies', '1' ); // Return the Emoji Settings options. return $options; }, 11 ); I hope this helps 🙂 Cheers! This reply was modified 7 years, 3 months ago by Sybre Waaijer . Reason: Added POST value check for the option Hi, thanks for the reply and the added filter possibility! I ended up deactivating the emoji support manually for now, but it’s nice to know that I could still use your plugin if disabling the emojis would change in future WordPress versions. Also, my smilies all have “noses”, so that’s probably why I don’t have any encoding problems so far. 🙂 Greetings, Johannes It’s nice to know this wasn’t a big hindrance after all; I’m glad you’ve found a workaround 🙂 Have a great week!

Comments

3 shown
Sybre Waaijer 2019-02-15T10:26:00+00:00

Hi Johannes, Thanks for reaching out, and my apologies for such a late reply! In the 1.0.5 update, it reads (I should’ve used a spellchecker…): Fixed: New WordPress installations (4.3 and up) don’t have the option to turn off smileys. This leads to incorrect character encoding of smiley abbrev[i]ations, like 🙂 and :D . Therefor[e] the whole function to encode characters will be removed if emojis are set to “disabled”. Fixed: Old WordPress installations with WordPress 4.3 and up will automatically set the smileys to off if Emoji support is disabled for the same reason as above. This only has effect after updating the page for the first time. To summarize, it means that there’s a discrepancy between old and new WordPress installations, and that function accounts for it as a catch-all; so to alleviate confusion and general annoyance: People who use this plugin often don’t want smileys at all. Nevertheless, you can override the [ cw_emoji_settings(), 'wp430_support' ] callback like so ( code ref ): // Adds a sanitation filter to the 'enable_emoji' setting; runs right after it's done sanitizing (priority 11). add_filter( 'sanitize_option_enable_emoji', function( $options ) { // Reenable the option after Emoji Settings disables it. if ( isset( $_POST['use_smilies'] ) && '1' === $_POST['use_smilies'] ) update_option( 'use_smilies', '1' ); // Return the Emoji Settings options. return $options; }, 11 ); I hope this helps 🙂 Cheers! This reply was modified 7 years, 3 months ago by Sybre Waaijer . Reason: Added POST value check for the option

Ammaletu 2019-02-15T10:31:00+00:00

Hi, thanks for the reply and the added filter possibility! I ended up deactivating the emoji support manually for now, but it’s nice to know that I could still use your plugin if disabling the emojis would change in future WordPress versions. Also, my smilies all have “noses”, so that’s probably why I don’t have any encoding problems so far. 🙂 Greetings, Johannes

Sybre Waaijer 2019-02-15T10:34:00+00:00

It’s nice to know this wasn’t a big hindrance after all; I’m glad you’ve found a workaround 🙂 Have a great week!