diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2016-11-09 11:49:57 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2016-11-09 11:13:34 +0000 |
| commit | a71d6a2a847828b737e5b6cc5eefe877613085b7 (patch) | |
| tree | 2fe758330a61cab221715749973911acbc82aa88 /dbaccess/source/ui/dlg/adtabdlg.cxx | |
| parent | 243977708ceaf877e520ef0bb393aa49040d65c9 (diff) | |
loplugin:expandablemethods in dbaccess
Change-Id: I6087a3eff46926646ac1637615a0af30b38956a4
Reviewed-on: https://gerrit.libreoffice.org/30712
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'dbaccess/source/ui/dlg/adtabdlg.cxx')
| -rw-r--r-- | dbaccess/source/ui/dlg/adtabdlg.cxx | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/dbaccess/source/ui/dlg/adtabdlg.cxx b/dbaccess/source/ui/dlg/adtabdlg.cxx index 0466f3c3480b..b6dd66a63996 100644 --- a/dbaccess/source/ui/dlg/adtabdlg.cxx +++ b/dbaccess/source/ui/dlg/adtabdlg.cxx @@ -405,17 +405,6 @@ void OAddTableDlg::Update() m_xCurrentList->updateTableObjectList( m_rContext.allowViews() ); } -void OAddTableDlg::impl_addTable() -{ - if ( m_xCurrentList->isLeafSelected() ) - { - OUString sSelectedName, sAliasName; - sSelectedName = m_xCurrentList->getSelectedName( sAliasName ); - - m_rContext.addTableWindow( sSelectedName, sAliasName ); - } -} - IMPL_LINK_NOARG( OAddTableDlg, AddClickHdl, Button*, void ) { TableListDoubleClickHdl(nullptr); @@ -425,7 +414,13 @@ IMPL_LINK_NOARG( OAddTableDlg, TableListDoubleClickHdl, SvTreeListBox*, bool ) { if ( impl_isAddAllowed() ) { - impl_addTable(); + if ( m_xCurrentList->isLeafSelected() ) + { + OUString sSelectedName, sAliasName; + sSelectedName = m_xCurrentList->getSelectedName( sAliasName ); + + m_rContext.addTableWindow( sSelectedName, sAliasName ); + } if ( !impl_isAddAllowed() ) Close(); return true; // handled |