[buddypress-trac] [BuddyPress Trac] #7041: Can`t set 'Text size' in Email Customizer

[buddypress-trac]

#7041: Can`t set 'Text size' in Email Customizer
-----------------------------------------+--------------------
 Reporter:  WeddyWood                    |       Owner:
     Type:  defect (bug)                 |      Status:  new
 Priority:  high                         |   Milestone:  2.6
Component:  Appearance - Template Parts  |     Version:  2.5.0
 Severity:  normal                       |  Resolution:
 Keywords:  has-patch                    |
-----------------------------------------+--------------------
Changes (by dcavins):

 * keywords:   => has-patch


Comment:

 Ouch. The reason that this fails is that the sanitization callback passed
 into the customizer setting definition is attached to a filter in
 `WP_Customize_Setting::sanitize()`. That filter passes two arguments to
 the sanitization callback: `$value` and the instance of
 `WP_Customize_Setting`. Since `intval()` can accept an optional second
 argument (base), and `$this` is passed into it, it blows up.

 Switching to the WP function `absint()`, which is only expecting one
 argument, works fine though. Patch attached.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7041#comment:3>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac