[wp-trac] [WordPress Trac] #57246: Duotune filter svg assumes hex

WordPress Trac noreply at wordpress.org
Thu Dec 1 15:12:35 UTC 2022


#57246: Duotune filter svg assumes hex
--------------------------+-----------------------------
 Reporter:  mattf10       |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  General       |    Version:
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-----------------------------
 The function 'wp_get_duotone_filter_svg' and/or
 'wp_tinycolor_string_to_rgb' do not verify the format of the color code
 passed to it.   My theme uses a css variable "var(--nv-text-dark-bg)".
 After getting '$color' from 'string_to_rbg', it assumes the color array
 has valid values.   Since the original color is not one of the expected
 formats, this generates the following warnings:

  PHP Warning:  Trying to access array offset on value of type null in /wp-
 includes/block-supports/duotone.php on line 422PHP message: PHP Warning:
 Trying to access array offset on value of type null in /wp-includes/block-
 supports/duotone.php on line 423PHP message: PHP Warning:  Trying to
 access array offset on value of type null in /wp-includes/block-
 supports/duotone.php on line 424PHP message: PHP Warning:  Trying to
 access array offset on value of type null in /wp-includes/block-
 supports/duotone.php on line 425

 Please change one or the other method to check for css variables before
 using values from $color array.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57246>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list