forked from enviPath/enviPy
fix: more defensive ketcher reference
This commit is contained in:
@ -256,6 +256,31 @@
|
||||
<script language="javascript">
|
||||
var currentPackage = "{{ meta.current_package.url }}";
|
||||
|
||||
// Helper function to safely get Ketcher instance from iframe
|
||||
function getKetcherInstance(iframeId) {
|
||||
const ketcherFrame = document.getElementById(iframeId);
|
||||
if (!ketcherFrame) {
|
||||
console.error("Ketcher iframe not found:", iframeId);
|
||||
return null;
|
||||
}
|
||||
|
||||
try {
|
||||
if (
|
||||
"contentWindow" in ketcherFrame &&
|
||||
ketcherFrame.contentWindow.ketcher
|
||||
) {
|
||||
return ketcherFrame.contentWindow.ketcher;
|
||||
}
|
||||
} catch (err) {
|
||||
console.error(
|
||||
"Cannot access Ketcher iframe - possible CORS issue:",
|
||||
err,
|
||||
);
|
||||
}
|
||||
|
||||
return null;
|
||||
}
|
||||
|
||||
// Discourse API integration is now handled by discourse-api.js
|
||||
|
||||
// Function to render Discourse topics into cards
|
||||
@ -359,10 +384,16 @@
|
||||
}, 300);
|
||||
|
||||
// Transfer SMILES from Ketcher to text input if available
|
||||
if (window.indexKetcher && window.indexKetcher.getSmiles) {
|
||||
const smiles = window.indexKetcher.getSmiles();
|
||||
if (smiles && smiles.trim() !== "") {
|
||||
$("#index-form-text-input").val(smiles);
|
||||
const ketcher = getKetcherInstance("index-ketcher");
|
||||
if (ketcher && ketcher.getSmiles) {
|
||||
try {
|
||||
const smiles = ketcher.getSmiles();
|
||||
if (smiles && smiles.trim() !== "") {
|
||||
$("#index-form-text-input").val(smiles);
|
||||
}
|
||||
} catch (err) {
|
||||
console.error("Failed to sync Ketcher to text input:", err);
|
||||
// Non-critical error, just log it
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -431,13 +462,32 @@
|
||||
var textSmiles = "";
|
||||
|
||||
// Check if we're in Ketcher mode and extract SMILES
|
||||
if ($('input[type="checkbox"]').is(":checked") && window.indexKetcher) {
|
||||
textSmiles = window.indexKetcher.getSmiles().trim();
|
||||
if ($('input[type="checkbox"]').is(":checked")) {
|
||||
// Use the robust getter function
|
||||
const ketcher = getKetcherInstance("index-ketcher");
|
||||
if (ketcher && ketcher.getSmiles) {
|
||||
try {
|
||||
textSmiles = ketcher.getSmiles().trim();
|
||||
} catch (err) {
|
||||
console.error("Failed to get SMILES from Ketcher:", err);
|
||||
alert(
|
||||
"Unable to extract structure from the drawing editor. Please try again or switch to SMILES input mode.",
|
||||
);
|
||||
return;
|
||||
}
|
||||
} else {
|
||||
console.warn("Ketcher not available, possibly still loading");
|
||||
alert(
|
||||
"The drawing editor is still loading. Please wait a moment and try again.",
|
||||
);
|
||||
return;
|
||||
}
|
||||
} else {
|
||||
textSmiles = $("#index-form-text-input").val().trim();
|
||||
}
|
||||
|
||||
if (textSmiles === "") {
|
||||
alert("Please enter a SMILES string or draw a structure.");
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user