
06-05-2010, 02:25 PM
|
|
|
Join Date: May 2004
Location: Florida
Posts: 165
Благодарил(а): 0 раз(а)
Поблагодарили:
0 раз(а) в 0 сообщениях
|
|
Quote:
Originally Posted by GeekDrew
I just installed this a couple of days ago, and the first couple of users that tried to use it (other admins of mine, as I like to do extensive testing) managed to break the pins, because of the XML validation error that others have referenced as "running the XML file".
The cause of the problem is that the pimage link isn't being parsed safely for XML.
Here's a fix that I think will work (at least it works for me -- I do NOT guarantee this, if it causes spontaneous combustion, fire, death, etc., I'm not to blame):
*** I JUST SAID THAT I DO NOT GUARANTEE THIS WILL WORK. I DISCLAIM ALL LIABILITY. YOU MUST BACK UP YOUR DATABASE AND ALL OF THE FILES RELATED TO THIS MODIFICATION, SO THAT YOU CAN RESTORE THEM IF NECESSARY. ***
In /forums/includes/cron/vbgooglemapme_cron.php:
Find:
Immediately ABOVE the text I have quoted, place the following text:
In the same file, find this:
Immediately BELOW the text that I have quoted, place this:
That should render all entities in the 'pimage' field safe for XML, so it will validate. However, that script is only called when the CRON job is executed... when someone edits their entry online, a different copy of that same script is called (which is really bad form, in my opinion, but hey, I didn't develop this...). That second copy is located at /forums/includes/functions_vbgooglemapme.php. You need to make the exact same changes in that file as I have mentioned above.
Now... I am not claiming that this will solve everyone's issues. It more than likely will not. It will, however, fix problems with 'pimage' being the reason that the XML won't validate. I think that this will probably fix the problems that quite a few of you are happening.
|
Really old post of yours GeekDrew but the .txt file on your server is long gone. I need to apply this patch again and if you still have that (or anyone else reading), please let me know.
Also there are other fields that are in need of validation.
Has anyone gotten a bulletproof solution here yet?
|