FTP upload not working

Bug #585658 reported by Julien Marchand
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Lookit
Fix Released
Undecided
Unassigned

Bug Description

The FTP upload does not work due to two little things :

Traceback (most recent call last):
  File "/usr/bin/lookit", line 139, in capture_screen
    self.process_pixbuf(pb)
  File "/usr/bin/lookit", line 147, in process_pixbuf
    self.upload_image(hashstring)
  File "/usr/bin/lookit", line 169, in upload_image
    self.prefs['url'],
  File "/usr/lib/pymodules/python2.6/lookit/uploader.py", line 78, in upload_file_ftp
    f = ftplib.FTP()
NameError: global name 'ftplib' is not defined

(ftplib is not imported)

Traceback (most recent call last):
  File "/usr/bin/lookit", line 139, in capture_screen
    self.process_pixbuf(pb)
  File "/usr/bin/lookit", line 147, in process_pixbuf
    self.upload_image(hashstring)
  File "/usr/bin/lookit", line 169, in upload_image
    self.prefs['url'],
  File "/usr/lib/pymodules/python2.6/lookit/uploader.py", line 86, in upload_file_ftp
    print f.storbinary('STOR ' + f, i)
TypeError: cannot concatenate 'str' and 'instance' objects

(f is the filename of the file to upload, and is later owerwritten as an instance of ftplib.FTP())

A diff that fixes the problem is attached.

Revision history for this message
Julien Marchand (jmarchan) wrote :
Revision history for this message
Zach Tibbitts (zachtib) wrote :

thanks! I need to start reviewing my source after I come down off my coffee binges...

Revision history for this message
Julien Marchand (jmarchan) wrote :

You're welcome ! Nice software :)

Zach Tibbitts (zachtib)
Changed in lookit:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.