Can`t set 'Text size' in Email Customizer

[WeddyWood]

Text size changes only in online view, but value is not set.

Reason: in bp_email_get_customizer_settings function is wrong sanitize_callback

'intval'

If change it to empty string - everything works good.

''

[DJPaul]

Confirmed. I am baffled as to what's going on. To save time for the next person, these values are provided from the Customiser as a string, and the idea with intval is to coerce it to an integer. The updated value is not getting saved to the database (wp_options, key "bp_email_options").

[dcavins]

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.

[dcavins]

Use absint() to validate integer inputs in the customizer.

[r-a-y]

Nice investigative work, @dcavins!

Should this also be part of 2.5.4?

[dcavins]

In 10814:

Use absint() to validate numeric input in the WP Customizer.

intval() fails when used with the WP
Customizer API as a sanitization callback.
The callback function is attached to
a filter in WP_Customize_Setting::sanitize()
that passes two arguments to the
sanitization callback: the $value to sanitize
and the instance of WP_Customize_Setting.
Since intval() can accept an optional
second argument ($base), but isn’t
expecting an object, it fails.
absint() works because it is only
expecting the $value argument.

Fixes #7041.

Props WeddyWood.