Text widget does not show Title field or TinyMCE editor

[aholstrya]

Hello,
Further to the update towards the version of WP 4.8 I had a concern(marigold) with the widgets text which do not seem to support(bear) the function(office) " flex "...
Can you tell it to me more?
I am handed on to the version 4.7.5 and my widgets works again correctly
(Saddened for the translation Reverso, I do not speak English...)
I join(contact) you an image of widgets text (version 4.8) in version blocking.
Have a good day.
J-P Bourdet

[aholstrya]

blocked version of widget text

[westonruter]

@aholstrya Welcome. We'll need more information. It seems like the problem is that the title field and the TinyMCE editor aren't showing in the widget. What plugins do you have active?

[aholstrya]

Bonjour (hello !),

At first thank you for your answer

mes plugins :
Extensions en place :
Advanced Database Cleaner (2.0.0)
Akismet Anti-Spam (3.3.2)
Contact Form 7 (4.8)
Cookie Notice (1.2.38)
Disable Google Fonts (1.3)
Email Templates (1.2)
Enhanced Media Library (2.4.4)
Forum_wordpress_fr (4.1)
Hupso Share Buttons for Twitter, Facebook & Google+ (4.1.2), Imagify (1.6.4)
MaxButtons (6.19.1)
Really Simple SSL (2.5.16)
Responsive Lightbox (1.7.1)
Simple Page Ordering (2.2.4)
TinyMCE Advanced (4.5.6), To Top (1.5.3)
WooCommerce (3.0.8)
Yoast SEO (4.9)

The problem is not clean, with a basic theme, it works, with a theme child of the basic theme, it also works, but with my theme (child of Storefront) widgets works with the version 4.7.5 of WorPress but not with the version 4.8 ...

It could be an incompatibility with the function(office) " flex ", because it is having to add this function(office) in my style.css that I noticed the problem.
However I had not verified my widgets texts before making these modifications and cannot tell you if they worked correctly with the version 4.8 before my changes...

Hoping that this information will be useful for you, I hope that I could solve my problem and make the update towards carefree WP 4.8.
Thank you for your interest.
@aholstrya

[westonruter]

@aholstrya if you disable all plugins, does the Text widget work?

Also, do the media widgets work? Try the Video, Audio, and Image widgets.

If you can then try re-activating each plugin one-by-one and check if the Text widget stops working, that will help identify if there is a specific plugin incompatibility.

Merci !

[westonruter]

I got a separate report for a plugin incompatibility that had the same effect of the controls not appearing for the Text and media widgets, via @salvatorebalzano:

the problem is customize-widgets-plus plugin is not compatible with the way the new text widget gets registered. ( same applies for the new Media widgets )
​https://github.com/xwp/wp-customize-widgets-plus/blob/master/php/class-optimized-widget-registration.php#L124-L125
and
​https://github.com/xwp/wp-customize-widgets-plus/blob/master/php/class-optimized-widget-registration.php#L272-L273

new Text Widget uses _register() method to load TinyMCE editor and that method never gets called in customize-widget-plus

Ironically the incompatibility is with a plugin I wrote myself. 😞

I have a patch in 40951.0.diff​ with a proposed fix. Can you test?

[aholstrya]

Replying to westonruter:

@aholstrya if you disable all plugins, does the Text widget work?

Also, do the media widgets work? Try the Video, Audio, and Image widgets.

If you can then try re-activating each plugin one-by-one and check if the Text widget stops working, that will help identify if there is a specific plugin incompatibility.

Merci !

Hello,

none of the new widget works (sound, video, image, text).

Extensions were activated or were deactivated, the problem remains the same : Impossible to open widgets to settle them

[aholstrya]

Replying to westonruter:

I got a separate report for a plugin incompatibility that had the same effect of the controls not appearing for the Text and media widgets, via @salvatorebalzano:

the problem is customize-widgets-plus plugin is not compatible with the way the new text widget gets registered. ( same applies for the new Media widgets )
​https://github.com/xwp/wp-customize-widgets-plus/blob/master/php/class-optimized-widget-registration.php#L124-L125
and
​https://github.com/xwp/wp-customize-widgets-plus/blob/master/php/class-optimized-widget-registration.php#L272-L273

