[prev in list] [next in list] [prev in thread] [next in thread] 

List:       konq-bugs
Subject:    [Bug 80619] not able to set proxy port other than 8080
From:       Dawit Alemayehu <adawit () kde ! org>
Date:       2004-05-07 7:05:45
Message-ID: 20040507070545.22707.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
      
http://bugs.kde.org/show_bug.cgi?id=80619      
adawit kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From adawit kde org  2004-05-07 09:05 -------
CVS commit by adawit: 

- Make the proxy setup a little bit fragile. Fixes BR# 80619.

CCMAIL: 80619-done bugs kde org


  M +6 -3      kmanualproxydlg.cpp   1.23.2.2


--- kdebase/kcontrol/kio/kmanualproxydlg.cpp  #1.23.2.1:1.23.2.2
 @ -271,5 +271,6  @ bool KManualProxyDlg::validate()
     KURL url = dlg->leHttp->text();
 
-    if (url.isValid() || KURIFilter::self()->filterURI (url, filters))
+    if ( (url.isValid() || KURIFilter::self()->filterURI (url, filters)) &&
+         url.hasHost() )
     {
         dlg->leHttp->setText (url.url());
 @ -287,5 +288,6  @ bool KManualProxyDlg::validate()
         url = dlg->leHttps->text();
 
-        if (url.isValid() || KURIFilter::self()->filterURI (url, filters))
+        if ( (url.isValid() || KURIFilter::self()->filterURI (url, filters)) &&
+             url.hasHost() )
         {
             dlg->leHttps->setText (url.url());
 @ -301,5 +303,6  @ bool KManualProxyDlg::validate()
         url = dlg->leFtp->text();
 
-        if (url.isValid() || KURIFilter::self()->filterURI (url, filters))
+        if ( (url.isValid() || KURIFilter::self()->filterURI (url, filters)) &&
+             url.hasHost() )
         {
             dlg->leFtp->setText (url.url());
_______________________________________________
Konq-bugs mailing list
Konq-bugs@mail.kde.org
https://mail.kde.org/mailman/listinfo/konq-bugs
[prev in list] [next in list] [prev in thread] [next in thread]