[TYPO3-core] RFC #4759: Incorrect values of colPos, sys_language_uid and sorting after moving records in web > list

[Stanislas Rolland]

Hi Steffen,
> looking at would you've done crosschecking tcemain and co what you've
> done sounds reasonable to me. I got a question:
> In v1 you fixed "moveL10nOverlayRecords" internally to ignore destPid
> always moving translation records after to original one.
>
> In v2 you do not "fix" this function anymore, but adapt 2 calls of it,
> by handing over another $destPid value.
>
I think it is the other way around: I fixed this function in  the second 
version (v2) so that the localization records get inserted in the 
correct order.

> Could you explain, why it makes sense to keep the function with
> possibilities to create struggle?
>
You may be right. Since the localization records should always be moved 
after the record they localize, we could ignore (and deprecate in trunk) 
the third parameter of this function and just use -$uid (using the 
second parameter) as $destPid, as in attached v3.

Regards,
Stan


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: typo3core_bugfix_4759_trunk_v3.patch
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20110208/9283f21c/attachment.asc>