WordPress.org

Ready to get started?Download WordPress

Forums

WordPress Social Login
Error with Vkontakte provider (2 posts)

  1. masimkin
    Member
    Posted 1 year ago #

    Hi, great plugin! thanks!
    But I've found a few issues (In my opinion :))
    I use 1.1.9
    1. includes/plugin.auth.php
    line 107: should be this check on empty($user_login), validate_username is true for empty.
    2. hybridauth/Hybrid/Providers/Vkontakte.php
    line 92: should be check if 'nickname' is not null, because in Vkontakte API this is nullable fileld. If nickname is null, displayName will be 'first_name' + 'last_name'. But, pay attention, first_name and last_name can be in cyrillic.
    3. includes/plugin.auth.php
    If you update users metadata will be better to add something like that, for getting a friendly error messages.

    // update user metadata
    		if( $user_id && is_integer( $user_id ) ){
    			update_user_meta( $user_id, $provider, $hybridauth_user_profile->identifier );
    		}
    else if (is_wp_error($user_id)) {
    echo $user_id->get_error_message();
    }
    		else{
    			die( "An error occurred while creating a new user!" );
    		}

    Thank you, and sorry for my english)

    http://wordpress.org/extend/plugins/wordpress-social-login/

  2. Miled
    Member
    Plugin Author

    Posted 1 year ago #

    Thanks @masimkin for your feedback.
    I was just working on the plugin now, so I took into account some of your suggestions

    for cyrillic names i think using this plugin should fix the issue http://wordpress.org/extend/plugins/wordpress-special-characters-in-usernames

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic