[wp-trac] [WordPress Trac] #47699: Remove redundant JSON polyfills for PHP native functionality

WordPress Trac noreply at wordpress.org
Wed Sep 25 12:34:00 UTC 2019


#47699: Remove redundant JSON polyfills for PHP native functionality
-------------------------------------------------+-------------------------
 Reporter:  jrf                                  |       Owner:  desrosj
     Type:  task (blessed)                       |      Status:  reviewing
 Priority:  normal                               |   Milestone:  5.3
Component:  General                              |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests needs-dev-  |     Focuses:  coding-
  note close                                     |  standards
-------------------------------------------------+-------------------------

Comment (by jrf):

 @dd32 Makes perfect sense to me and makes life so much easier in this
 case. Whoever thought this up: 💖

 As you already drafted that last patch which is needed for this ticket,
 would you like to upload a patch or would you prefer I work this out
 further ?

 Should this be set up in a way that both Site Health as well as the
 `update-core.php` file can use the same list of required extensions ?
 I'm in two minds about this as Site Health does a check ''after''
 install/upgrade, while `update-core` will block upgrades ''before'' they
 are rolled out.

 Maybe an inline comment in Site Health to update the list in `update-core`
 whenever an extension gets set to `required` and visa versa would suffice
 ?

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


More information about the wp-trac mailing list