Circumvent Undefined property warning resolves kaltura#405. #1

djarran · 2024-02-19T00:11:33Z

Description: This resolves kaltura#405 (see for more information) allowing plugin install to occur without warnings or errors

Problem

== Setting new default values ==

Warning: Undefined property: stdClass::$lti_version in /var/www/site/local/kaltura/locallib.php on line 795

Code that is causing the error

795if (!empty($configsettings->kaf_uri) && $configsettings->lti_version == LTI_VERSION_1P3) {
796    if (isset($configsettings->lti_version) && (!empty($configsettings->kaf_uri) && $configsettings->lti_version == LTI_VERSION_1P3)) {
797        $configsettings->public_keyset_url = $configsettings->kaf_uri . '/hosted/index/lti-advantage-key-set';
798        $configsettings->public_keyset_url = $configsettings->kaf_uri . '/hosted/index/lti-advantage-key-set';
799        $configsettings->launch_url = $configsettings->kaf_uri . '/hosted/index/oidc-init';
800        $configsettings->launch_url = $configsettings->kaf_uri . '/hosted/index/oidc-init';
801        $configsettings->redirection_uris = $configsettings->kaf_uri . '/hosted/index/oauth2-launch';
802        $configsettings->redirection_uris = $configsettings->kaf_uri . '/hosted/index/oauth2-launch';
803    }

Solution

Check that $configsettings->lti_version is set first

if (isset($configsettings->lti_version) && (!empty($configsettings->kaf_uri) && $configsettings->lti_version == LTI_VERSION_1P3)) {
    ...
}

golenkovm · 2024-02-19T00:23:39Z

Thanks @djarran

Circumvent Undefined property warning resolves kaltura#405.

82a43a6

golenkovm merged commit b2cbf71 into CATALYST_401_STABLE Feb 19, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Circumvent Undefined property warning resolves kaltura#405. #1

Circumvent Undefined property warning resolves kaltura#405. #1

djarran commented Feb 19, 2024 •

edited

Loading

golenkovm commented Feb 19, 2024

Circumvent Undefined property warning resolves kaltura#405. #1

Circumvent Undefined property warning resolves kaltura#405. #1

Conversation

djarran commented Feb 19, 2024 • edited Loading

golenkovm commented Feb 19, 2024

djarran commented Feb 19, 2024 •

edited

Loading