Changeset 6952

Timestamp:

03/28/2018 05:01:24 AM (8 years ago)

Author:

SergeyBiryukov

Message:

Make/Meetings: Correct timezone for "Occurrence in a month" option.

Use sprintf() in DateTime strings for better readability.

See #2218.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-meeting-posttype/wporg-meeting-posttype.php (modified) (5 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-meeting-posttype/wporg-meeting-posttype.php

@@ -122 +122 @@
                 try {
                     // from the start date, advance the week until it's past now
-                    $start = new DateTime( $post->start_date . ' ' . $post->time . ' GMT' );
+                    $start = new DateTime( sprintf( '%s %s GMT', $post->start_date, $post->time ) );
                     $next  = $start;
                     $now   = new DateTime();
@@ -141 +141 @@
                 try {
                     // advance the start date 2 weeks at a time until it's past now
-                    $start = new DateTime( $post->start_date . ' ' . $post->time . ' GMT' );
+                    $start = new DateTime( sprintf( '%s %s GMT', $post->start_date, $post->time ) );
                     $next  = $start;
                     $now   = new DateTime();
@@ -157 +157 @@
                 try {
                     // advance the occurrence day in the current month until it's past now
-                    $start = new DateTime( $post->start_date . ' ' . $post->time . ' GMT' );
+                    $start = new DateTime( sprintf( '%s %s GMT', $post->start_date, $post->time ) );
                     $next  = $start;
                     $now   = new DateTime();
 
-                    $day_index = date( 'w', strtotime( $post->start_date . ' ' . $post->time . ' GMT' ) );
+                    $day_index = date( 'w', strtotime( sprintf( '%s %s GMT', $post->start_date, $post->time ) ) );
                     $day_name  = $GLOBALS['wp_locale']->get_weekday( $day_index );
                     $numerals  = array( 'first', 'second', 'third', 'fourth' );
@@ -168 +168 @@
                     foreach ( $post->occurrence as $index ) {
                         foreach ( $months as $month ) {
-                            $next = new DateTime( $numerals[ $index - 1 ] . ' ' . $day_name . ' of ' . $month );
+                            $next = new DateTime( sprintf( '%s %s of %s %s GMT', $numerals[ $index - 1 ], $day_name, $month, $post->time ) );
                             if ( $next > $now ) {
                                 break 2;
@@ -183 +183 @@
                 try {
                     // advance the start date 1 month at a time until it's past now
-                    $start = new DateTime( $post->start_date . ' ' . $post->time . ' GMT' );
+                    $start = new DateTime( sprintf( '%s %s GMT', $post->start_date, $post->time ) );
                     $next  = $start;
                     $now   = new DateTime();