Project

General

Profile

Actions

Bug #19209

closed

Deprecated code into stdgraphic

Added by Vincent MARIANI over 16 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Should have
Category:
Communication
Target version:
-
Start date:
2008-08-11
Due date:
% Done:

0%

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

Description

the class stdgraphic don't use the function chosen in Install Tool (between recode, iconv or mbstring), thus Typo3 crashes if we haven't recode installed on our server.

Fatal error: Call to undefined function recode_string() in /.../t3lib/class.t3lib_stdgraphic.php on line 2014

Is it possible to make a switch case there ?

(issue imported from #M9152)


Files

9152.diff (2.73 KB) 9152.diff Administrator Admin, 2008-08-18 16:06

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #19887: After update to 4.1.8 image rendering doesn't work anymoreClosedMichael Stucki2009-01-22

Actions
Actions #1

Updated by Benni Mack over 16 years ago

Hey Vincent,

well, pretty simple: if you choose "recode", you gotta make sure that this php extension is installed on your system. It's not a matter of having the error, more of choosing the right configuration for your system available.

What do you think?

Actions #2

Updated by Dmitry Dulepov over 16 years ago

Benny, the function is hard-coded in the TYPO3 code.

Actions #3

Updated by Dmitry Dulepov over 16 years ago

Btw, the option is deprecated from TYPO3 3.6 but the fix is very simple...

Actions #4

Updated by Vincent MARIANI over 16 years ago

Yes, recode was hard-coded and deprecated like you said. Since I'm not familiar with this fonction (recode). I didn't want to fix it by myself even it seems easy ;)

Your patch seems enough for the fix, doesn't it ?

Actions #5

Updated by Dmitry Dulepov over 16 years ago

Yes, the patch is enough. And it is now committed to 4.1, 4.2 and 4.3

Actions

Also available in: Atom PDF