[wp-trac] [WordPress Trac] #59195: deprecation notice triggered in post.php by passing null to parameter #2 in preg_match()

[WordPress Trac]

#59195: deprecation notice triggered in post.php by passing null to parameter #2 in
preg_match()
-------------------------------------------------+-------------------------
 Reporter:  nosilver4u                           |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  Media                                |     Version:  6.3
 Severity:  normal                               |  Resolution:
 Keywords:  php81 has-patch has-unit-tests       |     Focuses:  php-
  needs-testing                                  |  compatibility
-------------------------------------------------+-------------------------
Changes (by ironprogrammer):

 * keywords:  php81 changes-requested has-patch has-unit-tests => php81 has-
     patch has-unit-tests needs-testing


Comment:

 I agree, @nosilver4u, that nulls and blank MIME types are unhelpful return
 values here. Thanks for digging deeper into what was happening!

 I've pushed up a [https://github.com/WordPress/wordpress-develop/pull/6443
 PR] that tries to address the first option noted in comment:11, and adds
 test coverage to this function and the relatively new pre filter (added in
 6.4).

 Removing `changes-requested` for clarity in case it gets re-added for this
 new PR 😅.

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