AjaxMadeSimple Module is not properly cleaning input params

stetman · Post by **stetman** » Sat Jan 21, 2012 9:54 am

I am using cmsms 1.10 and just installed the polls module, everything seems to work fine.
Only thing is that my admin log has 100's of pages full with:
AjaxMadeSimple Module is not properly cleaning input params

Does anybody know what's happening ?

Marc

staartmees · Post by **staartmees** » Sat Jan 21, 2012 12:04 pm

This project is marked as stale. This means that it may work fine, but it has no file releases or repository commits in over a year. Use your own discretion when using this project.
It's obvious this module is no longer supported and it isn't compatible with cmsms 1.10.x

oliver341 · Post by **oliver341** » Sun Apr 15, 2012 12:23 am

I have the same problem, with "Module is not properly cleaning input params" appearing in the admin log every time a page is loaded on the site, AjaxMadeSimple is being blamed (I use it for the Polls module).

Module was updated on 2012-02-13 so it seems it isn't stale.

CMSMS version: 1.10.3
AjaxMadeSimple version: 0.3.3

My developer Page on the Forge · Post by **Jo Morg** » Sun Apr 15, 2012 5:23 pm

AFAIK it is more of a warning than an error, although, IIRC, that may change in CMSMS 1.11 and became an error. I think it doesn't hurt to file a bug report, if there isn't one already, on the forge.

oliver341 · Post by **oliver341** » Sun Apr 15, 2012 9:47 pm

There already seems to be an open bug about this on the forge.

http://dev.cmsmadesimple.org/bug/view/6699

It has been open for about 9 months.

My developer Page on the Forge · Post by **Jo Morg** » Sun Apr 15, 2012 11:03 pm

Yeah! It's not a problem as it does not interfere in the module normal functions, at least as far as CMSMS 1.10.X (at least as far as I recall). It may become a problem with CMSMS 1.11 if the module is not updated until then.
Unless someone more knowledgeable than me corrects me...

oliver341 · Post by **oliver341** » Mon Apr 16, 2012 1:19 am

Jo Morg wrote:It's not a problem as it does not interfere in the module normal functions

It is a problem in that the Admin Log is flooded with messages to the point of becoming useless.

vilkis · Post by **vilkis** » Mon Apr 16, 2012 6:02 am

Really, it floods Admin log. It should be onetime message but not repetitive. It is issue of CMSMS and not the module. For the moment I hacked CMSMS core file to disable these messages.

vilkis

oliver341 · Post by **oliver341** » Mon Apr 16, 2012 9:54 am

Yeah, a one-time message might be good, or something like "the last message was repeated 355 times" like syslog does, maybe.

But ultimately something inside the module needs fixing.

vilkis · Post by **vilkis** » Mon Apr 16, 2012 10:07 am

This problem is the same as in http://dev.cmsmadesimple.org/bug/view/7016
The solution could be to make this message not repetitive in CMSMS core.
vilkis

oliver341 · Post by **oliver341** » Mon Apr 16, 2012 10:15 am

If it's simple I'll just insert one or two lines of code into the code of AjaxMadeSimple. Does anyone know how you "clean input params"?

CMS Made Simple Forums

AjaxMadeSimple Module is not properly cleaning input params

AjaxMadeSimple Module is not properly cleaning input params

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa

Re: AjaxMadeSimple Module is not properly cleaning input pa