Introduce jquery formvalidator for com_redirect

[dgrammatiko]

Executive summary

This PR converts the form validation on com_redirect to use plain jquery (no mootools call on every form

[smanzi]

@test success

[anibalsanchez]

I have applied the patch on 3.3.6, but I'm getting:

An error has occurred. 500 Layout default_batch not found

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5052.

[smanzi]

@anibalsanchez That's strange... I've tested it on 3.3.6 too: no problem. Do you use com_patchtester for applying PRs?

[dgrammatiko]

@anibalsanchez Checking the repo 3.3.6 I see that this file doesn’t exist there. It must have been added later on

[smanzi]

Guys, I just tested it again, just to be sure, and... no problem! What's going on here??

[anibalsanchez]

Yes, patchtester applies it Ok... weird ... how a fresh install have become corrupted in com_redirect.

I have applied #4888 and #5052 in other site, it works OK. @test success!

[smanzi]

@anibalsanchez can it be that you have other unrelated PRs applied? Unhappily in com_patchtester we don't have a filter to see which one are currently applied and we must scroll the whole list...

[dgrammatiko]

@smanzi I already requested one more view in com_patchtester, take a look here ianmacl/patchtester#9

[anibalsanchez]

It is a clean installation for these tests ... anyway I'm re-installing and keeping an eye on patching consistency.

[smanzi]

@DGT41 Ooops... I just did the same request here: joomla-extensions/patchtester#54

[smanzi]

@DGT41 Actually not the same request. I think you made yours in the wrong repo...

[dgrammatiko]

@roland-d this, as well, got 2 good tests (you have to go through the comments)

[roland-d]

@DGT41 Thanks for the info. That is why we are trying to get people to use the Test results module on every issue, so we can easily see how many tests it has. Reading long commments isn't always as clear ;)

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5052.

[roland-d]

Moving to RTC as we have 2 successful tests.

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5052.

[dgrammatiko]

@roland-d I can see all the hard word needed in the current workflow, that’s why I proposed this but it was turned down...

[roland-d]

@DGT41 Yeah the current workflow isn't ideal. Thanks for the link, I will comment there.

[smanzi]

@DGT41 there are problems with this...
Creating a new redirection I get in console:

TypeError: document.formvalidator is null index.php:30
Use of getAttributeNode() is deprecated. Use getAttribute() instead. jquery-migrate.min.js:2
TypeError: b is null core.js:1

And I cannot save the redirection...

[smanzi]

Also, tooltips not correctly rendered...

[dgrammatiko]

Fixed, for tooltips there is #5137

[smanzi]

Confirmed!
Tooltips malfunction was related, but it is now OK!

@test success for this PR
Tested the set of: #5046 #5048 #5049 #5050 #5051 #5052 #5053 #5054 #5056 #5058 #5113 #5255