Bug 33332 - Preview image not set during file transfer
Summary: Preview image not set during file transfer
Status: RESOLVED FIXED
Alias: None
Product: papyon
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
Hardware: All All
: medium normal
Assignee: Louis-Francis Ratté-Boulianne
QA Contact: Louis-Francis Ratté-Boulianne
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-21 05:47 UTC by Andrea Stagi
Modified: 2011-03-30 14:59 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Modification in papyon.msnp2p.filetransfer.py to show preview (3.08 KB, patch)
2011-01-23 18:22 UTC, Andrea Stagi
Details | Splinter Review
Added control whether preview is present in papyon.msnp2p.filetransfer.py to show preview (3.12 KB, patch)
2011-01-24 05:21 UTC, Andrea Stagi
Details | Splinter Review
Added preview sending (3.19 KB, patch)
2011-01-30 19:02 UTC, Andrea Stagi
Details | Splinter Review
Preview fix in patch format (1.20 KB, patch)
2011-02-21 16:46 UTC, Andrea Stagi
Details | Splinter Review

Description Andrea Stagi 2011-01-21 05:47:17 UTC
Hi everybody, I'm working on Emesene 2 using Papyon libs. I've found a bug regarding preview image during file transfers...Looking at the Papyon code I've found the problem in msnp2p.filetransfer.py. In this file the class FileTransferSession has this method to fetch the context

    def parse_context(self, context):
        try:
            info = struct.unpack("<5I", context[0:20])
            self._size = info[2]
            self._has_preview = not bool(info[4])
            self._filename = unicode(context[20:570], "utf-16-le").rstrip("\x00")
        except:
            raise FTParseError(context)

but preview attribute is not set, and remains constantly None, even if has_preview attribute is True..

Bests
Comment 1 Andrea Stagi 2011-01-23 18:22:24 UTC
Created attachment 42352 [details] [review]
Modification in papyon.msnp2p.filetransfer.py to show preview

Hi all, I've modified file papyon.msnp2p.filetransfer.py to show previews...This patch has been tested successfully in Emesene 2 development. Please, modifiy this file...Regards..
Comment 2 Andrea Stagi 2011-01-24 05:21:07 UTC
Created attachment 42369 [details] [review]
Added control whether preview is present in papyon.msnp2p.filetransfer.py to show preview

Hi all, I missed to control if preview is present before catch it...Bests
Comment 3 Andrea Stagi 2011-01-30 19:02:18 UTC
Created attachment 42731 [details] [review]
Added preview sending

Sending and receiving preview now works well :)
Comment 4 Andrea Stagi 2011-02-21 16:46:27 UTC
Created attachment 43628 [details] [review]
Preview fix in patch format

I've transformed all my modifications in a patch format...


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.