Tested. All good

The addScriptDeclaration is being injected before the addScript statement so would this not lead to undefined JS variables?

Is that a problem? addScript always injects into head

@C-Lodder It would be nice to follow the actual chain of events, first the file and then the initialization

hmm but this would cause a redundancy in the code

@C-Lodder The inline scripts are always added after the files so it's not an issue. That's why we load jQuery no captcha as a file rather than a script

Looks good

@test It works Ok, no mootools.

@dgt41 @Qlimax90 what is the status here? Can we leave it now as it is or need we to change something? As we get multible successful tests here Thanks.

I think we can leave it as it is :)

@Qlimax90 just do a merge on staging to resolve the conflicts

@dgt41 It seems that i have not the rights to do this?
"This pull request contains merge conflicts that must be resolved.
Only those with write access to this repository can merge pull requests."

@Qlimax90 if you pull rebase to latest staging in your local copy, solve the conflicts and push the code back to your online copy, everything should be ok!

@Qlimax90 Will you update your PR or should I recreate it? I think that it is a good change (get rid of the loading of the Mootools framework). If you are still keen to do it, then please do it in the next days if possible! If you do it, then please also add a blank line after the switch statement. Thank you in advance!

@dgt41 @Kubik-Rubik Please feel free to recreate, at the moment I am struggeling with the local copy due to the fact that I created the PR online. The local branch is not working, sorry...

Closed in favor of #7403. Thank you @Qlimax90 for your contribution!