Project

General

Profile

Actions

Bug #16811

closed

fullQuoteStr() method not available with TYPO3 v3.7.x

Added by Francois Suter almost 18 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2006-12-26
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The new patched version of rtehtmlarea for TYPO3 v3.7.x (version 1.1.4) calls on the fullQuoteStr() of class.t3lib_db.php. But this method only exists in TYPO3 v3.8.0+ resulting in a crash with TYPO3 v3.7.x.

There is a single call to fullQuoteStr() in rtehtmlarea/class.tx_rtehtmlarea_base.php, which I replaced by a call to quoteStr() and that solved the problem. I include the corresponding diff file with this bug.

TYPO3 v3.7.0
rtehtmlarea v1.1.4
(issue imported from #M4699)


Files

class.tx_rtehtmlarea_base.diff (1.82 KB) class.tx_rtehtmlarea_base.diff Administrator Admin, 2006-12-26 15:26
class.tx_rtehtmlarea_base-2.diff (1.95 KB) class.tx_rtehtmlarea_base-2.diff Administrator Admin, 2006-12-26 17:21
Actions #1

Updated by Francois Suter almost 18 years ago

I made my patch a bit better, by simulating exactly the behaviour of fullQuoteStr(). It also appeared that the query where this method is called doesn't return any results in my case. So I added a test to check whether the DB resource was false or not, to avoid generating a PHP warning.

This is all included in the second patch.

Actions #2

Updated by Michael Stucki almost 18 years ago

Hi Francois,

thanks for sharing this patch with us. However, since TYPO3 3.7.x is very outdated anyway, there won't be a new version with this patch.

I also don't think that it would be useful to patch rtehtmlarea directly because again this would be inviting to use an outdated product...

- michael

Actions #3

Updated by Francois Suter almost 18 years ago

Michael, I understand your reasons and I assure you I don't delve into 3.7.0 for the sheer pleasure of nostalgia :-)

Version 1.1.4 of rtethmlarea is the patched version recommended in the recent security announcement. Which is why I tried to use it, to patch some client's site that wanted the security breach problem addressed, but didn't want a full migration to a more recent version of TYPO3.

Actions #4

Updated by Martin Kutschker almost 18 years ago

A strange lapsus that the recommended version for a security fix does not work on the target platform.

Actions #5

Updated by Francois Suter almost 18 years ago

I was indeed perplex. I can be wrong, but I don't see where...

Actions #6

Updated by Michael Stucki almost 18 years ago

OK, you are right. Sorry for closing in the first run. I have sent a mail to the security team and expect someone of them will take care of this...

Actions #7

Updated by Rupert Germann almost 18 years ago

I uploaded version 1.1.5 of rtehtmlarea to TER which doesn't contain the call to fullQuoteStr().
http://typo3.org/extensions/repository/view/rtehtmlarea/1.1.5/

Actions #8

Updated by Stanislas Rolland about 17 years ago

Is this problem resolved?

May it be closed?

Actions #9

Updated by Francois Suter about 17 years ago

Yes it is fixed, you can close the issue. Sorry for answering late I didn't receive any e-mail about your latest note.

Actions

Also available in: Atom PDF