[wp-trac] [WordPress Trac] #30988: Setting's default value doesn't apply on preview window

WordPress Trac noreply at wordpress.org
Fri Jan 30 06:25:43 UTC 2015


#30988: Setting's default value doesn't apply on preview window
--------------------------+-----------------------------
 Reporter:  Aniruddh      |       Owner:  ocean90
     Type:  defect (bug)  |      Status:  reviewing
 Priority:  normal        |   Milestone:  4.1.1
Component:  Customize     |     Version:  4.1
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:  administration
--------------------------+-----------------------------
Changes (by westonruter):

 * keywords:  needs-testing reporter-feedback needs-patch => has-patch
 * owner:  westonruter => ocean90


Comment:

 @ocean90: in [attachment:30988.3.diff]:

 * Add specific test case for issue in this ticket: https://github.com/xwp
 /wordpress-develop/commit/2da22194ecceb45eaf2a4b404beefb363c4fe40a
 * Split up test case method in half, trying to keep things DRY:
 https://github.com/xwp/wordpress-
 develop/commit/2c92f831d04626ef661b0b1a77a6885d286db0d3
 * Removed closures: https://github.com/xwp/wordpress-
 develop/compare/c4a0da93112e942ab35d6378c467e3d2fdde8ac5...edf7836af3bed880502b4ed5050fbb294be05fb4
 * Removed separate `parse_post_data` method, but keeping `$default`
 argument on the `post_value()` method: https://github.com/xwp/wordpress-
 develop/commit/39d5d8f34dbee10c8883d9e1be3f84adab2987bc

 Full changes in PR: https://github.com/xwp/wordpress-develop/pull/62/files

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


More information about the wp-trac mailing list