Bug #37221
closed
Follow up #35260: Missing selection field in t3lib_TCEmain::getPreviousLocalizedRecordUid()
Added by Felix Nagel over 12 years ago.
Updated almost 7 years ago.
Category:
DataHandler aka TCEmain
Description
Please revert the patch of #35260 as its obviously wrong. Patch of #35260 was included in all branches of TYPO3 afaics, so my patch should be applied to all branches of TYPO3.
Take a look at the function getPreviousLocalizedRecordUid as in TYPO3 4.5.15. First, the whole idea of reusing $sortRow as select statement is just bad practice. Second, using a combination like "sorting,colSort" as an index for $row wont work. Third, the current implementation creates SQL like ' AND sorting,colPos<0 AND ' which wont work.
The idea of adding colSort to select was correct (the function was broken before, too!) but the proposed fix introduces new problems and fixes none.
I have no idea how this could be reviewed by three people as its pretty obvious :-/
Files
Sadly I was not able to push my changes to Gerrit.
When trying to use "scp -p -P 29418 fnagel@review.typo3.org:hooks/commit-msg .git/hooks/" error message is "Permission denied (Publickey)". Any idea how to solve this aka how to copy the hooks?
Patch is attached as a file.
Yes, but I triple checked my username, SSH key and everything :-(
- Status changed from New to Under Review
Thanks for spotting this mistake and sorry for the trouble it caused. I must say I'm really puzzled that it worked at all, given that it seems really wrong, but I can assure you that it solved problems for me or I wouldn't have committed it. Your patch makes it really cleaner. I updated a second version with a modified commit message.
Im just glad this is fixed now and I finally managed to push my first commit to Gerrit :-)
Thanks for correcting my commit!
Would you mind explain to me why my first push was rejected (build marked as failed) by Gerrit?
And should I verify my own patch? I guess not ;-)
By the way: I managed to push to Gerrit by downloading the hooks manually (like advised in http://forge.typo3.org/projects/team-forge/wiki/Working_with_Git_and_Gerrit) and using 'git commit --amend'.
- Assignee set to Francois Suter
- PHP Version set to 5.3
Would you mind explain to me why my first push was rejected (build marked as failed) by Gerrit?
And should I verify my own patch? I guess not ;-)
I didn't understand either at first. The problem was that some lines were longer than 74 characters. To find out, you have to follow the link posted by Mr Jenkins and there click on the "Console output" link on the left-hand side. This lists all the details about Mr. Jenkins' complaints.
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF