summaryrefslogtreecommitdiff
path: root/dbaccess/source/ui/dlg/queryorder.cxx
diff options
context:
space:
mode:
authorLionel Elie Mamane <lionel@mamane.lu>2013-07-11 16:53:23 +0200
committerLionel Elie Mamane <lionel@mamane.lu>2013-07-11 17:53:30 +0200
commit40370f759c403c5f07fb4d77680bd8f954e55231 (patch)
treed720a2343ae95967dc19877423e7435b037f6b37 /dbaccess/source/ui/dlg/queryorder.cxx
parentad1049716a6e62066a196c78b163c30af3a2d264 (diff)
ORDER BY columns are prioritarily *SELECT* columns
as opposed to *table* columns, and notwithstanding HSQLDB 1.8 (our embedded database) bugs. Actually, supporting ORDER BY on non-select (but table) columns is OPTIONAL for DBMSs (but quite common) Change-Id: I6725dfda36b09429a78262bff6f3d3e3dd9032b6
Diffstat (limited to 'dbaccess/source/ui/dlg/queryorder.cxx')
-rw-r--r--dbaccess/source/ui/dlg/queryorder.cxx25
1 files changed, 1 insertions, 24 deletions
diff --git a/dbaccess/source/ui/dlg/queryorder.cxx b/dbaccess/source/ui/dlg/queryorder.cxx
index 60860c32d4cc..c241c77076b6 100644
--- a/dbaccess/source/ui/dlg/queryorder.cxx
+++ b/dbaccess/source/ui/dlg/queryorder.cxx
@@ -234,30 +234,7 @@ OUString DlgOrderCrit::GetOrderList( ) const
sOrder += OUString(",");
String sName = m_aColumnList[i]->GetSelectEntry();
- try
- {
- sal_Bool bFunction = sal_False;
- Reference< XPropertySet > xColumn;
- if ( xColumns.is() && xColumns->hasByName( sName ) && (xColumns->getByName( sName ) >>= xColumn) && xColumn.is() )
- {
- if ( xColumn->getPropertySetInfo()->hasPropertyByName(PROPERTY_REALNAME) )
- {
- OUString sRealName;
- xColumn->getPropertyValue(PROPERTY_REALNAME) >>= sRealName;
- sName = sRealName;
- static OUString sFunction("Function");
- if ( xColumn->getPropertySetInfo()->hasPropertyByName(sFunction) )
- xColumn->getPropertyValue(sFunction) >>= bFunction;
- }
- }
- if ( bFunction )
- sOrder += sName;
- else
- sOrder += ::dbtools::quoteName(sQuote,sName);
- }
- catch(const Exception&)
- {
- }
+ sOrder += ::dbtools::quoteName(sQuote,sName);
if(m_aValueList[i]->GetSelectEntryPos())
sOrder += sDESC;
else