[wp-trac] [WordPress Trac] #55121: classic widgets with no settings won't show up in 5.9

WordPress Trac noreply at wordpress.org
Fri Feb 11 09:07:57 UTC 2022


#55121: classic widgets with no settings won't show up in 5.9
----------------------------+----------------------
 Reporter:  joncampbell     |       Owner:  (none)
     Type:  defect (bug)    |      Status:  closed
 Priority:  normal          |   Milestone:
Component:  Widgets         |     Version:  5.9
 Severity:  normal          |  Resolution:  invalid
 Keywords:  has-patch php8  |     Focuses:
----------------------------+----------------------

Comment (by andraganescu):

 Hey folks adding an extra opinion. The way I read this problem is:

 - plugins do something which is wrong, overloading a method and returning
 something unexpected to core
 - nevertheless that ''used to work'', which makes things more complicated
 - because we are fixing a problem by creating another one
 - and we can avoid this with some harmless extra code

 I agree with @jrf in that "is not WordPress' job to be forgiving for every
 single type of developer error", but @peterwilsoncc also has a point that
 "WordPress's often stated goal is to maintain backward compatibility".

 So, in my head, we should **try** fixing ''for both situations'': make PHP
 8 not break and bad plugins continue to function, because it is possible,
 unless this is impossible. The decision to not support back compat
 problems is valid only if:

 - the back compat code is likely to create future problems
 - or if it is impossible today to both support back compat and the
 present.

 The present or the future are more important than the past only if the
 past prevents us from accessing the present or the future. Otherwise we
 own our past, right? :)

 So, as far as I undestand, the change appearing in 5.9 is ''also'' a
 backward compatibility problem because it worked "doing it the wrong way"
 before 5.9. To be clear, I am not a fan of this kind of patching, it's
 obvious that we're trying to provide for a principial developer error,
 even the OP decided this ticket can be closed. But patching to include
 both PHP 8 safety ''and'' back compat is owning to the fact that WP was
 not as strict as it could have been years ago.

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


More information about the wp-trac mailing list