Fix initial call of Joomla.loadOptions()

[Fedik]

Summary of Changes

This patch fix an initial call of Joomla.loadOptions() when page do not contain the Script options, which could lead to JavaScript error.

Testing Instructions

Use latest staging,
Add in to index.php of Isis template:

JHtml::_('behavior.core');
$this->addScriptDeclaration('
  try{Joomla.getOptions("blabla");alert("Works!");}catch(e){alert("Error!");};
');

And open "Control panel" page.

Expected result

You see alert "Works!"

Actual result

You see alert "Error!"

Documentation Changes Required

none

[andrepereiradasilva]

I have tested this item ✅ successfully on 61b334a

works as described

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16488.}

[andrepereiradasilva]

@dgt41 (or other tester) please test since there another PRs waiting for this one to get merged
#16452 and #15529

[andrepereiradasilva]

@Fedik i'm still getting the error. maybe there's another place you need to do that

[andrepereiradasilva]

seems the error is here https://github.com/joomla/joomla-cms/blob/staging/media/system/js/core-uncompressed.js#L159

core.js?9398acd…:1 Uncaught TypeError: Cannot read property 'system.keepalive' of null at Object.Joomla.getOptions (core.js?9398acd…:1) at HTMLDocument.<anonymous> (keepalive.js?9398acd…:1) Joomla.getOptions @ core.js?9398acd…:1 (anonymous) @ keepalive.js?9398acd…:1

[C-Lodder]

What happens if you do console.log(key); on the line you linked to in the comment above?

[andrepereiradasilva]

console.log(key);
console.log(Joomla.optionsStorage);

results in

system.keepalive
null

[Fedik]

I found the problem,
should be good now

[andrepereiradasilva]

I have tested this item ✅ successfully on 9660d35

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16488.}

[Quy]

I have tested this item ✅ successfully on 9660d35

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16488.}

[ghost]

RTC after two successful tests.