[wp-trac] [WordPress Trac] #27979: Pass Customizer Setting instance to `customize_update_` and `customize_preview_`

WordPress Trac noreply at wordpress.org
Mon Jun 2 15:18:32 UTC 2014


#27979: Pass Customizer Setting instance to `customize_update_` and
`customize_preview_`
----------------------------------------+------------------
 Reporter:  danielbachhuber             |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  4.0
Component:  Appearance                  |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  needs-patch good-first-bug  |     Focuses:
----------------------------------------+------------------

Comment (by celloexpressions):

 Hi tormorten,

 Thanks for looking into this. A few notes:
 - Rather than upload the whole file, can you create a patch from the
 WordPress root? We use patches so that it's easier to review and commit
 changes.
 - Be sure to update the inline documentation to include the new parameter
 (something like `@param WP_Customize_Setting $this  WP_Customize_Setting
 instance.`).
 - When you upload patches in the future, be sure to leave a comment and
 add the "has-patch" keyword to the ticket. Attaching files doesn't send
 notifications, only comments do, and adding has-patch will help with the
 review process.

 If you have any questions, feel free to ask for clarification!

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


More information about the wp-trac mailing list