new Text Widget uses _register() method to load TinyMCE editor and that method never gets called in customize-widget-plus

Ironically the incompatibility is with a plugin I wrote myself.

I have a patch in 40951.0.diff​ with a proposed fix. Can you test?

@westonruter

I tested this solution, but no change...

[westonruter]

@aholstrya are there JS errors shown in the web console?

[aholstrya]

Replying to westonruter:

@aholstrya are there JS errors shown in the web console?

@westonruter

Sorry but I do not know how I could supply you this information.

[westonruter]

@aholstrya are you at WordCamp Europe by chance? It will be hard to identify the problem without seeing your installation myself.

Otherwise, you could give me access by sending me a private message on Slack.

[aholstrya]

Replying to westonruter:

@aholstrya are you at WordCamp Europe by chance? It will be hard to identify the problem without seeing your installation myself.

Otherwise, you could give me access by sending me a private message on Slack.

No I would not be in WordCamp, but I can actually give you an access to the site.
How do I have to make? Sorry but I don't know Slack

[westonruter]

Add missing $number param

[aholstrya]

Hello, I stayed with the new parameter (number), but it changes nothing the problem...
If you wish always how tell I to send you, in private, an access on the site.
Have a good day.

[westonruter]

@aholstrya Thanks for testing. You can sign up for Slack via the instructions here: ​https://make.wordpress.org/chat/

Otherwise we could use email.

[aholstrya]

Replying to westonruter:

@aholstrya Thanks for testing. You can sign up for Slack via the instructions here: ​https://make.wordpress.org/chat/

Otherwise we could use email.

Hello Sir..
OK for email...
In fact I know Slack, idiot whom I am, I joined it by means of the support(medium) of French-speaking WordPress but, having said that, I do not know how it works then let us leave rather on the basis of the email that I know!
And, otherwise, how find you the translations Reverso?

[westonruter]

@aholstrya I don't see you in WordPress Slack. Please email me at trac-wp-core-41021@xwp.co.

[aholstrya]

Replying to westonruter:

@aholstrya I don't see you in WordPress Slack. Please email me at trac-wp-core-41021@xwp.co.

@westonruter : OK it's done

[westonruter]

Unfortunately I was unable to access @aholstrya's install, so I'll need someone else to be able to reproduce the issue and share access.

Nevertheless, 40951.1.diff​ does fix the issue as I've been able to reproduce myself, fixing compatibility with a plugin. @obenland thoughts on the patch?

[obenland]

Heh, just the other day I was thinking how neat it would be if we had something like lodash's once() or jQuery's one() for hooks :)

What's the reason for moving it to _register_one()?

[westonruter]

@obenland basically the reason is that _register is a caller of _register_one, where the latter does the real work. As seen in the Customize Widgets Plus plugin above, it's the _register_one method that will invariably be called whereas _register could get bypassed.

[westonruter]

Replying to aholstrya:

Extensions en place

I tried installing and activating all of the plugins, and I also activated the Storefront theme, and I still cannot reproduce the issue. The Text widget shows the title field and the editor as expected.

[westonruter]

I'll go ahead and commit this, even though it may not fix the specific issue originally reported. It fixes an issue that I was able to reproduce that had the same symptoms.

[westonruter]

In 41028:

Widgets: Enqueue assets needed by media and text widgets in their _register_one() methods.

The WP_Widget::_register_one() method is more guaranteed to be called as opposed to its wrapper WP_Widget::_register() which plugins may bypass for performance reasons.

Amends [40631], [40640].
See #35243, #32417.
Fixes #41021.

[westonruter]

Re-opening for 4.8.1.

[westonruter]

In 41052:

Widgets: Enqueue assets needed by media and text widgets in their _register_one() methods.

The WP_Widget::_register_one() method is more guaranteed to be called as opposed to its wrapper WP_Widget::_register() which plugins may bypass for performance reasons.

Merges [41028] to the 4.8 branch.
Amends [40631], [40640].
See #35243, #32417.
Fixes #41021 for 4.8.1.