[TYPO3-core] RFC #16517: Bug: t3lib_beFunc::getRecordWSOL() does not use it's param $useDeleteClause in else branch

[Ernesto Baschny [cron IT]]

Hi Steffen,

also +1 by reading from my side.

really a no-brainer, it makes no sense to ignore this flag if a
different list of fields is requested. Go ahead and commit.

I tracked down when this problem was introduced, and it was with the
commit from Ingmar and Olly for the "IRRE" feature in 4.2 (2006, rev.
1809 [1]).

Thanks!

Cheers,
Ernesto

[1] http://forge.typo3.org/repositories/revision/typo3v4-core/1809

Steffen Ritter schrieb am 22.11.2010 18:24:
> Hey list,
> This is an SVN patch request.
> 
> Type: Bug
> 
> Bugtracker references:
> http://bugs.typo3.org/view.php?id=16517
> 
> Branches:
> trunk, 4-4, 4-3
> 
> Problem:
> 
> Solution:
> t3lib_beFunc::getRecordWSOL() has the last param $userDeleteClause which
> defaults to true....
> this one is passed to t3lib_beFunc::getRecord...
> 
> This param is missing in the else branch - when '*' fields are required...
> This is wrong, and returns NULL even if you requested to include deleted
> records...
> 
> Notes:
> I would see this as a nobrainer, but as it's about several branches and
> t3lib_beFunc acts relative widely I think this should be considered as
> normal rfc.
> 
> Notes2:
> This one is required for the tcaTree.
> 
> regards
> Steffen