Project

General

Profile

Actions

Bug #17790

closed

Remove "text_properties" etc. from tt_content table

Added by Benni Mack almost 17 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2007-11-14
Due date:
% Done:

0%

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

Description

There are these ultra-old buttons and dropdowns to select the font size etc. when editing a content element (from the RTE). I never ever used them, we have now a good RTE and if the RTE is disabled, people code HTML in the bodytext field. So there is no need for it.
Of course it breaks compatibility, but we have to do something to make TYPO3 more usable.

Anyway, if we'd remove these fields "text_face", "text_size"... the DB table would get smaller (and so every record) and the BE interface less cluttered. I don't know anybody who actually uses this feature, nor do I know an extension that uses it. I think this is even before 3.5 and completely outdated.
(issue imported from #M6713)


Files

Actions #1

Updated by Benni Mack almost 17 years ago

There are two files. One with some whitespace changes (that will be commited) and one for devs to read (no_whitespace...).

The patch hasn't been tested yet by me.

Also. This should work quite well with the update wizard.
  • new installations won't even know that these fields ever existed.
  • old installations (I doubt any installation with T3 3.8+ uses these fields) will set the fields to deprecated and if wanted they can be deleted with the DB analyzer in the install tool.
    We need a updated_wizard notice though so people will know. Also: The documentation needs to be updated.
Actions #2

Updated by Steffen Kamper almost 17 years ago

Hi Benni,

i tested second patch and works for me. Very good to remove unneccessary old functions, it's a good cleanup!

+1 from me.

I will test first patch later.

Actions #3

Updated by Benni Mack almost 17 years ago

you don't have to test both patches since they are made the same way. The one with the "no whitespace" is just easier to make a code review.

Actions #4

Updated by Benni Mack almost 17 years ago

Btw... Steffen: You're more than welcome to send this patch to the core list since I'm gonna be away the next days.

Actions #5

Updated by Steffen Kamper almost 17 years ago

np - i did the RFC in your name - have some nice and relaxed days!

Actions

Also available in: Atom PDF