Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated from errorlog #753

Merged
merged 2 commits into from
Dec 20, 2019
Merged

Conversation

joshcanhelp
Copy link
Contributor

@joshcanhelp joshcanhelp commented Dec 18, 2019

Changes

  • Replace WP_Auth0_ErrorManager::insert_auth0_error() with WP_Auth0_ErrorLog::insert_error()
  • Remove WP_Auth0_ErrorManager
  • Remove WP_Auth0_ErrorLog::init()

Note to reviewers: Most of the changes here are moves from one file to another and re-naming.

Testing

  • This change adds unit test coverage
  • This change has been tested on the latest version of the platform/language or why not

Checklist

  • All existing and new tests complete without errors
  • All code quality tools/guidelines in the Contribution guide have been run/followed
  • All active GitHub CI checks have passed

@joshcanhelp joshcanhelp added this to the 4.0.0 milestone Dec 18, 2019
@joshcanhelp joshcanhelp force-pushed the remove-deprecated-from-errorlog branch from d92092d to b4c867b Compare December 18, 2019 20:11
@joshcanhelp joshcanhelp force-pushed the remove-deprecated-from-errorlog branch from b4c867b to 2735a7d Compare December 18, 2019 20:16
if ( empty( $_POST['nonce'] ) || ! wp_verify_nonce( $_POST['nonce'], 'clear_error_log' ) ) {
wp_die( __( 'Not allowed.', 'wp-auth0' ) );
}
public static function insert_error( $section, $error ) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved as-is from WP_Auth0_ErrorManager

*
* @hook admin_action_wpauth0_clear_error_log
*/
function wp_auth0_errorlog_clear_error_log() {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved here from WP_Auth0_ErrorLog

@joshcanhelp joshcanhelp marked this pull request as ready for review December 18, 2019 20:23
@joshcanhelp joshcanhelp requested a review from a team December 18, 2019 20:23
@joshcanhelp joshcanhelp merged commit 25efd1e into master Dec 20, 2019
@joshcanhelp joshcanhelp deleted the remove-deprecated-from-errorlog branch January 2, 2020 16:49
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants