From 4ab0912cd214a4b0bdcbcdd2711b831dab9ca60a Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Wed, 11 Apr 2018 14:38:11 +0100 Subject: weld PlaceEditDialog MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: I971784afc8c2ba6ebd7809c20ef8d730aafbfe31 Reviewed-on: https://gerrit.libreoffice.org/52727 Tested-by: Jenkins Reviewed-by: Caolán McNamara Tested-by: Caolán McNamara --- svtools/source/dialogs/PlaceEditDialog.cxx | 223 +++++++++++------------ svtools/source/dialogs/ServerDetailsControls.cxx | 210 ++++++++++----------- svtools/uiconfig/ui/placeedit.ui | 82 ++++----- 3 files changed, 247 insertions(+), 268 deletions(-) (limited to 'svtools') diff --git a/svtools/source/dialogs/PlaceEditDialog.cxx b/svtools/source/dialogs/PlaceEditDialog.cxx index 23eeaa21809b..a0da381d703c 100644 --- a/svtools/source/dialogs/PlaceEditDialog.cxx +++ b/svtools/source/dialogs/PlaceEditDialog.cxx @@ -17,73 +17,72 @@ using namespace com::sun::star::uno; -PlaceEditDialog::PlaceEditDialog(vcl::Window* pParent) - : ModalDialog(pParent, "PlaceEditDialog", "svt/ui/placeedit.ui") +PlaceEditDialog::PlaceEditDialog(weld::Window* pParent) + : GenericDialogController(pParent, "svt/ui/placeedit.ui", "PlaceEditDialog") , m_xCurrentDetails() , m_nCurrentType( 0 ) - , bLabelChanged( false ) + , m_bLabelChanged( false ) , m_bShowPassword( true ) + , m_xEDServerName(m_xBuilder->weld_entry("name")) + , m_xLBServerType(m_xBuilder->weld_combo_box_text("type")) + , m_xEDUsername(m_xBuilder->weld_entry("login")) + , m_xFTUsernameLabel(m_xBuilder->weld_label("loginLabel")) + , m_xBTOk(m_xBuilder->weld_button("ok")) + , m_xBTCancel(m_xBuilder->weld_button("cancel")) + , m_xBTDelete(m_xBuilder->weld_button("delete")) + , m_xBTRepoRefresh(m_xBuilder->weld_button("repositoriesRefresh")) + , m_xCBPassword(m_xBuilder->weld_check_button("rememberPassword")) + , m_xEDPassword(m_xBuilder->weld_entry("password")) + , m_xFTPasswordLabel(m_xBuilder->weld_label("passwordLabel")) + , m_xTypeGrid(m_xBuilder->weld_widget("TypeGrid")) { - get( m_pEDServerName, "name" ); - get( m_pLBServerType, "type" ); - get( m_pEDUsername, "login" ); - get( m_pFTUsernameLabel, "loginLabel" ); - get( m_pBTOk, "ok" ); - get( m_pBTCancel, "cancel" ); - get( m_pBTDelete, "delete" ); - get( m_pBTRepoRefresh, "repositoriesRefresh" ); - get( m_pCBPassword, "rememberPassword" ); - get( m_pEDPassword, "password" ); - get( m_pFTPasswordLabel, "passwordLabel" ); - - m_pBTOk->SetClickHdl( LINK( this, PlaceEditDialog, OKHdl) ); - m_pBTOk->Enable( false ); - - m_pEDServerName->SetModifyHdl( LINK( this, PlaceEditDialog, EditLabelHdl) ); + m_xBTOk->connect_clicked( LINK( this, PlaceEditDialog, OKHdl) ); + m_xBTOk->set_sensitive( false ); + + m_xEDServerName->connect_changed( LINK( this, PlaceEditDialog, EditLabelHdl) ); // This constructor is called when user request a place creation, so // delete button is hidden. - m_pBTDelete->Hide(); + m_xBTDelete->hide(); - m_pLBServerType->SetSelectHdl( LINK( this, PlaceEditDialog, SelectTypeHdl ) ); - m_pEDUsername->SetModifyHdl( LINK( this, PlaceEditDialog, EditUsernameHdl ) ); - m_pEDPassword->SetModifyHdl( LINK( this, PlaceEditDialog, EditUsernameHdl ) ); + m_xLBServerType->connect_changed( LINK( this, PlaceEditDialog, SelectTypeHdl ) ); + m_xEDUsername->connect_changed( LINK( this, PlaceEditDialog, EditUsernameHdl ) ); + m_xEDPassword->connect_changed( LINK( this, PlaceEditDialog, EditUsernameHdl ) ); InitDetails( ); } -PlaceEditDialog::PlaceEditDialog(vcl::Window* pParent, const std::shared_ptr& rPlace) - : ModalDialog(pParent, "PlaceEditDialog", "svt/ui/placeedit.ui") +PlaceEditDialog::PlaceEditDialog(weld::Window* pParent, const std::shared_ptr& rPlace) + : GenericDialogController(pParent, "svt/ui/placeedit.ui", "PlaceEditDialog") , m_xCurrentDetails( ) - , bLabelChanged( true ) + , m_bLabelChanged( true ) , m_bShowPassword( false ) + , m_xEDServerName(m_xBuilder->weld_entry("name")) + , m_xLBServerType(m_xBuilder->weld_combo_box_text("type")) + , m_xEDUsername(m_xBuilder->weld_entry("login")) + , m_xFTUsernameLabel(m_xBuilder->weld_label("loginLabel")) + , m_xBTOk(m_xBuilder->weld_button("ok")) + , m_xBTCancel(m_xBuilder->weld_button("cancel")) + , m_xBTDelete(m_xBuilder->weld_button("delete")) + , m_xBTRepoRefresh(m_xBuilder->weld_button("repositoriesRefresh")) + , m_xCBPassword(m_xBuilder->weld_check_button("rememberPassword")) + , m_xEDPassword(m_xBuilder->weld_entry("password")) + , m_xFTPasswordLabel(m_xBuilder->weld_label("passwordLabel")) + , m_xTypeGrid(m_xBuilder->weld_widget("TypeGrid")) { - get( m_pEDServerName, "name" ); - get( m_pLBServerType, "type" ); - get( m_pEDUsername, "login" ); - get( m_pFTUsernameLabel, "loginLabel" ); - get( m_pBTOk, "ok" ); - get( m_pBTCancel, "cancel" ); - get( m_pBTDelete, "delete" ); - get( m_pBTRepoRefresh, "repositoriesRefresh" ); - get( m_pTypeGrid, "TypeGrid" ); - get( m_pCBPassword, "rememberPassword" ); - get( m_pEDPassword, "password" ); - get( m_pFTPasswordLabel, "passwordLabel" ); - - m_pEDPassword->Hide(); - m_pFTPasswordLabel->Hide(); - m_pCBPassword->Hide(); - - m_pBTOk->SetClickHdl( LINK( this, PlaceEditDialog, OKHdl) ); - m_pBTDelete->SetClickHdl ( LINK( this, PlaceEditDialog, DelHdl) ); - - m_pEDServerName->SetModifyHdl( LINK( this, PlaceEditDialog, ModifyHdl) ); - m_pLBServerType->SetSelectHdl( LINK( this, PlaceEditDialog, SelectTypeHdl ) ); + m_xEDPassword->hide(); + m_xFTPasswordLabel->hide(); + m_xCBPassword->hide(); + + m_xBTOk->connect_clicked( LINK( this, PlaceEditDialog, OKHdl) ); + m_xBTDelete->connect_clicked( LINK( this, PlaceEditDialog, DelHdl) ); + + m_xEDServerName->connect_changed( LINK( this, PlaceEditDialog, ModifyHdl) ); + m_xLBServerType->connect_changed( LINK( this, PlaceEditDialog, SelectTypeHdl ) ); InitDetails( ); - m_pEDServerName->SetText(rPlace->GetName()); + m_xEDServerName->set_text(rPlace->GetName()); // Fill the boxes with the URL parts bool bSuccess = false; @@ -96,41 +95,23 @@ PlaceEditDialog::PlaceEditDialog(vcl::Window* pParent, const std::shared_ptrSetText( INetURLObject::decode( rUrl.GetUser( ), + m_xEDUsername->set_text( INetURLObject::decode( rUrl.GetUser( ), INetURLObject::DecodeMechanism::WithCharset ) ); m_aDetailsContainers[i]->setUsername( INetURLObject::decode( rUrl.GetUser( ), INetURLObject::DecodeMechanism::WithCharset ) ); } - m_pLBServerType->SelectEntryPos( i ); - SelectTypeHdl( *m_pLBServerType ); + m_xLBServerType->set_active(i); + SelectType(true); } } // In edit mode user can't change connection type - m_pTypeGrid->Hide(); + m_xTypeGrid->hide(); } PlaceEditDialog::~PlaceEditDialog() { - disposeOnce(); -} - -void PlaceEditDialog::dispose() -{ - m_pEDServerName.clear(); - m_pLBServerType.clear(); - m_pEDUsername.clear(); - m_pFTUsernameLabel.clear(); - m_pBTOk.clear(); - m_pBTCancel.clear(); - m_pBTDelete.clear(); - m_pEDPassword.clear(); - m_pFTPasswordLabel.clear(); - m_pCBPassword.clear(); - m_pBTRepoRefresh.clear(); - m_pTypeGrid.clear(); - ModalDialog::dispose(); } OUString PlaceEditDialog::GetServerUrl() @@ -139,7 +120,7 @@ OUString PlaceEditDialog::GetServerUrl() if (m_xCurrentDetails.get()) { INetURLObject aUrl = m_xCurrentDetails->getUrl(); - OUString sUsername = m_pEDUsername->GetText( ).trim( ); + OUString sUsername = m_xEDUsername->get_text().trim(); if ( !sUsername.isEmpty( ) ) aUrl.SetUser( sUsername ); if ( !aUrl.HasError( ) ) @@ -151,7 +132,7 @@ OUString PlaceEditDialog::GetServerUrl() std::shared_ptr PlaceEditDialog::GetPlace() { - return std::make_shared(m_pEDServerName->GetText(), GetServerUrl(), true); + return std::make_shared(m_xEDServerName->get_text(), GetServerUrl(), true); } void PlaceEditDialog::InitDetails( ) @@ -171,7 +152,7 @@ void PlaceEditDialog::InitDetails( ) Sequence< OUString > aTypesUrlsList( officecfg::Office::Common::Misc::CmisServersUrls::get( xContext ) ); Sequence< OUString > aTypesNamesList( officecfg::Office::Common::Misc::CmisServersNames::get( xContext ) ); - unsigned int nPos = 0; + int nPos = 0; for ( sal_Int32 i = 0; i < aTypesUrlsList.getLength( ) && aTypesNamesList.getLength( ); ++i ) { OUString sUrl = aTypesUrlsList[i].replaceFirst("", SvtResId(STR_SVT_PORT) + ">"); @@ -184,43 +165,43 @@ void PlaceEditDialog::InitDetails( ) continue; } - nPos = m_pLBServerType->InsertEntry( aTypesNamesList[i].replaceFirst("Other CMIS", SvtResId(STR_SVT_OTHER_CMIS)), nPos ); + m_xLBServerType->insert_text(nPos, aTypesNamesList[i].replaceFirst("Other CMIS", SvtResId(STR_SVT_OTHER_CMIS))); - std::shared_ptr xCmisDetails(std::make_shared(this, this, sUrl)); + std::shared_ptr xCmisDetails(std::make_shared(m_xBuilder.get(), m_xDialog.get(), sUrl)); xCmisDetails->setChangeHdl( LINK( this, PlaceEditDialog, EditHdl ) ); m_aDetailsContainers.push_back(xCmisDetails); - nPos++; + ++nPos; } // Create WebDAV / FTP / SSH details control - std::shared_ptr xDavDetails(std::make_shared(this)); + std::shared_ptr xDavDetails(std::make_shared(m_xBuilder.get())); xDavDetails->setChangeHdl( LINK( this, PlaceEditDialog, EditHdl ) ); m_aDetailsContainers.push_back(xDavDetails); - std::shared_ptr xFtpDetails(std::make_shared(this, 21, "ftp")); + std::shared_ptr xFtpDetails(std::make_shared(m_xBuilder.get(), 21, "ftp")); xFtpDetails->setChangeHdl( LINK( this, PlaceEditDialog, EditHdl ) ); m_aDetailsContainers.push_back(xFtpDetails); - std::shared_ptr xSshDetails(std::make_shared(this, 22, "ssh")); + std::shared_ptr xSshDetails(std::make_shared(m_xBuilder.get(), 22, "ssh")); xSshDetails->setChangeHdl( LINK( this, PlaceEditDialog, EditHdl ) ); m_aDetailsContainers.push_back(xSshDetails); // Create Windows Share control - std::shared_ptr xSmbDetails(std::make_shared(this)); + std::shared_ptr xSmbDetails(std::make_shared(m_xBuilder.get())); xSmbDetails->setChangeHdl( LINK( this, PlaceEditDialog, EditHdl ) ); m_aDetailsContainers.push_back(xSmbDetails); // Set default to first value - m_pLBServerType->SelectEntryPos( 0 ); + m_xLBServerType->set_active(0); - if ( m_pLBServerType->GetSelectedEntry() == "--------------------" ) - m_pLBServerType->SelectEntryPos( 1 ); + if (m_xLBServerType->get_active_text() == "--------------------" ) + m_xLBServerType->set_active(1); - SelectTypeHdl( *m_pLBServerType ); + SelectType(true); } -IMPL_LINK( PlaceEditDialog, OKHdl, Button*, /*pBtn*/, void) +IMPL_LINK( PlaceEditDialog, OKHdl, weld::Button&, /*rBtn*/, void) { if ( !m_xCurrentDetails.get() ) return; @@ -231,7 +212,7 @@ IMPL_LINK( PlaceEditDialog, OKHdl, Button*, /*pBtn*/, void) || sUrl.startsWith( ALFRESCO_CLOUD_BASE_URL ) || sUrl.startsWith( ONEDRIVE_BASE_URL ) ) { - m_pBTRepoRefresh->Click(); + m_xBTRepoRefresh->clicked(); sUrl = m_xCurrentDetails->getUrl().GetHost( INetURLObject::DecodeMechanism::WithCharset ); INetURLObject aHostUrl( sUrl ); @@ -239,7 +220,7 @@ IMPL_LINK( PlaceEditDialog, OKHdl, Button*, /*pBtn*/, void) if ( !sRepoId.isEmpty() ) { - EndDialog( RET_OK ); + m_xDialog->response(RET_OK); } else { @@ -248,97 +229,101 @@ IMPL_LINK( PlaceEditDialog, OKHdl, Button*, /*pBtn*/, void) } else { - EndDialog( RET_OK ); + m_xDialog->response(RET_OK); } } -IMPL_LINK( PlaceEditDialog, DelHdl, Button*, /*pButton*/, void) +IMPL_LINK( PlaceEditDialog, DelHdl, weld::Button&, /*rButton*/, void) { // ReUsing existing symbols... - EndDialog( RET_NO ); + m_xDialog->response(RET_NO); } IMPL_LINK_NOARG( PlaceEditDialog, EditHdl, DetailsContainer*, void ) { - if( !bLabelChanged ) + if( !m_bLabelChanged ) { - if( !m_pEDUsername->GetText().isEmpty( ) ) + if( !m_xEDUsername->get_text().isEmpty( ) ) { OUString sLabel = SvtResId( STR_SVT_DEFAULT_SERVICE_LABEL ); - OUString sUser = m_pEDUsername->GetText(); + OUString sUser = m_xEDUsername->get_text(); int nLength = sUser.indexOf( '@' ); if( nLength < 0 ) nLength = sUser.getLength(); sLabel = sLabel.replaceFirst( "$user$", sUser.copy( 0, nLength ) ); - sLabel = sLabel.replaceFirst( "$service$", m_pLBServerType->GetSelectedEntry() ); + sLabel = sLabel.replaceFirst( "$service$", m_xLBServerType->get_active_text() ); - m_pEDServerName->SetText( sLabel ); - bLabelChanged = false; + m_xEDServerName->set_text( sLabel ); + m_bLabelChanged = false; } else { - m_pEDServerName->SetText( m_pLBServerType->GetSelectedEntry( ) ); + m_xEDServerName->set_text( m_xLBServerType->get_active_text( ) ); } } OUString sUrl = GetServerUrl( ); - OUString sName = m_pEDServerName->GetText().trim( ); - m_pBTOk->Enable( !sName.isEmpty( ) && !sUrl.isEmpty( ) ); + OUString sName = m_xEDServerName->get_text().trim(); + m_xBTOk->set_sensitive( !sName.isEmpty( ) && !sUrl.isEmpty( ) ); } -IMPL_LINK_NOARG( PlaceEditDialog, ModifyHdl, Edit&, void ) +IMPL_LINK_NOARG( PlaceEditDialog, ModifyHdl, weld::Entry&, void ) { EditHdl(nullptr); } -IMPL_LINK_NOARG( PlaceEditDialog, EditLabelHdl, Edit&, void ) +IMPL_LINK_NOARG( PlaceEditDialog, EditLabelHdl, weld::Entry&, void ) { - bLabelChanged = true; + m_bLabelChanged = true; EditHdl(nullptr); } -IMPL_LINK_NOARG( PlaceEditDialog, EditUsernameHdl, Edit&, void ) +IMPL_LINK_NOARG( PlaceEditDialog, EditUsernameHdl, weld::Entry&, void ) { for ( std::vector< std::shared_ptr< DetailsContainer > >::iterator it = m_aDetailsContainers.begin( ); it != m_aDetailsContainers.end( ); ++it ) { - ( *it )->setUsername( m_pEDUsername->GetText() ); - ( *it )->setPassword( m_pEDPassword->GetText() ); + ( *it )->setUsername( m_xEDUsername->get_text() ); + ( *it )->setPassword( m_xEDPassword->get_text() ); } EditHdl(nullptr); } -IMPL_LINK_NOARG( PlaceEditDialog, SelectTypeHdl, ListBox&, void ) +IMPL_LINK_NOARG( PlaceEditDialog, SelectTypeHdl, weld::ComboBoxText&, void ) { - if ( m_pLBServerType->GetSelectedEntry() == "--------------------" ) + SelectType(false); +} + +void PlaceEditDialog::SelectType(bool bSkipSeparator) +{ + if ( m_xLBServerType->get_active_text() == "--------------------" ) { - if( !m_pLBServerType->IsTravelSelect() ) - m_pLBServerType->SelectEntryPos( m_nCurrentType ); + if (bSkipSeparator) + m_xLBServerType->set_active(m_nCurrentType); else - m_pLBServerType->SetNoSelection(); - + m_xLBServerType->set_active(-1); return; } if (m_xCurrentDetails.get()) m_xCurrentDetails->show(false); - const sal_Int32 nPos = m_pLBServerType->GetSelectedEntryPos( ); + const int nPos = m_xLBServerType->get_active( ); m_xCurrentDetails = m_aDetailsContainers[nPos]; m_nCurrentType = nPos; m_xCurrentDetails->show(); - m_pCBPassword->Show( m_bShowPassword && m_xCurrentDetails->enableUserCredentials() ); - m_pEDPassword->Show( m_bShowPassword && m_xCurrentDetails->enableUserCredentials() ); - m_pFTPasswordLabel->Show( m_bShowPassword && m_xCurrentDetails->enableUserCredentials() ); - m_pEDUsername->Show( m_xCurrentDetails->enableUserCredentials() ); - m_pFTUsernameLabel->Show( m_xCurrentDetails->enableUserCredentials() ); + m_xCBPassword->show( m_bShowPassword && m_xCurrentDetails->enableUserCredentials() ); + m_xEDPassword->show( m_bShowPassword && m_xCurrentDetails->enableUserCredentials() ); + m_xFTPasswordLabel->show( m_bShowPassword && m_xCurrentDetails->enableUserCredentials() ); + m_xEDUsername->show( m_xCurrentDetails->enableUserCredentials() ); + m_xFTUsernameLabel->show( m_xCurrentDetails->enableUserCredentials() ); - SetSizePixel(GetOptimalSize()); + m_xDialog->resize_to_request(); EditHdl(nullptr); } diff --git a/svtools/source/dialogs/ServerDetailsControls.cxx b/svtools/source/dialogs/ServerDetailsControls.cxx index 4cb45ae21915..827fd3ae1953 100644 --- a/svtools/source/dialogs/ServerDetailsControls.cxx +++ b/svtools/source/dialogs/ServerDetailsControls.cxx @@ -23,6 +23,8 @@ #include #include +#include + #include using namespace std; @@ -32,17 +34,23 @@ using namespace com::sun::star::task; using namespace com::sun::star::ucb; using namespace com::sun::star::uno; -DetailsContainer::DetailsContainer( VclBuilderContainer* pBuilder ) +DetailsContainer::DetailsContainer(weld::Builder* pBuilder) + : m_xDetailsGrid(pBuilder->weld_widget("Details")) + , m_xHostBox(pBuilder->weld_widget("HostDetails")) + , m_xEDHost(pBuilder->weld_entry("host")) + , m_xFTHost(pBuilder->weld_label("hostLabel")) + , m_xEDPort(pBuilder->weld_spin_button("port")) + , m_xFTPort(pBuilder->weld_label("portLabel")) + , m_xEDRoot(pBuilder->weld_entry("path")) + , m_xFTRoot(pBuilder->weld_label("pathLabel")) +{ + m_xEDPort->connect_output(LINK(this, DetailsContainer, FormatPortHdl)); +} + +//format without thousand separator +IMPL_LINK(DetailsContainer, FormatPortHdl, weld::SpinButton&, rSpinButton, void) { - pBuilder->get( m_pDetailsGrid, "Details" ); - pBuilder->get( m_pHostBox, "HostDetails" ); - pBuilder->get( m_pEDHost, "host" ); - pBuilder->get( m_pFTHost, "hostLabel" ); - pBuilder->get( m_pEDPort, "port-nospin" ); - pBuilder->get( m_pFTPort, "portLabel" ); - pBuilder->get( m_pEDRoot, "path" ); - pBuilder->get( m_pFTRoot, "pathLabel" ); - m_pEDPort->SetUseThousandSep( false ); + rSpinButton.set_text(OUString::number(rSpinButton.get_value())); } DetailsContainer::~DetailsContainer( ) @@ -51,11 +59,11 @@ DetailsContainer::~DetailsContainer( ) void DetailsContainer::show( bool ) { - m_pDetailsGrid->Enable(); + m_xDetailsGrid->set_sensitive(true); - m_pEDHost->SetModifyHdl( LINK( this, DetailsContainer, ValueChangeHdl ) ); - m_pEDPort->SetModifyHdl( LINK( this, DetailsContainer, ValueChangeHdl ) ); - m_pEDRoot->SetModifyHdl( LINK( this, DetailsContainer, ValueChangeHdl ) ); + m_xEDHost->connect_changed( LINK( this, DetailsContainer, ValueChangeHdl ) ); + m_xEDPort->connect_changed( LINK( this, DetailsContainer, ValueChangeHdl ) ); + m_xEDRoot->connect_changed( LINK( this, DetailsContainer, ValueChangeHdl ) ); } INetURLObject DetailsContainer::getUrl( ) @@ -76,12 +84,12 @@ void DetailsContainer::notifyChange( ) m_aChangeHdl.Call( this ); } -IMPL_LINK_NOARG( DetailsContainer, ValueChangeHdl, Edit&, void ) +IMPL_LINK_NOARG( DetailsContainer, ValueChangeHdl, weld::Entry&, void ) { notifyChange( ); } -HostDetailsContainer::HostDetailsContainer( VclBuilderContainer* pBuilder, sal_uInt16 nPort, const OUString& sScheme ) : +HostDetailsContainer::HostDetailsContainer( weld::Builder* pBuilder, sal_uInt16 nPort, const OUString& sScheme ) : DetailsContainer( pBuilder ), m_nDefaultPort( nPort ), m_sScheme( sScheme ) @@ -91,28 +99,28 @@ HostDetailsContainer::HostDetailsContainer( VclBuilderContainer* pBuilder, sal_u void HostDetailsContainer::show( bool bShow ) { - m_pFTHost->Show( bShow ); - m_pHostBox->Show( bShow ); - m_pEDRoot->Show( bShow ); - m_pFTRoot->Show( bShow ); + m_xFTHost->show( bShow ); + m_xHostBox->show( bShow ); + m_xEDRoot->show( bShow ); + m_xFTRoot->show( bShow ); DetailsContainer::show( bShow ); if ( bShow ) { - if ( m_pEDPort->GetValue( ) == 0 ) - m_pEDPort->SetValue( m_nDefaultPort ); - m_pEDHost->SetText( m_sHost ); + if ( m_xEDPort->get_value( ) == 0 ) + m_xEDPort->set_value( m_nDefaultPort ); + m_xEDHost->set_text( m_sHost ); } else - m_pEDPort->SetValue( 0 ); + m_xEDPort->set_value( 0 ); } INetURLObject HostDetailsContainer::getUrl( ) { - OUString sHost = m_pEDHost->GetText().trim( ); - sal_Int64 nPort = m_pEDPort->GetValue(); - OUString sPath = m_pEDRoot->GetText().trim( ); + OUString sHost = m_xEDHost->get_text().trim(); + sal_Int64 nPort = m_xEDPort->get_value(); + OUString sPath = m_xEDRoot->get_text().trim(); OUString sUrl; if ( !sHost.isEmpty( ) ) @@ -136,9 +144,9 @@ bool HostDetailsContainer::setUrl( const INetURLObject& rUrl ) if ( bSuccess ) { m_sHost = rUrl.GetHost( ); - m_pEDHost->SetText( rUrl.GetHost( ) ); - m_pEDPort->SetValue( rUrl.GetPort( ) ); - m_pEDRoot->SetText( rUrl.GetURLPath() ); + m_xEDHost->set_text( rUrl.GetHost( ) ); + m_xEDPort->set_value( rUrl.GetPort( ) ); + m_xEDRoot->set_text( rUrl.GetURLPath() ); } return bSuccess; @@ -149,11 +157,11 @@ bool HostDetailsContainer::verifyScheme( const OUString& sScheme ) return sScheme == ( m_sScheme + "://" ); } -DavDetailsContainer::DavDetailsContainer( VclBuilderContainer* pBuilder ) : - HostDetailsContainer( pBuilder, 80, "http" ) +DavDetailsContainer::DavDetailsContainer(weld::Builder* pBuilder) + : HostDetailsContainer(pBuilder, 80, "http") + , m_xCBDavs(pBuilder->weld_check_button("webdavs")) { - pBuilder->get( m_pCBDavs, "webdavs" ); - m_pCBDavs->SetToggleHdl( LINK( this, DavDetailsContainer, ToggledDavsHdl ) ); + m_xCBDavs->connect_toggled(LINK(this, DavDetailsContainer, ToggledDavsHdl)); show( false ); } @@ -163,9 +171,9 @@ void DavDetailsContainer::show( bool bShow ) HostDetailsContainer::show( bShow ); if ( !bShow ) - m_pCBDavs->Check( false ); + m_xCBDavs->set_active(false); - m_pCBDavs->Show( bShow ); + m_xCBDavs->show(bShow); } bool DavDetailsContainer::verifyScheme( const OUString& rScheme ) @@ -174,24 +182,24 @@ bool DavDetailsContainer::verifyScheme( const OUString& rScheme ) if ( rScheme == "http://" ) { bValid = true; - m_pCBDavs->Check( false ); + m_xCBDavs->set_active(false); } else if ( rScheme == "https://" ) { bValid = true; - m_pCBDavs->Check(); + m_xCBDavs->set_active(true); } return bValid; } -IMPL_LINK( DavDetailsContainer, ToggledDavsHdl, CheckBox&, rCheckBox, void ) +IMPL_LINK( DavDetailsContainer, ToggledDavsHdl, weld::ToggleButton&, rCheckBox, void ) { // Change default port if needed - bool bCheckedDavs = rCheckBox.IsChecked(); - if ( m_pEDPort->GetValue() == 80 && bCheckedDavs ) - m_pEDPort->SetValue( 443 ); - else if ( m_pEDPort->GetValue() == 443 && !bCheckedDavs ) - m_pEDPort->SetValue( 80 ); + bool bCheckedDavs = rCheckBox.get_active(); + if ( m_xEDPort->get_value() == 80 && bCheckedDavs ) + m_xEDPort->set_value( 443 ); + else if ( m_xEDPort->get_value() == 443 && !bCheckedDavs ) + m_xEDPort->set_value( 80 ); OUString sScheme( "http" ); if ( bCheckedDavs ) @@ -201,22 +209,21 @@ IMPL_LINK( DavDetailsContainer, ToggledDavsHdl, CheckBox&, rCheckBox, void ) notifyChange( ); } -SmbDetailsContainer::SmbDetailsContainer( VclBuilderContainer* pBuilder ) : - DetailsContainer( pBuilder ) +SmbDetailsContainer::SmbDetailsContainer(weld::Builder* pBuilder) + : DetailsContainer(pBuilder) + , m_xEDShare(pBuilder->weld_entry("share")) + , m_xFTShare(pBuilder->weld_label("shareLabel")) { - pBuilder->get( m_pEDShare, "share" ); - pBuilder->get( m_pFTShare, "shareLabel" ); - - m_pEDShare->SetModifyHdl( LINK( this, DetailsContainer, ValueChangeHdl ) ); + m_xEDShare->connect_changed( LINK( this, DetailsContainer, ValueChangeHdl ) ); show( false ); } INetURLObject SmbDetailsContainer::getUrl( ) { - OUString sHost = m_pEDHost->GetText().trim( ); - OUString sShare = m_pEDShare->GetText().trim( ); - OUString sPath = m_pEDRoot->GetText().trim( ); + OUString sHost = m_xEDHost->get_text().trim( ); + OUString sShare = m_xEDShare->get_text().trim( ); + OUString sPath = m_xEDRoot->get_text().trim( ); OUString sUrl; if ( !sHost.isEmpty( ) ) @@ -250,9 +257,9 @@ bool SmbDetailsContainer::setUrl( const INetURLObject& rUrl ) sPath = sFullPath.copy( nPos ); } - m_pEDHost->SetText( rUrl.GetHost( ) ); - m_pEDShare->SetText( sShare ); - m_pEDRoot->SetText( sPath ); + m_xEDHost->set_text( rUrl.GetHost( ) ); + m_xEDShare->set_text( sShare ); + m_xEDRoot->set_text( sPath ); } return bSuccess; @@ -260,76 +267,75 @@ bool SmbDetailsContainer::setUrl( const INetURLObject& rUrl ) void SmbDetailsContainer::show( bool bShow ) { - m_pEDShare->Show( bShow ); - m_pFTShare->Show( bShow ); - m_pEDRoot->Show( bShow ); - m_pFTRoot->Show( bShow ); - - m_pFTHost->Show( bShow ); - m_pHostBox->Show( bShow ); - m_pEDPort->Enable( !bShow ); - m_pFTPort->Enable( !bShow ); + m_xEDShare->show( bShow ); + m_xFTShare->show( bShow ); + m_xEDRoot->show( bShow ); + m_xFTRoot->show( bShow ); + + m_xFTHost->show( bShow ); + m_xHostBox->show( bShow ); + m_xEDPort->set_sensitive( !bShow ); + m_xFTPort->set_sensitive( !bShow ); } -CmisDetailsContainer::CmisDetailsContainer(VclBuilderContainer* pBuilder, Dialog* pParentDialog, OUString const & sBinding) : +CmisDetailsContainer::CmisDetailsContainer(weld::Builder* pBuilder, weld::Dialog* pParentDialog, OUString const & sBinding) : DetailsContainer( pBuilder ), m_sUsername( ), m_xCmdEnv( ), m_aRepoIds( ), m_sRepoId( ), m_sBinding( sBinding ), - m_xParentDialog( VCLUnoHelper::GetInterface(pParentDialog) ) + m_xParentDialog(pParentDialog->GetXWindow()), + m_xRepositoryBox(pBuilder->weld_widget("RepositoryDetails")), + m_xFTRepository(pBuilder->weld_label("repositoryLabel")), + m_xLBRepository(pBuilder->weld_combo_box_text("repositories")), + m_xBTRepoRefresh(pBuilder->weld_button("repositoriesRefresh")) { Reference< XComponentContext > xContext = ::comphelper::getProcessComponentContext(); Reference< XInteractionHandler > xGlobalInteractionHandler( InteractionHandler::createWithParent(xContext, m_xParentDialog), UNO_QUERY); m_xCmdEnv = new ucbhelper::CommandEnvironment( xGlobalInteractionHandler, Reference< XProgressHandler >() ); - pBuilder->get( m_pFTRepository, "repositoryLabel" ); - pBuilder->get( m_pLBRepository, "repositories" ); - pBuilder->get( m_pBTRepoRefresh, "repositoriesRefresh" ); - pBuilder->get( m_pRepositoryBox, "RepositoryDetails" ); - show( false ); } void CmisDetailsContainer::show( bool bShow ) { - m_pLBRepository->SetSelectHdl( LINK( this, CmisDetailsContainer, SelectRepoHdl ) ); - m_pBTRepoRefresh->SetClickHdl( LINK( this, CmisDetailsContainer, RefreshReposHdl ) ); + m_xLBRepository->connect_changed( LINK( this, CmisDetailsContainer, SelectRepoHdl ) ); + m_xBTRepoRefresh->connect_clicked( LINK( this, CmisDetailsContainer, RefreshReposHdl ) ); - m_pEDHost->SetText( m_sBinding ); + m_xEDHost->set_text( m_sBinding ); if( ( m_sBinding == GDRIVE_BASE_URL ) || m_sBinding.startsWith( ALFRESCO_CLOUD_BASE_URL ) || ( m_sBinding == ONEDRIVE_BASE_URL ) ) { - m_pFTHost->Show( false ); - m_pHostBox->Show( false ); - m_pFTRepository->Show( false ); - m_pRepositoryBox->Show( false ); - m_pEDRoot->Show( false ); - m_pFTRoot->Show( false ); + m_xFTHost->show( false ); + m_xHostBox->show( false ); + m_xFTRepository->show( false ); + m_xRepositoryBox->show( false ); + m_xEDRoot->show( false ); + m_xFTRoot->show( false ); } else { - m_pFTHost->Show( bShow ); - m_pHostBox->Show( bShow ); - m_pFTRepository->Show( bShow ); - m_pRepositoryBox->Show( bShow ); - m_pEDRoot->Show( bShow ); - m_pFTRoot->Show( bShow ); + m_xFTHost->show( bShow ); + m_xHostBox->show( bShow ); + m_xFTRepository->show( bShow ); + m_xRepositoryBox->show( bShow ); + m_xEDRoot->show( bShow ); + m_xFTRoot->show( bShow ); } DetailsContainer::show( bShow ); - m_pEDPort->Enable( !bShow ); - m_pFTPort->Enable( !bShow ); + m_xEDPort->set_sensitive( !bShow ); + m_xFTPort->set_sensitive( !bShow ); } INetURLObject CmisDetailsContainer::getUrl( ) { - OUString sBindingUrl = m_pEDHost->GetText().trim( ); - OUString sPath = m_pEDRoot->GetText().trim( ); + OUString sBindingUrl = m_xEDHost->get_text().trim(); + OUString sPath = m_xEDRoot->get_text().trim(); bool bSkip = true; if( ( m_sBinding == GDRIVE_BASE_URL ) @@ -369,8 +375,8 @@ bool CmisDetailsContainer::setUrl( const INetURLObject& rUrl ) m_sBinding = aHostUrl.GetURLNoMark( ); m_sRepoId = aHostUrl.GetMark( ); - m_pEDHost->SetText( m_sBinding ); - m_pEDRoot->SetText( rUrl.GetURLPath() ); + m_xEDHost->set_text( m_sBinding ); + m_xEDRoot->set_text( rUrl.GetURLPath() ); } return bSuccess; } @@ -388,7 +394,7 @@ void CmisDetailsContainer::setPassword( const OUString& rPass ) void CmisDetailsContainer::selectRepository( ) { // Get the repo ID and call the Change listener - const sal_Int32 nPos = m_pLBRepository->GetSelectedEntryPos( ); + const int nPos = m_xLBRepository->get_active(); if( static_cast(nPos) < m_aRepoIds.size() ) { m_sRepoId = m_aRepoIds[nPos]; @@ -396,13 +402,13 @@ void CmisDetailsContainer::selectRepository( ) } } -IMPL_LINK_NOARG( CmisDetailsContainer, RefreshReposHdl, Button*, void ) +IMPL_LINK_NOARG( CmisDetailsContainer, RefreshReposHdl, weld::Button&, void ) { Reference< XComponentContext > xContext = ::comphelper::getProcessComponentContext(); Reference< XPasswordContainer2 > xMasterPasswd = PasswordContainer::create( xContext ); - OUString sBindingUrl = m_pEDHost->GetText().trim( ); + OUString sBindingUrl = m_xEDHost->get_text().trim( ); OUString sEncodedUsername = ""; @@ -416,8 +422,8 @@ IMPL_LINK_NOARG( CmisDetailsContainer, RefreshReposHdl, Button*, void ) } // Clean the listbox - m_pLBRepository->Clear( ); - m_aRepoIds.clear( ); + m_xLBRepository->clear(); + m_aRepoIds.clear(); // Compute the URL OUString sUrl; @@ -467,7 +473,7 @@ IMPL_LINK_NOARG( CmisDetailsContainer, RefreshReposHdl, Button*, void ) Reference< XRow > xRow( xResultSet, UNO_QUERY ); OUString sName = xRow->getString( 1 ); - m_pLBRepository->InsertEntry( sName ); + m_xLBRepository->append_text(sName); } } catch ( const Exception& ) @@ -475,9 +481,9 @@ IMPL_LINK_NOARG( CmisDetailsContainer, RefreshReposHdl, Button*, void ) } // Auto-select the first one - if ( m_pLBRepository->GetEntryCount( ) > 0 ) + if (m_xLBRepository->get_count() > 0) { - m_pLBRepository->SelectEntryPos( 0 ); + m_xLBRepository->set_active(0); selectRepository( ); } @@ -490,7 +496,7 @@ IMPL_LINK_NOARG( CmisDetailsContainer, RefreshReposHdl, Button*, void ) {} } -IMPL_LINK_NOARG( CmisDetailsContainer, SelectRepoHdl, ListBox&, void ) +IMPL_LINK_NOARG( CmisDetailsContainer, SelectRepoHdl, weld::ComboBoxText&, void ) { selectRepository( ); } diff --git a/svtools/uiconfig/ui/placeedit.ui b/svtools/uiconfig/ui/placeedit.ui index 13261dcc5bf8..8d211ef2352d 100644 --- a/svtools/uiconfig/ui/placeedit.ui +++ b/svtools/uiconfig/ui/placeedit.ui @@ -1,5 +1,5 @@ - + @@ -12,31 +12,13 @@ False gtk-refresh - - - - - - - - WebDAV - - - FTP - - - SSH - - - Windows Share - - - - 400 False 6 File Services + True + 0 + 0 dialog @@ -126,10 +108,10 @@ True False - 1 Type: True type + 1 0 @@ -137,17 +119,16 @@ - + True False True - liststore1 - - - - 0 - - + + WebDAV + FTP + SSH + Windows Share + 1 @@ -171,10 +152,10 @@ True False - 0 Host: True host + 0 0 @@ -185,10 +166,10 @@ True False - 0 Root: True path + 0 0 @@ -200,6 +181,7 @@ True True True + True 1 @@ -210,10 +192,10 @@ True False - 0 Share: True share + 0 0 @@ -225,6 +207,7 @@ True True True + True 1 @@ -235,10 +218,10 @@ True False - 0 Repository: True repositories + 0 0 @@ -263,10 +246,10 @@ True False - 0 User: True login + 0 0 @@ -278,6 +261,7 @@ True True True + True 1 @@ -288,10 +272,10 @@ True False - 0 Label: True name + 0 0 @@ -303,6 +287,7 @@ True True True + True 1 @@ -319,6 +304,7 @@ True True True + True True @@ -330,10 +316,10 @@ True False - 0 Port: True - port-nospin + port + 0 False @@ -342,13 +328,11 @@ - - 50 + True True - 5 - 5 - 0 + True + 0 adjustment1 True @@ -370,7 +354,7 @@ False 6 - + True False True @@ -404,9 +388,9 @@ True False - 0 Password: password + 0 0 @@ -418,6 +402,7 @@ True True False + True 1 @@ -454,10 +439,13 @@ + help delete ok cancel - help + + + -- cgit v1.2.3