[TYPO3-core] RFC: #7370: Remove page doktype 5 (not in menu) and merge Normal with Advanced

[Oliver Hader]

Hi Dmitry,

Dmitry Dulepov [typo3] schrieb:
> Hi!
> 
> Oliver Hader wrote:
>> That this feature can go into the core isn't subject of the discussion 
>> anymore. The crux of matter is the moment when it shall be committed.
> 
> I fully agree!

Great! :)

> I see several major points:
> - existing pages with doktype=(2|5) <== RESOLVED by Steffen with upgrade 
> script
> - TS that uses doktype <== RARE thing, instructions can go to NEWS.txt
> - resource leak in latest patch <== TRIVIAL
> - extension that use doktype=(2|5) <== NEED TO SOLVE ("die('No such 
> doktype')"???)

I think a die() is a bit too much. Maybe this works (I have to test):
| trigger_error('Deprecated disposal of doktype 5', E_USER_WARNING);
This should put out a nice annoying warning and put it to the error log.

> If that last two points are solved, I am quite ok with committing. But 
> we have to maek this commit and entry in news.txt with "!!!" of course.

Yes. Additionally in the release notes for TYPO3 4.2(-beta1, -RC).

olly
-- 
Oliver Hader
http://inpublica.de/