[wp-trac] [WordPress Trac] #10309: Publish Date (Schedule) does not handle inputs properly.

WordPress Trac wp-trac at lists.automattic.com
Sat Sep 5 18:50:21 UTC 2009


#10309: Publish Date (Schedule) does not handle inputs properly.
--------------------------+-------------------------------------------------
 Reporter:  hakre         |       Owner:  camper2020 
     Type:  defect (bug)  |      Status:  accepted   
 Priority:  normal        |   Milestone:  2.8.5      
Component:  Date/Time     |     Version:  2.8        
 Severity:  normal        |    Keywords:  needs-patch
--------------------------+-------------------------------------------------

Comment(by hakre):

 hi camper,

 thanks for the patch. please consider to not that much duplicate code (for
 example, multiple of the cases in the switch statement containt the same
 code, you can write those below each other) and the the coding guidelines
 (I normally do write the brackets like you do, but in this project there
 is some other guideline):

 http://codex.wordpress.org/WordPress_Coding_Standards

 Then I do not understand why you remove the 'trash' code in your patch.
 maybe this is by error?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10309#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list