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

Timestamp:

08/31/2017 12:47:09 PM (8 years ago)

Author:

SergeyBiryukov

Message:

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

Next occurrence should be calculated in the current month, not in the Start Date month.

See #2218.

File:

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

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

@@ -156 +156 @@
             } else if ( 'occurrence' === $post->recurring ) {
                 try {
-                    // advance the start date in the current month until it's past now
+                    // advance the occurrence day in the current month until it's past now
                     $start = new DateTime( $post->start_date . ' ' . $post->time . ' GMT' );
                     $next  = $start;
@@ -168 +168 @@
                     foreach ( $post->occurrence as $index ) {
                         foreach ( $months as $month ) {
-                            $next->modify( $numerals[ $index - 1 ] . ' ' . $day_name . ' of ' . $month );
+                            $next = new DateTime( $numerals[ $index - 1 ] . ' ' . $day_name . ' of ' . $month );
                             if ( $next > $now ) {
                                 break 2;