Related tdf#96833: fix behaviour of 'Remote Files' edit dialog.

Fixed:
- TCP port got lost/reset when editing a server;
- in Gtk widget disabled comma;
- secure protocol checkbox always unchecked when editing a
  server.

Change-Id: I358fc7375552b29e4840a3616e4e47b95b88ed2c
Reviewed-on: https://gerrit.libreoffice.org/21039Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Tested-by: Jan-Marek Glogowski <glogow@fbihome.de>

svtools/source/dialogs/ServerDetailsControls.cxx

@@ -42,6 +42,7 @@ DetailsContainer::DetailsContainer( VclBuilderContainer* pBuilder ) :
     pBuilder->get( m_pFTPort, "portLabel" );
     pBuilder->get( m_pEDRoot, "path" );
     pBuilder->get( m_pFTRoot, "pathLabel" );
+    m_pEDPort->SetUseThousandSep( false );
 }
 
 DetailsContainer::~DetailsContainer( )
@@ -104,9 +105,12 @@ void HostDetailsContainer::show( bool bShow )
 
     if ( bShow )
     {
+        if ( m_pEDPort->GetValue( ) == 0 )
             m_pEDPort->SetValue( m_nDefaultPort );
         m_pEDHost->SetText( m_sHost );
     }
+    else
+        m_pEDPort->SetValue( 0 );
 }
 
 INetURLObject HostDetailsContainer::getUrl( )
@@ -163,10 +167,10 @@ void DavDetailsContainer::show( bool bShow )
 {
     HostDetailsContainer::show( bShow );
 
-    m_pCBDavs->Show( bShow );
-
-    if ( bShow )
+    if ( !bShow )
         m_pCBDavs->Check( false );
+
+    m_pCBDavs->Show( bShow );
 }
 
 bool DavDetailsContainer::verifyScheme( const OUString& rScheme )