Bug 31528

Summary:	content roaming bugs
Product:	papyon	Reporter:	Riccardo (c10ud) <c10ud.dev>
Component:	general	Assignee:	papyon-bugs
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Riccardo (c10ud) 2010-11-10 09:59:37 UTC

[18:44] <c10ud> this account dies with 500 when trying to update the dp: http://emesene.pastebin.com/kYiCHErH
[18:44] <c10ud> (full error included)

[18:47] <c10ud> This one, 400 Bad Request when calling upd profile: http://emesene.pastebin.com/kcxbXU06

[18:53] <c10ud> third account: http://emesene.pastebin.com/zhepntX3
[18:53] <c10ud> 500, itemdoesnotexists

Comment 1 Riccardo (c10ud) 2010-11-10 12:04:51 UTC

diff --git a/papyon/service/ContentRoaming/scenario/store_profile.py b/papyon/se
index c4c64a7..8f5cb69 100644
--- a/papyon/service/ContentRoaming/scenario/store_profile.py
+++ b/papyon/service/ContentRoaming/scenario/store_profile.py
@@ -57,7 +57,7 @@ class StoreProfileScenario(BaseScenario):
                                      0)
 
     def __update_profile_callback(self):
-        if self.display_picture is not None:
+        if self.__display_picture_id is not None:
             self.__storage.DeleteRelationships(
                 (self.__delete_relationship_profile_callback,),
                 (self.__store_profile_errback,),


fixes 
[20:55] <lfrb> ResourceID = None..
[20:56] <lfrb> that because papyon is trying to delete a relationship that doesn't exist

Comment 2 Riccardo (c10ud) 2010-11-11 08:17:48 UTC

--- a/papyon/service/ContentRoaming/scenario/store_profile.py
+++ b/papyon/service/ContentRoaming/scenario/store_profile.py
@@ -57,10 +57,11 @@ class StoreProfileScenario(BaseScenario):
                                      0)
 
     def __update_profile_callback(self):
-        if self.display_picture is not None:
+        if self.__display_picture_id is not None and \
+           self.display_picture is not None:
             self.__storage.DeleteRelationships(
                 (self.__delete_relationship_profile_callback,),


this may actually be better, since it allows nick&pm update even when displaypic is faulty

Comment 3 Louis-Francis Ratté-Boulianne 2011-08-14 07:08:23 UTC

Fixed in papyon 0.5.3

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.