WordPress.org

Ready to get started?Download WordPress

Forums

Two Factor Auth
[resolved] $_GET('warning_button_clicked') (4 posts)

  1. Thomas van der Beek
    Member
    Posted 6 months ago #

    First of all, let me say I love your work! Just have a little request for change :-)

    When having WP_DEBUG set to TRUE, you receive an Undefined index Notice on the /wp-admin/admin.php?page=two-factor-auth-user page(s).

    Fairly easy to fix in /two-factor-auth/user_settings.php (line 14):
    if( $_GET['warning_button_clicked'] == 1 ) { ... }

    my way:
    if( isset( $_GET['warning_button_clicked'] ) && 1 == $_GET['warning_button_clicked'] ) { ... }

    Could you take this one in your next release? Thanks!

    http://wordpress.org/plugins/two-factor-auth/

  2. oskarhane
    Member
    Plugin Author

    Posted 6 months ago #

    Ah, yes, of course!
    Thanks.

  3. Thomas van der Beek
    Member
    Posted 6 months ago #

    if( isset( $_GET['warning_button_clicked'] ) ) {...}

    Would do the same. The value doesn't matter in this case.

    Thanks!

  4. oskarhane
    Member
    Plugin Author

    Posted 5 months ago #

    This is now out in v.4.3!
    Thanks for the report.

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.