Project

General

Profile

Actions
Copy link

Bug #25489

closed

Wrong quote of ORDER BY

Added by Patrick Schriner over 13 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Xavier Perseguers
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2011-03-31
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

DBAL wrongly quotes ORBER BY {NUMBER}.

The patch checks whether for this ORDER BY there is no TABLE, and the FIELD is just a number.

This causes issues with Extbase, as Extbase is using such queries.

Files

class_ux_t3lib_db.php.patch (821 Bytes) class_ux_t3lib_db.php.patch Patch proposal Patrick Schriner, 2011-03-31 12:24
Actions
Copy link
 #1

Updated by Xavier Perseguers over 13 years ago

ORDER BY {number} successfully tested with MSSQL

Actions
Copy link
 #2

Updated by Xavier Perseguers over 13 years ago

ORDER BY {number} successfully tested with Oracle

Actions
Copy link
 #3

Updated by Patrick Schriner over 13 years ago

My "failure" case was using ADODB + MySQL.

Actions
Copy link
 #4

Updated by Mr. Hudson over 13 years ago

Patch set 1 of change I4e0f8ecd265c3bde3f1da59b1ea53de8382c11b5 has been pushed to the review server.
It is available at http://review.typo3.org/1505

Actions
Copy link
 #5

Updated by Xavier Perseguers over 13 years ago

  • Status changed from New to Accepted
  • Assignee set to Xavier Perseguers
  • Target version set to 1.2.3 (TYPO3 4.5.x)
Actions
Copy link
 #6

Updated by Mr. Hudson over 13 years ago

Patch set 1 of change I0978b0aa72489dbc415455c01a391eab24d142f4 has been pushed to the review server.
It is available at http://review.typo3.org/1542

Actions
Copy link
 #7

Updated by Mr. Hudson over 13 years ago

Patch set 2 of change I4e0f8ecd265c3bde3f1da59b1ea53de8382c11b5 has been pushed to the review server.
It is available at http://review.typo3.org/1505

Actions
Copy link
 #8

Updated by Xavier Perseguers over 13 years ago

  • Status changed from Accepted to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #9

Updated by Patrick Schriner over 13 years ago

Thanks for the quick handling of this patch!

Actions
Copy link
 #10

Updated by Michael Stucki almost 11 years ago

  • Project changed from 329 to TYPO3 Core
  • Target version deleted (1.2.3 (TYPO3 4.5.x))
Actions
Copy link
 #11

Updated by Michael Stucki almost 11 years ago

  • Category set to 999
Actions
Copy link
 #12

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF