• Hi there,

    I’m currently having issues with registering and logging in users. When I click the register or login buttons, the page doesn’t refresh automatically but still registers/logs in my user in the backend. I notice the user being logged in when manually refreshing the account page after clicking the register/login button.

    Please could you help as soon as possible as I’m developing my site for my client. I’m using the latest version of WordPress with default jQuery version (in built with WordPress).

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author weblizar

    (@weblizar)

    Hello @l0cam0cha,

    Thanks for using our plugin

    When you click login register/login do you get an error message?
    and also check your console do you have any error regarding job portal plugin.

    Thanks

    Hi i’m experiencing a similar issue with the plugin when registering users. Internal Server Error is popping up when i try to register and it does not refresh. But for some reason the user will already be registered and when i refresh the browser i will be logged in.

    Hi Guys, anyone who can help me with the above?

    Thread Starter l0cam0cha

    (@l0cam0cha)

    Hey guys,

    Oddly enough I managed to workaround this issue by commenting out a wp_login action from a bluehost-wordpress-plugin function file called “track-last-login”. The file looks like this:

    <?php
    /**
     * File handles tracking the last time a user logged in.
     *
     * @package Mojo Marketplace
     */
    
    #add_action( 'wp_login', 'bluehost_set_last_login', 10, 2 ); - this is the line I commented out
    
    /**
     * Sets the time in ISO8601 format for the last time a user logged in.
     *
     * @param string   $username Current user login name
     * @param \WP_User $user     Current user object
     */
    function bluehost_set_last_login( $username, WP_User $user ) {
    
    	// Store last login for current user
    	update_user_meta( $user->ID, 'eig_last_login', date( 'c' ) );
    
    	// Store last login for entire site for any user that can impact content.
    	if ( $user->has_cap( 'edit_posts' ) ) {
    		update_option( 'eig_last_site_login', date( 'c' ) );
    	}
    
    }
    

    Once I did that, the portal login worked just fine. I know this isn’t a practical solution as this function is necessary for bluehost management. Wondering if there is a plugin fix around login that will be made soon? @weblizar

    I suspect this may have to do with the plugin using native login events instead of custom ones.

    • This reply was modified 4 years, 1 month ago by l0cam0cha.
    Thread Starter l0cam0cha

    (@l0cam0cha)

    LOGIN METHOD FIX

    Alright, I managed to do some digging into the plugin and found this – https://www.ads-software.com/support/topic/do_actionwp_login-argument-missed-and-heavy-quersi/

    So, in the WL_JP_User.php file of jobportal plugin ([yoursite]\app\public\wp-content\plugins\jobs-portal\public\inc\WL_JP_User.php), I managed to update the wp_login action within the login function. It seems you were missing another $user argument when calling ‘do_action( ‘wp_login’, $user->user_login );’ method:

    	/* Login */
    	public static function login() {
    		if ( ! wp_verify_nonce( $_POST['login'], 'login' ) ) {
    			die();
    		}
    
    		$username = isset( $_POST['username'] ) ? $_POST['username'] : NULL;
    		$password = isset( $_POST['password'] ) ? $_POST['password'] : NULL;
    
    		$user = wp_authenticate( $username, $password );
    		if ( is_wp_error( $user ) ) {
    			wp_send_json_error( $user->get_error_message() );
    		}
    
            wp_set_current_user( $user->ID, $user->user_login );
            wp_set_auth_cookie( $user->ID );
            // do_action( 'wp_login', $user->user_login ); - COMMENTED THIS OUT
    		do_action( 'wp_login', $user->user_login, $user ); - ADDED THIS IN WITH MISSING 3RD ARGUMENT
    
    		wp_send_json_success( array( 'message' => esc_html__( "You are logged in successfully.", WL_JP_DOMAIN ), 'reload' => true ) );
    	}
    

    Once I updated that action with the added argument, it seemed to fix the issue with login only. Perhaps this is something you can implement in the next version of the plugin @weblizar?

    I’m still looking into the registration call as that is still giving a “500 Internal server error” when submitting the form for a new user.

    • This reply was modified 4 years, 1 month ago by l0cam0cha.
    • This reply was modified 4 years, 1 month ago by l0cam0cha.
    • This reply was modified 4 years, 1 month ago by l0cam0cha.
    Thread Starter l0cam0cha

    (@l0cam0cha)

    REGISTRATION METHOD FIX

    Aaand now I’ve fixed the autologin in the registration method (which sits in the same file as well):

    /* Signup */
    	public static function signup() {
    		if ( ! wp_verify_nonce( $_POST['signup'], 'signup' ) ) {
    			die();
    		}
    
    		$username         = isset( $_POST['username'] ) ? sanitize_text_field( $_POST['username'] ) : NULL;
    		$email            = isset( $_POST['email'] ) ? sanitize_email( $_POST['email'] ) : NULL;
    		$password         = isset( $_POST['password'] ) ? $_POST['password'] : NULL;
    		$confirm_password = isset( $_POST['confirm_password'] ) ? $_POST['confirm_password'] : NULL;
    
    		/* Validations */
    		$errors = [];
    		if ( empty( $username ) ) {
    			$errors['username'] = esc_html__( 'Please provide username.', WL_JP_DOMAIN );
    		}
    
    		if ( empty( $email ) ) {
    			$errors['email'] = esc_html__( 'Please provide email address.', WL_JP_DOMAIN );
    		}
    
    		if ( ! filter_var( $email, FILTER_VALIDATE_EMAIL ) ) {
    			$errors['email'] = esc_html__( 'Please provide a valid email address.', WL_JP_DOMAIN );
    		}
    
    		if ( empty( $password ) ) {
    			$errors['password'] = esc_html__( 'Please provide password.', WL_JP_DOMAIN );
    		}
    
    		if ( empty( $confirm_password ) ) {
    			$errors['password_confirm'] = esc_html__( 'Please confirm password.', WL_JP_DOMAIN );
    		}
    
    		if ( $password !== $confirm_password ) {
    			$errors['password'] = esc_html__( 'Passwords do not match.', WL_JP_DOMAIN );
    		}
    		/* End validations */
    
    		if ( count( $errors ) < 1 ) {
    			$data = array(
    			    'user_login' => $username,
    			    'user_email' => $email,
    			    'user_pass'  => $password
    			);
    
    			$user_id = wp_insert_user( $data );
    			if ( is_wp_error( $user_id ) ) {
    				wp_send_json_error( $user_id->get_error_message() );
    			}
    
    			update_user_meta( $user_id, 'wljp_signup_as', 'candidate' );
    			
    			$user = get_user_by( 'id', $user_id );
                if( $user ) {
                    wp_set_current_user( $user_id, $user->user_login );
                    wp_set_auth_cookie( $user_id );
                    do_action( 'wp_login', $user->user_login, $user );
                }
    			else{
    				wp_send_json_error( $user_id->get_error_message() );
    			}
    
    	        // wp_set_current_user( $user_id, $username );
    	        // wp_set_auth_cookie( $user_id );
    	        // do_action( 'wp_login', $username );
    
    			wp_send_json_success( array( 'message' => esc_html__( 'Thank you for signing up.', WL_JP_DOMAIN ), 'reload' => true ) );
    		}
    		wp_send_json_error( $errors );
    	}
    

    This fix was based on this post on StackOverflow –

    • This reply was modified 4 years, 1 month ago by l0cam0cha.
    • This reply was modified 4 years, 1 month ago by l0cam0cha.
    • This reply was modified 4 years, 1 month ago by l0cam0cha.
Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Register/Login Errors’ is closed to new replies.