Move setting the previous pastebin content to after the upload in do_pastebin_xmlrpc().

eikehein · eikehein · commit d804cc966311 · 2012-05-28T16:16:35.000+02:00
This is in line with how I did it in do_pastebin_helper(),
and makes overall more sense. With the old flow, you could
get a dupe error just when retrying after the last upload
failed due to e.g. a network error.
diff --git a/bpython/repl.py b/bpython/repl.py
@@ -797,8 +797,6 @@ def do_pastebin_xmlrpc(self, s):
                                  (self.config.pastebin_url, str(e)))
             return
 
-        self.prev_pastebin_content = s
-
         self.interact.notify('Posting data to pastebin...')
         try:
             paste_id = pasteservice.pastes.newPaste('pycon', s, '', '', '',
@@ -807,6 +805,8 @@ def do_pastebin_xmlrpc(self, s):
             self.interact.notify('Upload failed: %s' % (str(e), ) )
             return
 
+        self.prev_pastebin_content = s
+
         paste_url_template = Template(self.config.pastebin_show_url)
         paste_id = urlquote(paste_id)
         paste_url = paste_url_template.safe_substitute(paste_id=paste_id)
