Context Navigation

← Previous Change
Next Change →

wcor-reminder.php

Timestamp:

02/18/2019 11:49:51 PM (7 years ago)

Author:

iandunn

Message:

WordCamp Organizer Reminders: Only sync postmeta for WordCamp posts.

Previously this assumed that the current post being saved was a wordcamp post, and until recently that was always true. Something changed that allowed this to run in other contexts, like when saving an Organizer Reminder post during the process of manually sending a reminder. When that happened, the meta values for the WordCamp post would be wiped out, since there wouldn't be any matching values in $_POST from the Organizer Reminder post.

This is probably related to r8220, but it's not immediately obvious, and isn't worth sinking more time into.

File:

: 1 edited

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-organizer-reminders/wcor-reminder.php (modified) (1 diff)

Legend:

: Unmodified
: Added
: Removed

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-organizer-reminders/wcor-reminder.php

r8236	r8270
436	436	}
437	437
438		~~wp_die( 'Manually sending reminders is currently disabled while I track down a nasty bug. -Ian' );~~
439
440	438	$wordcamp = get_post( $form_values['wcor_manually_send_wordcamp'] );
441	439	$WCOR_Mailer->send_manual_email( $email, $wordcamp );

Note: See TracChangeset for help on using the changeset viewer.

Making WordPress.org

Context Navigation

Changeset 8270 for sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-organizer-reminders/wcor-reminder.php

Legend:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-organizer-reminders/wcor-reminder.php

Download in other formats: