Discussion on Masterstudy - Education WordPress Theme (Page 2)

about 1 month ago

I’m running Masterstudy LMS + WPML and hit a critical error that broke user registration and various AJAX calls (including front-end account/authorization flows). I don’t have ongoing support, but I thought you would like to know, given how severely it breaks things:

The symptom was:

All of these were failing: Front-end registration, editing anything with Elementor, making any changes to settings to Masterstudy in the background, and some AJAX requests failing.

Debug log showing: PHP Fatal error: Uncaught Error: Call to a member function get_object_id() on null in /wp-content/plugins/sitepress-multilingual-cms/inc/template-functions.php:383 Stack trace: #0 /wp-includes/class-wp-hook.php(341): wpml_object_id_filter(‘40306’, ‘post’) #1 /wp-includes/plugin.php(205): WP_Hook->apply_filters(‘40306’, Array) #2 /wp-content/plugins/masterstudy-lms-learning-management-system/_core/lms/helpers.php(1198): apply_filters(‘wpml_object_id’, ‘40306’, ‘post’) #3 /wp-content/plugins/masterstudy-lms-learning-management-system/_core/lms/helpers.php(1216): ms_plugin_user_account_url(‘edit-course’) #4 /wp-admin/admin-ajax.php(22): require_once(’/wp-load.php’)

So apply_filters( ‘wpml_object_id’, ... ) is being called from Masterstudy while WPML’s $sitepress object is still null, causing the fatal.

Files and functions involved File: /wp-content/plugins/masterstudy-lms-learning-management-system/_core/lms/helpers.php

Functions I had to patch:

ms_plugin_user_account_url() (primary) ms_plugin_authorization() (secondary)

Change #1 – ms_plugin_user_account_url() Original:

function ms_plugin_user_account_url( $sub_page = ’’ ) { $settings = get_option( ‘stm_lms_settings’, array() ); $account_page_id = apply_filters( ‘wpml_object_id’, $settings[‘user_url’] ?? null, ‘post’ ); }

if ( empty( $account_page_id ) || ! did_action( 'init' ) ) {
    return home_url( '/' );
}

$user_account_url = get_the_permalink( $account_page_id );

if ( ! empty( $sub_page ) ) {
    $user_account_url .= "$sub_page/";
}

return $user_account_url;

Patched it to this:

function ms_plugin_user_account_url( $sub_page = ’’ ) { $settings = get_option( ‘stm_lms_settings’, array() ); $account_page_id = isset( $settings[‘user_url’] ) ? $settings[‘user_url’] : null; }

// Check if WPML is initialized before calling the filter
if ( ! empty( $account_page_id ) && function_exists( 'wpml_object_id' ) ) {
    $wpml_account_page_id = apply_filters( 'wpml_object_id', $account_page_id, 'post', true );
    if ( ! empty( $wpml_account_page_id ) ) {
        $account_page_id = $wpml_account_page_id;
    }
}

if ( empty( $account_page_id ) || ! did_action( 'init' ) ) {
    return home_url( '/' );
}

$user_account_url = get_the_permalink( $account_page_id );

if ( ! empty( $sub_page ) ) {
    $user_account_url .= "$sub_page/";
}

return $user_account_url;

Key points:

Guard with function_exists( ‘wpml_object_id’ ) before calling the filter

Use the 4th parameter true so WPML returns the original ID if no translation exists

Avoid calling WPML at all if user_url is empty

Change #2 – ms_plugin_authorization() Original:

function ms_plugin_authorization() { if ( ! is_user_logged_in() ) { $settings = get_option( ‘stm_lms_settings’, array() ); $settings[‘instructor_registration_page’] = $settings[‘instructor_registration_page’] ?? false; $page_id = get_the_ID(); $current_id = apply_filters( ‘wpml_object_id’, $page_id, ‘post’ ); $wpml_pages = array( apply_filters( ‘wpml_object_id’, intval( $settings[‘user_url’] ), ‘post’ ), apply_filters( ‘wpml_object_id’, intval( $settings[‘instructor_url_profile’] ), ‘post’ ), apply_filters( ‘wpml_object_id’, intval( $settings[‘student_url_profile’] ), ‘post’ ), ); $pages = array( intval( $settings[‘user_url’] ), intval( $settings[‘instructor_url_profile’] ), intval( $settings[‘student_url_profile’] ), ); if ( $settings[‘instructor_registration_page’] ) { $wpml_pages[] = apply_filters( ‘wpml_object_id’, intval( $settings[‘instructor_registration_page’] ), ‘post’ ); $pages[] = intval( $settings[‘instructor_registration_page’] ); } }

if ( in_array( $current_id, $wpml_pages, true ) || in_array( $page_id, $pages, true ) ) {
        return;
    }
}

STM_LMS_Templates::show_lms_template(
    'components/authorization/main',
    array(
        'modal' => true,
        'type'  => 'login',
    )
);

Patched it to:

function ms_plugin_authorization() { if ( ! is_user_logged_in() ) { $settings = get_option( ‘stm_lms_settings’, array() ); $settings[‘instructor_registration_page’] = $settings[‘instructor_registration_page’] ?? false; $page_id = get_the_ID(); }

// Safely handle current page WPML translation
    $current_id = $page_id;
    if ( function_exists( 'wpml_object_id' ) ) {
        $wpml_current_id = apply_filters( 'wpml_object_id', $page_id, 'post', true );
        if ( ! empty( $wpml_current_id ) ) {
            $current_id = $wpml_current_id;
        }
    }
}

// Initialize arrays
$wpml_pages = array();
$pages      = array();

// Helper function to safely add page to arrays
$add_page_safely = function( $page_id_value ) use ( &$wpml_pages, &$pages ) {
    if ( empty( $page_id_value ) ) {
        return;
    }
};

$page_id_int = intval( $page_id_value );
$pages[]     = $page_id_int;

if ( function_exists( 'wpml_object_id' ) ) {
    $wpml_page_id = apply_filters( 'wpml_object_id', $page_id_int, 'post', true );
    if ( ! empty( $wpml_page_id ) ) {
        $wpml_pages[] = $wpml_page_id;
    }
} else {
    $wpml_pages[] = $page_id_int;
}

// Add all configured pages
$add_page_safely( $settings['user_url'] ?? null );
$add_page_safely( $settings['instructor_url_profile'] ?? null );
$add_page_safely( $settings['student_url_profile'] ?? null );

if ( ! empty( $settings['instructor_registration_page'] ) ) {
    $add_page_safely( $settings['instructor_registration_page'] );
}

if ( in_array( $current_id, $wpml_pages, true ) || in_array( $page_id, $pages, true ) ) {
    return;
}

STM_LMS_Templates::show_lms_template(
    'components/authorization/main',
    array(
        'modal' => true,
        'type'  => 'login',
    )
);

Again, the main changes are:

Guard all wpml_object_id calls with function_exists( ‘wpml_object_id’ ) Use the 4th parameter true Avoid calling the filter when setting values are empty Don’t assume WPML is always initialized when these functions run (especially during AJAX)

Result After these two patches, the fatal get_object_id() on null error stopped, AJAX calls stopped 500’ing, and user registration/auth flows started working again.

It would be great if Masterstudy could:

Incorporate these guards into the plugin code Audit other apply_filters( ‘wpml_object_id’, ...) usages for the same pattern

I’m happy to provide more logs or details if needed.

Discussion on Masterstudy - Education WordPress Theme

Discussion on Masterstudy - Education WordPress Theme

StylemixThemes supports this item

8635 comments found.

StylemixThemes

Tell us what you think!