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

List:       kopete-devel
Subject:    [kopete-devel] [Bug 140959] setGlobalPhoto urlencodes dir with space
From:       Will Stephenson <wstephenson () kde ! org>
Date:       2007-03-28 13:48:36
Message-ID: 20070328134836.20229.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=140959         
wstephenson kde org changed:

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



------- Additional Comments From wstephenson kde org  2007-03-28 15:48 -------
SVN commit 647486 by wstephens:

Don't feed the encoded URL to the KURLRequester; QImage can't read it
BUG: 140959


 M  +3 -1      kopeteidentityconfig.cpp  


--- branches/work/kopete/avdevice_overhaul/kopete/config/identity/kopeteidentityconfig.cpp #647485:647486
 @ -323,7 +323,7  @
 		}
 	}
 
-	d->m_view->comboPhotoURL->setURL(d->currentIdentity->customPhoto().url());
+	d->m_view->comboPhotoURL->setURL(d->currentIdentity->customPhoto().pathOrURL());
 	Kopete::MetaContact::PropertySource photoSource = d->currentIdentity->photoSource();
 
 	d->m_view->radioPhotoCustom->setChecked(photoSource == Kopete::MetaContact::SourceCustom);
 @ -378,7 +378,9  @
 			photo = Kopete::photoFromContact(selectedNameSourceContact());
 			break;
 		case Kopete::MetaContact::SourceCustom:
+			QString url = d->m_view->comboPhotoURL->url();
 			photo = QImage(d->m_view->comboPhotoURL->url());
+			bool null = photo.isNull();
 			break;
 	}
_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic