search

jegelstaff / formulize

Unified data management. One system to handle all data entry, collection, and reporting, across all your unique workflows. Simplify processes. Make people happy.
https://www.formulize.org
GNU General Public License v2.0
27 stars 14 forks source link

Last commits for calling database mades an issue #47

Closed flaggada closed 10 years ago

flaggada commented 10 years ago

Hello, in admin side, with the last version, this message

Fatal error: Call to undefined method XoopsMySQLDatabaseProxy::escape() in /home/www/web/modules/formulize/admin/formindex.php on line 121

Unable to apply the 4.0 patch Xoops 2.5.5 PHP 5.3

Thanks

eikonos commented 10 years ago

Looks like Xoops has changed the database object and removed the escape() function. In preparation to move from mysql to mysqli, we've removed direct calls to mysql_real_escape_string() in favour of using database object calls. It may be possible to switch to the quote() function instead, as that seems to exist in both codebases.

jegelstaff commented 10 years ago

I think escape method was added by ImpressCMS and was never in XOOPS, most likely. I've started a pull request with XOOPS to see if they'll add it. But in the meantime, I've also updated the code in Formulize so that we should be able to get around this issue, though in not as nice a way...by using the quote method in the $xoopsDB, which is only adding slashes, not using the full mysql_real_escape_string (even though it's deprecated in PHP, which is why we started this change in the first place, it is still better than just adding slashes).

Please see if the latest change fixes the issue for you?

flaggada commented 10 years ago

Hello

It's working now, thanks for the change. :-)

But is it possible to transform "ICMS_member" in "member" in class/usersGroupPerms.php, cause it makes xoops crash.

And now it seems that the new Menu Entries doesn't work. Unable to create or modify some links.

Is it another xoops issue due to an older php way of Life ? :-(