Bug #27761
closed
adodb doesn't quote simple quotes correctly
Added by Anonymous over 13 years ago.
Updated almost 10 years ago.
Description
The adodb drivers are very cool and stuff but there are some problems with the quoting of parameters. For example, everytime you put a simple quote ( ' ) into a parameter, the string is wrong quoted <<<<<<<and the query parsing throws an error.
This happens with dbal using adodb driver mssql and, as far as I have seen it, it might fail for every other driver too where the "replaceString" isn't defined properly.
Currently the replaceString for the mssql driver and mssqlnative driver is '' (two simple quotes). This should be replaced by \'\' (two escaped double quotes) I think.
Greets
Tizian
Did there change anything?
Did not investigate yet. You told me you found some problems when this change is performed from a FE or a BE perspective. Would be useful to put this information here.
Hello Xavier,
in facts this the "reescaping" of the quotes is not a proper solution and should not be pursued.
I finally figured out that the problem is more likely to be solved in the dbal parser instead of the adodb core.
Greets Tizian
- Status changed from New to Needs Feedback
- Is Regression set to No
Hi,
as this issue is very old. Does the problem still exists within newer versions of TYPO3 CMS (6.2.4)?
We once posted a patch for this but I'm not sure it made it through gerrit.
It seems that actually, the method "replaceString" has been completely removed from the 6.2.x version, so this may or may not be an issue anymore.
As we'll be probably performing an update to 6.2 for a system using dbal, we might be able to tell you more about it about that time (2-3 months from now).
Best Regards
Tizian
Hi,
3 month are over ... so anything to report?
Yes, sorry about that.
We had to postpone said update, I won't be able to give any feedback before March or April 2015.
Best Regards
Tizian
- Status changed from Needs Feedback to Closed
So I close this issue for now.
If you have upgraded and the issue occurs feel free to open a new issue.
Thanks for your help.
Also available in: Atom
PDF