Yeah I know, I didn't change its original code radically. Should have done
it though..

---
Akbar Gumbira
Sent from mobile
On Mar 16, 2014 11:59 PM, "Tim Sutton" <notifications@github.com> wrote:

> In safe_qgis/utilities/file_downloader.py:
>
> > +        :type progress_dialog: QWidget
> > +
> > +        """
> > +        self.manager = manager
> > +        self.url = url
> > +        self.output_path = output_path
> > +        self.progress_dialog = progress_dialog
> > +        self.output_file = None
> > +        self.reply = None
> > +        self.downloaded_file_buffer = None
> > +        self.finished_flag = False
> > +
> > +    def download(self):
> > +        """Downloading the file.
> > +
> > +        :returns: True if success, otherwise returns a tuple with format like
>
> Won't
>
> if foo.download():
>     # blah blah blah
>
> always evaluate to true in this case? Seems like the API doesnt follow the
> contract it implies then...
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/AIFDR/inasafe/pull/886/files#r10640321>
> .
>