Merge lp:~enli/lernid/async-slide-download into lp:lernid

Proposed by Peeyoosh Sangolekar
Status: Merged
Merge reported by: Michael Budde
Merged at revision: not available
Proposed branch: lp:~enli/lernid/async-slide-download
Merge into: lp:lernid
Diff against target: 17 lines (+5/-2)
1 file modified
lernid/widgets/Slide.py (+5/-2)
To merge this branch: bzr merge lp:~enli/lernid/async-slide-download
Reviewer Review Type Date Requested Status
Michael Budde Pending
Review via email: mp+20823@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lernid/widgets/Slide.py'
--- lernid/widgets/Slide.py 2010-03-04 12:40:09 +0000
+++ lernid/widgets/Slide.py 2010-03-06 16:05:28 +0000
@@ -143,8 +143,11 @@
143 path = os.path.join(save_cache_path('lernid'), 'slides.pdf')143 path = os.path.join(save_cache_path('lernid'), 'slides.pdf')
144 try:144 try:
145 logging.debug('downloading slides from %s' % session.slides)145 logging.debug('downloading slides from %s' % session.slides)
146 Statusbar.push_message(_('Downloading session slides...'), 'slidesession')146 def reporthook(blocks, bs, size):
147 urllib.urlretrieve(session.slides, path)147 Statusbar.pop_message('slidesession')
148 msg = _('Downloading session slides ({0} % of {1:.1} MB)...').format(100*blocks*bs/size, float(size)/float(1024*1024))
149 Statusbar.push_message(msg, 'slidesession')
150 urllib.urlretrieve(session.slides, path, reporthook)
148 Statusbar.push_message(_('Slides have been downloaded'), 'slidesession')151 Statusbar.push_message(_('Slides have been downloaded'), 'slidesession')
149 self._session_slide_downloaded = True152 self._session_slide_downloaded = True
150 self.emit('slides-downloaded')153 self.emit('slides-downloaded')

Subscribers

People subscribed via source and target branches