Project

General

Profile

Actions

Bug #25489

closed

Wrong quote of ORDER BY

Added by Patrick Schriner almost 13 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
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 #1

Updated by Xavier Perseguers almost 13 years ago

ORDER BY {number} successfully tested with MSSQL

Actions #2

Updated by Xavier Perseguers almost 13 years ago

ORDER BY {number} successfully tested with Oracle

Actions #3

Updated by Patrick Schriner almost 13 years ago

My "failure" case was using ADODB + MySQL.

Actions #4

Updated by Mr. Hudson almost 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 #5

Updated by Xavier Perseguers almost 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 #6

Updated by Mr. Hudson almost 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 #7

Updated by Mr. Hudson almost 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 #8

Updated by Xavier Perseguers almost 13 years ago

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

Updated by Patrick Schriner almost 13 years ago

Thanks for the quick handling of this patch!

Actions #10

Updated by Michael Stucki over 10 years ago

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

Updated by Michael Stucki over 10 years ago

  • Category set to 999
Actions #12

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF