[TYPO3-core] Announcing TYPO3 4.5.18, 4.6.11 and 4.7.3

Markus Klein klein.t3 at mfc-linz.at
Thu Aug 9 13:52:31 CEST 2012


Hi Michael,

looks like we have a different opinion on what is right or wrong. ;-)
The given example is IMO wrong, not the result.

Selecting 'pages.uid', but referring to it as only 'uid' is in my eyes the mistake.

Kind regards
Markus

> 
> Hi Markus,
> 
> >> Well, it is clearly a regression because it changes the behaviour in
> >> an unexpected way.
> >>
> >> See Stefans example:
> >>
> >> select.selectFields = pages.uid, pages.title
> >>
> >> Old result: SELECT pages.uid, pages.title New result: SELECT
> >> pages.uid, pages.title, tt_content.uid as uid, tt_content.pid as pid,
> >> tt_content.t3ver_state as t3ver_state
> >
> > Well, usually we fix a bug to achieve the wanted behavior. In this case, the intended functionality was always to NOT add the fields,
> when a * is present.
> > The new resulting query for Stefan's example is still valid. It still contains the two fields.
> > His problem was more the usage of the result.
> 
> I'm not sure how you conclude that this is a wanted behaviour.
> 
> http://wiki.typo3.org/TSref/select
> 
> The new query adds the fields though, but it also selects more fields.
> As this can lead to conflicts like we've seen in this example, it is a wrong behaviour, isn't it?
> 
> To reproduce the problem the unit test file for tslib_content should be extended with the following patch (see attachment).
> 
> - michael



More information about the TYPO3-team-core mailing list