[TYPO3-core] RFC #12512: Cannot move multiple records with clipboard

Xavier Perseguers typo3 at perseguers.ch
Fri Nov 20 16:58:01 CET 2009


Hi Ernesto,

> It would not be tricky to review if the bug could be reproduced (even if
> not in DBAL). I could move multiple records, so this is probably not the
> way to reproduce the problem.

You did not what I said... You used tt_content as source of records and the problem obviously arises only when TCA does not contain a value for ['ctrl']['transOrigPointerField'] which is not the case 
for tt_content...

>> However, even with MySQL, you may add debugging code into
>> t3lib_BEfunc::getRecordsByField, filtering by $theTable and showing
>> that, e.g., when moving tt_news categories (that is, records from table
>> 'tt_news_cat'), the generated query is invalid when you try to move one
>> or more records using clipboard #1-#3. I least, it seems easy for
>> anybody to give a +1 by reading and "thinking" :-) Hopefully, this will
>> be sufficient...
> 
> Still it would be nice to know what the problem exactly is. If the
> records are being moved despite the wrong query, what does that query
> do?? Where can I see the negative impact of it?

Read the patch and my explanation. It's obvious that if 'transOrigPointerField' is not found in TCA, an invalid query will be issued. This method is used to move translations of the selected record. 
As long as you work with records that have 'transOrigPointerField' defined, you cannot reproduce the bug. Now, when using mysql, the invalid query is silently ignored by MySQL and the other records 
are copied/moved, but when using DBAL without 'native' driver, the parser dies and as such only first record is moved/copied.

-- 
Xavier Perseguers
http://xavier.perseguers.ch/en

One contribution a day keeps the fork away


More information about the TYPO3-team-core mailing list