Project

General

Profile

Actions
Copy link

Feature #51219

closed

Autodetection for integrated charset-conversions

Added by Stefan Neufeind over 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Performance
Target version:
-
Start date:
2013-08-20
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Currently t3lib_conv_method and t3lib_cs_utils needs to be specified in TYPO3_CONV_VARS. If not given, it defaults to "homemade PHP-code".

Add a proper autodetection so default-installations (no value set) become faster.

Files

typo3_character_preset.png (65.3 KB) typo3_character_preset.png Alexander Opitz, 2014-02-21 11:13
Actions
Copy link
 #1

Updated by Stefan Neufeind over 11 years ago

Reading on the internet there is no clear preference for either mbstring or iconv. I've found a post where somebody mentioned mbstring was a lot faster for him, while others mentioned that iconv is more likely to be shipped with PHP out-of-the-box than mbstring because it is more portable.

Thus I'd propose as autodetection-order: mbstring, iconv, (recode,) internal

For PHP6 "ICU" will be used. This is also available in current PHP-installations with the "intl"-extension already. But TYPO3 doesn't yet have ICU-support. (separate feature we might consider)

Actions
Copy link
 #2

Updated by Gerrit Code Review over 11 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23205

Actions
Copy link
 #3

Updated by Stefan Neufeind about 11 years ago

  • Status changed from Under Review to Needs Feedback

Christian Kuhn mentioned that in 6.2 is supposed to work via a "preset"-autodetection in the installtool. I didn't so far verify this. But on that basis the patch so far was abandoned.

Actions
Copy link
 #4

Updated by Alexander Opitz almost 11 years ago

Till yet mbstring, iconv and internal is supported.

So can we close this issue?

Actions
Copy link
 #5

Updated by Stefan Neufeind almost 11 years ago

Problem is/was that there was no autodetection and if the setting was not set it defaulted to slow, handcrafted PHP-code. Christian Kuhn said he wanted to do something different in this area for 6.2 - but I'm not sure what finally arrived in master now.

Actions
Copy link
 #6

Updated by Christian Kuhn almost 11 years ago

Yeah, should be done. The configuration presets in install tool can detect mbstring and iconv (prefer mbstring), so this configuration can be selected with 1 click. During installation, the "best" alternative is chosen automatically (mbstring if available).

Actions
Copy link
 #7

Updated by Alexander Opitz over 10 years ago

  • Status changed from Needs Feedback to Closed
  • Assignee deleted (Stefan Neufeind)

Ok, so closing this issue as 6.2 is out and it works like Christian described.

Actions
Copy link

Also available in: Atom PDF