Answered Can't save Mobile Detection plugin – other resources/elements do save

Hello

Rev 2.3.5 || php5.4 || mysql 5.1.73

I'm trying to add a 'tablet' tag into the mobiledetection plugin but when I save the changes the manager goes awry (see attached images 'md_error' & 'md_error2').
Clicking the 'save' button then tries to resave the file with the same result. The duplicate button brings up the duplication window but isn't usable. The Help button shows the help pop-up but with no content and the close button throws me out to my preview site (www.domain.com/?). If I refresh the page it all loads correctly but without any changes I have made.

I thought it might have been to do with my reworked code but I've also had the same error when loading the plugin page and pressing 'save' without actually altering anything.

I looked into the php mod_security issue but seeing as it's only this plugin that I'm getting the error on (I can save all other kinds of elements/resources) I presume it's more to do with that than the server. Does anyone know if there is anything I can do/research?

Thanks

You can save the original plugin code OK?

Afraid not. If I open the plugin and click save without doing anything it does the same.
I've had it before with some MigX TVs but I wasn't able to nail the solution down. One had a mistake in the input fields code I had created and another was an error with a text area element that I had put some html into (I think it was an error in the html that was causing the problem).

Well, it installed fine here and I can save the plugin (even some minor modifications I made to it) with no problem. So the only thing that comes to mind is in fact some kind of mod_security issue. I've run into some pretty odd issues with that on different servers.

About the only thing you can do is trial-and-error to pin down just what is causing it. Start with copying the code to a text editor, then saving it empty. Then start adding small sections of code until it stops working.

Or... you can put the whole thing in a separate .php file in the core/components/mobiledetection/ folder, and just use a single include() in the actual plugin code.

The first thing I did was to download the latest mdetect.php file, and put it in core/components/mobiledetection/lib/, instead of in assets. There shouldn't be any .php or other executables in assets. Then I changed the path of the include in the plugin to MODX_CORE_PATH . components/mobiledetedtion/lib/mdetect.php

http://blog.mobileesp.com/?page_id=20 - the source URL in the plugin comments is obsolete, google code has been shut down so it's on github now.

Thanks Susan.
I tried changing the mdetect.php plugin as suggested but couldn't save the plugin after changing the file path.
I then tried reinstalling the plugin, then uninstalling and reinstalling to no avail.

I have just removed all the code and re-input it line by line and have found a few lines that were causing the error when I saved the file. I'm not sure why because my php knowledge isn't too hot but something may jump out at you or another forum member?

Line 138

Line 140-141

Line 143

Line 145

And finally Line 192 (the mdetect path)

It's time to talk to your hosting provider's support desk. They can look at their security logs and tell you if that's what the problem is. It sure looks like a mod_security issue to me.

Yep. It was that cheeky little mod_security again.
Thanks for all the responses.