[wp-trac] [WordPress Trac] #41622: Issue with upgrading text widgets that contain html to 4.8.1

[WordPress Trac]

#41622: Issue with upgrading text widgets that contain html to 4.8.1
-------------------------------+------------------------------
 Reporter:  codezen8           |       Owner:
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  Awaiting Review
Component:  Widgets            |     Version:  4.8.1
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |     Focuses:
-------------------------------+------------------------------
Changes (by westonruter):

 * keywords:   => reporter-feedback


Comment:

 @codezen8 thank you for the examples.

 You're right, in order to test this you must create the new Text widgets
 in 4.7.x and then switch to 4.8.1 in order to see how the legacy mode
 detection applies.

 I tried creating those three widgets in 4.7.5 and when I switched to 4.8.1
 the first two widgets got recognized as legacy as expected, and the third
 one got migrated to visual as also expected. The only problem I saw was
 the PHP warnings raised by `loadHTML` but after saving each widget, the
 error went away since either TinyMCE fixed the malformed HTML or the
 legacy mode was permanently applied and so `loadHTML` no longer was
 invoked as part of the `is_legacy_instance` method.

 To confirm, is the presence of these “DOMDocument::loadHTML()” warnings
 the only issue you are reporting?

 Here are screenshots of what the widgets looked like after switching to
 4.8.1: https://cloudup.com/c-7cAvSo32n

--
Ticket URL: <https://core.trac.wordpress.org/ticket/41622#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform