[SOLVED] Fabrik Form Module not processing on home page

[Brandon.parrish]

Hi, I have the fabrik form module processing properly on a different page. However when I have the same fabrik form module on the home page, it will not process. Any idea why?

https://www.ridersurance.com/
"Pick a plan" form at the bottom.

Thanks in advance.

Fabrik 3.5
J3.6

 

[cheesegrits]

I think it's because you have a module on the home page, "image and content fader", which is overriding a Mootools function (periodical()).

Try turning that module off.

-- hugh

 

[Brandon.parrish]

Thanks. unpublishing that module allowed fabrik to work correctly. Is there a workaround to have both modules work?

 

[cheesegrits]

Nope. You'd have to talk to the authors of that module, see if they can either not override that function prototype, or call the original when they've finished doing whatever it is they need to do.

-- hugh

 

[Brandon.parrish]

Here is the response from them. They gave me a new version that doesn't use mootools, it uses jquery now.

This new module does not use mootools, it is jquery only.

periodical is not a function that we declare, that is a default function of the javascript library itself, so it's not something we can change. I also checked your site and didn't see any javascript errors on the page.
​

The fabrik module will still not submit though. Anymore ideas?

 

[cheesegrits]

Point out this line to them:

Function.prototype.periodical=function (c,b,a){return setInterval(this.passx((a==null?[]:a),b),c);};

... line 71 in jd.gallery.jquery.js, which overrides the Mootools periodical() function, which is what I said in post #2.

And no, there won't be any errors on the page, but when any Mootools code calls periodical(), it is calling their function, instead of Mootools'.

Ask them nicely if they could use some other function name, perhaps jdPeriodical.

-- hugh

 

[Brandon.parrish]

They responded with:

We can try doing a find a replace for periodical to something else in the js file, but we are not going to re-write our script because another script is using the same name. You would have to replace all calls of periodical with a new function, it's not tested though, so no guarantees that it will work.​

 

[cheesegrits]

It's not "another script", it part of the Mootools framework:

http://mootools.net/core/docs/1.5.2/Types/Function#Function:periodical

Mootools may have fallen out of favor and is being deprecated in J! in favor of jQuery, but it is still a major JS framework library, and is still shipped with J!.

Overriding global function prototypes is NOT something which should be done lightly, and DEFINITELY should not tread on things like jQuery or Mootools.

-- hugh