JoomFish / joomfish-2.5

Joom!Fish 2.5 core development
http://www.joomfish.net/
34 stars 27 forks source link

Fatal error in menu params when there is a fieldset description #119

Closed Bakual closed 11 years ago

Bakual commented 11 years ago

$this->escape is not allowed in your class. It generates the following error:

Fatal error: Using $this when not in object context in /.../libraries/joomfish/translateparams/xml.php on line 218.

This PR just removes the escape call as a workaround. If the escaping is needed you need to rewrite it.

Bakual commented 11 years ago

Just saw the commit has a lot of changed lines, probably due to different line endings. This was done by the GitHub webinterface and doesn't really change anything in the code. I just changed the line 218 ;-)

klas commented 11 years ago

Thnx for reporting - I fixed this by replacing escape with htmlspecialchars and re-released jf-stable with this fix.

armand1981 commented 11 years ago

Hello klas, can you tell me how you have fixed this error.

Our error is:

Fatal error: Call to protected method JFormField::getName() from context 'TranslateParams_xml' in /home/blabla/domains/blablapublic_html/libraries/joomfish/translateparams/xml.php on line 114

We are looking for hours now but can't find the solution. How do you do" I fixed this by replacing escape with htmlspecialchars and re-released jf-stable with this fix."?

thanks

klas commented 11 years ago

Hi, your problem is not the same as this issue that was fixed. Please see other thread you posted in.