[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