[wp-trac] Re: [WordPress Trac] #9730: The new date_i18n hook creates potential bugs in the admin interface

WordPress Trac wp-trac at lists.automattic.com
Tue May 5 17:17:05 GMT 2009


#9730: The new date_i18n hook creates potential bugs in the admin interface
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:  Denis-de-Bernardy      
     Type:  defect (bug)       |      Status:  new                    
 Priority:  normal             |   Milestone:  2.8                    
Component:  i18n               |     Version:  2.8                    
 Severity:  major              |    Keywords:  has-patch needs-testing
-------------------------------+--------------------------------------------

Comment(by hakre):

 Replying to [comment:6 Denis-de-Bernardy]:
 > there is one I'm not 100% on:
 >
 >  - wp_get_archives(), $text = mysql2date($archive_day_date_format,
 $date); // no?
 >
 > and the calendar function might need to be looked into.


 ''wp_get_archives()'' is a template function used in themes. should be
 okay to be hooked.

 calender function? if you mean ''get_calendar()'' in genreal-template the
 I would say, is a place to take a look into:

 {{{
                 $thismonth = $wpdb->get_var("SELECT
 DATE_FORMAT((DATE_ADD('${thisyear}0101', INTERVAL $d DAY) ), '%m')");
 }}}

 and it makes heavy use of ''$wp_locale''.

 ----

 patch looks good to me. i tested it in the backend and the frontend. where
 applicable, the translated date is displayed. editing for posts works as
 known.

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


More information about the wp-trac mailing list