[wp-trac] [WordPress Trac] #41410: Set `'filter' => 'content'` on starter content "business info" widget

WordPress Trac noreply at wordpress.org
Sun Jul 23 05:44:56 UTC 2017


#41410: Set `'filter' => 'content'` on starter content "business info" widget
--------------------------+--------------------------
 Reporter:  dlh           |       Owner:  westonruter
     Type:  defect (bug)  |      Status:  accepted
 Priority:  normal        |   Milestone:  4.8.1
Component:  Customize     |     Version:  4.8
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:
--------------------------+--------------------------
Changes (by westonruter):

 * owner:   => westonruter
 * status:  new => accepted
 * version:  trunk => 4.8
 * milestone:  Awaiting Review => 4.8.1


Comment:

 @dlh Wow, excellent find! I wonder if fundamentally instead of setting
 `visual` to `true` if we should instead change the `text` to cause it to
 such that legacy mode would not have been triggered to begin with? In
 [attachment:41410.3.diff] you can see what I mean. Strictly `'visual' =>
 true` (or `'filter' => 'content'`) is even not necessary here because the
 `text` would not match `is_legacy_instance` anyway.

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


More information about the wp-trac mailing list