Event-Federation/wordpress-activitypub
Event-Federation/wordpress-activitypub
7
1
Fork 0
Code Issues 3 Pull requests Projects Releases Wiki Activity Actions

Merge pull request #103 from akirk/patch-1

Browse source 
Code Style: Filter option in favor of updating the database
This commit is contained in:
Matthias Pfefferle 2020-11-02 11:22:10 +01:00 committed by GitHub
commit f418e823c5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
includes/rest/class-inbox.php
View file

@ -298,12 +298,9 @@ class Inbox {
\remove_action( 'check_comment_flood', 'check_comment_flood_db', 10 );
// do not require email for AP entries
$option = get_option( 'require_name_email' );
update_option( 'require_name_email', 0 );
add_filter( 'pre_option_require_name_email', '__return_false' );
$state = \wp_new_comment( $commentdata, true );
update_option( 'require_name_email', $option );
remove_filter( 'pre_option_require_name_email', '__return_false' );
// re-add flood control
\add_action( 'check_comment_flood', 'check_comment_flood_db', 10, 4 );
@ -337,12 +334,9 @@ class Inbox {
\remove_action( 'check_comment_flood', 'check_comment_flood_db', 10 );
// do not require email for AP entries
$option = get_option( 'require_name_email' );
update_option( 'require_name_email', 0 );
add_filter( 'pre_option_require_name_email', '__return_false' );
$state = \wp_new_comment( $commentdata, true );
update_option( 'require_name_email', $option );
remove_filter( 'pre_option_require_name_email', '__return_false' );
// re-add flood control
\add_action( 'check_comment_flood', 'check_comment_flood_db', 10, 4 );