Here is my current permanent fix for version 1.12.6 and maybe above. Error or fix is marked in bold on cluevo-mod.js file. You might need to review the code there, see conclusions at the bottom.
- First I’ve added In theme
functions.php this code:
// Conditional enqueueing of modified script and its dependencies
function enqueue_mod_script_conditionally() {
// Check if either 'cluevo-scorm' or 'cluevo' plugin script is enqueued
if ( wp_script_is( 'cluevo-scorm', 'enqueued' ) || is_plugin_active( 'cluevo-lms/cluevo-lms.php' ) ) {
// Enqueue cluevo-scorm-wrapper if it's not already enqueued
if ( ! wp_script_is( 'cluevo-scorm-wrapper', 'enqueued' ) ) {
wp_enqueue_script( 'cluevo-scorm-wrapper' );
}
// Enqueue cluevo-scorm if it's not already enqueued
if ( ! wp_script_is( 'cluevo-scorm', 'enqueued' ) ) {
wp_enqueue_script( 'cluevo-scorm' );
}
// Enqueue your modified script
wp_enqueue_script( 'cluevo-mod-script', get_stylesheet_directory_uri() . '/assets/javascript/cluevo-mod.js' , array( 'cluevo-scorm' ), '1.0.0', true);
}
}
add_action( 'wp_enqueue_scripts', 'enqueue_mod_script_conditionally', 999 ); // Use a high priority to ensure it runs late
2. Made file in theme /assets/javascript/cluevo-mod.js below:
// Wrap the original function
(function() {
var originalShowResumePrompt = showResumePrompt;
// Redefine the showResumePrompt function with your fix
showResumePrompt = async function(itemId, data, callback) {
// Your fixed implementation here
cluevoRemoveTileOverlays();
var el =
'<div class="cluevo-module-tile-overlay"><h2>' +
cluevoStrings.start_over_dialog_header +
'</h2><div class="cluevo-prompt-btns-container"><div class="cluevo-btn yes">' +
cluevoStrings.start_over_opt_reset +
'</div><div class="cluevo-btn no">' +
cluevoStrings.start_over_opt_resume +
"</div></div></div>";
var dialog = jQuery(el);
var needLightbox =
jQuery(
'.cluevo-content-item-link[data-item-id="' +
itemId +
'"] .cluevo-post-thumb:first'
).length === 0;
dialog.on("click", ".cluevo-btn.yes", async function() {
cluevoShowLightboxSpinner();
var url =
cluevoWpApiSettings.root + "cluevo/v1/modules/" + itemId + "/new-attempt";
await jQuery.ajax({
url: url,
method: "GET",
contentType: "application/json",
dataType: "json",
beforeSend: function(xhr) {
xhr.setRequestHeader("X-WP-Nonce", cluevoWpApiSettings.nonce);
},
success: function(response) {
jQuery(dialog).remove();
if (needLightbox) {
cluevoCloseLightbox();
}
callback(true, response);
},
});
});
dialog.on("click", ".cluevo-btn.no", function(e) {
e.stopPropagation();
e.preventDefault();
jQuery(dialog).remove();
/******** the actual fix - next 3 lines *******/
if (needLightbox) {
cluevoCloseLightbox();
}
callback(false);
});
dialog.on("click", ".cluevo-btn:not(.no)", function(e) {
e.stopPropagation();
e.preventDefault();
jQuery(dialog).remove();
/******** nonsense maybe *******/
if (!needLightbox) {
cluevoCloseLightbox();
}
});
if (!needLightbox) {
jQuery(
'.cluevo-content-item-link[data-item-id="' +
itemId +
'"] .cluevo-post-thumb:first'
).append(dialog);
} else {
cluevoOpenLightbox(null, "", dialog);
cluevoShowLightbox();
}
jQuery(dialog).fadeIn();
}
})();
Conclusion:
1. “.cluevo-btn.no” does not have a lightbox close logic similar to “.cluevo-btn.yes”
2. “.cluevo-btn:not(.no)” => “.cluevo-btn.yes” OR “.cluevo-btn.cluevo-close-button” – you have specific logic for “.yes”, and no logic or function for close button, but if lighbox was not needed you close it anyway.
3. script “cluevo-scorm” requires “cluevo-scorm-wrapper” – it’s a small thing but I couldn’t add my mod without fixing the enqueue order. ChatGPT helped to write the code and enqueue my mod.
You may close the issue after reading it. But I hope you will fix it, if not it may serve the next person with this issue.
