Fix unicode and printing for python 3
authorMagnus Hagander <magnus@hagander.net>
Fri, 28 Jun 2019 12:57:20 +0000 (14:57 +0200)
committerMagnus Hagander <magnus@hagander.net>
Fri, 28 Jun 2019 14:28:15 +0000 (16:28 +0200)
hamnadmin/hamnadmin/mailqueue/models.py
hamnadmin/hamnadmin/register/feeds.py
hamnadmin/hamnadmin/register/management/commands/aggregate_feeds.py
hamnadmin/hamnadmin/register/management/commands/send_logs.py
hamnadmin/hamnadmin/register/models.py
hamnadmin/hamnadmin/register/views.py
hamnadmin/hamnadmin/util/varnish.py
posttotwitter.py
setuptwitter.py
synctwitter.py
twitterclient.py

index f2e7d198e2bb6dce56eb24d0d7461cec6c3b5ae1..f0cc442278bda875bd6be2190bff2c6775acd230 100644 (file)
@@ -7,5 +7,5 @@ class QueuedMail(models.Model):
        # anything, we just push them right in there!
        fullmsg = models.TextField(null=False, blank=False)
 
-       def __unicode__(self):
+       def __str__(self):
                return "%s: %s -> %s" % (self.pk, self.sender, self.receiver)
index 83e4c3c8ecd38ab2af0deb65e8c5bf606a4acaa9..0626e4d4c4de25ba057db3d61c892baadbd07ebc 100644 (file)
@@ -21,7 +21,7 @@ class PostFeed(Feed):
                return qs[:30]
 
        def item_title(self, item):
-               return u"{0}: {1}".format(item.feed.name, item.title)
+               return "{0}: {1}".format(item.feed.name, item.title)
 
        def item_link(self, item):
                if not item.shortlink:
index ffb54404cffae4d5be7f841fcb79425e87b3725f..ae4f77540633f7dbe51fc4b89dff4b4e8ebb9976 100644 (file)
@@ -77,14 +77,14 @@ class Command(BaseCommand):
                                                        send_simple_mail(settings.EMAIL_SENDER,
                                                                                         feed.user.email,
                                                                                         "Your blog at Planet PostgreSQL redirected",
-                                                                                        u"The blog aggregator at Planet PostgreSQL has picked up a redirect for your blog.\nOld URL: {0}\nNew URL: {1}\n\nThe database has been updated, and new entries will be fetched from the secure URL in the future.\n".format(feed.feedurl, results.url),
+                                                                                        "The blog aggregator at Planet PostgreSQL has picked up a redirect for your blog.\nOld URL: {0}\nNew URL: {1}\n\nThe database has been updated, and new entries will be fetched from the secure URL in the future.\n".format(feed.feedurl, results.url),
                                                                                         sendername="Planet PostgreSQL",
-                                                                                        receivername=u"{0} {1}".format(feed.user.first_name, feed.user.last_name),
+                                                                                        receivername="{0} {1}".format(feed.user.first_name, feed.user.last_name),
                                                                                         )
                                                        send_simple_mail(settings.EMAIL_SENDER,
                                                                                         settings.NOTIFICATION_RECEIVER,
                                                                                         "Blog redirect detected on Planet PostgreSQL",
-                                                                                        u"The blog at {0} by {1}\nis returning a redirect to a https version of itself.\n\nThe database has automatically been updated, and will start fetching using https in the future,\n\n".format(feed.feedurl, feed.user),
+                                                                                        "The blog at {0} by {1}\nis returning a redirect to a https version of itself.\n\nThe database has automatically been updated, and will start fetching using https in the future,\n\n".format(feed.feedurl, feed.user),
                                                                                         sendername="Planet PostgreSQL",
                                                                                         receivername="Planet PostgreSQL Moderators",
                                                        )
@@ -145,17 +145,17 @@ class Command(BaseCommand):
                                                                send_simple_mail(settings.EMAIL_SENDER,
                                                                                                 feed.user.email,
                                                                                                 "Many posts found at your blog at Planet PostgreSQL",
-                                                                                                u"The blog aggregator at Planet PostgreSQL has just picked up the following\nposts from your blog at {0}:\n\n{1}\n\nSince this is a large number of posts, they have been fetched\nand marked as hidden, to avoid possible duplicates.\n\nPlease go to https://planet.postgresql.org/register/edit/{2}\nand confirm (by unhiding) which of these should be posted.\n\nThank you!\n\n".format(
+                                                                                                "The blog aggregator at Planet PostgreSQL has just picked up the following\nposts from your blog at {0}:\n\n{1}\n\nSince this is a large number of posts, they have been fetched\nand marked as hidden, to avoid possible duplicates.\n\nPlease go to https://planet.postgresql.org/register/edit/{2}\nand confirm (by unhiding) which of these should be posted.\n\nThank you!\n\n".format(
                                                                                                         feed.blogurl,
                                                                                                         "\n".join(["* " + t for t in titles]),
                                                                                                         feed.id),
                                                                                                 sendername="Planet PostgreSQL",
-                                                                                                receivername=u"{0} {1}".format(feed.user.first_name, feed.user.last_name),
+                                                                                                receivername="{0} {1}".format(feed.user.first_name, feed.user.last_name),
                                                                )
                                                                send_simple_mail(settings.EMAIL_SENDER,
                                                                                                 settings.NOTIFICATION_RECEIVER,
                                                                                                 "Excessive posts from feed on Planet PostgreSQL",
-                                                                                                u"The blog at {0} by {1}\nreceived {2} new posts in a single fetch.\nAs this may be incorect, the posts have been marked as hidden.\nThe author may individually mark them as visible depending on\nprevious posts, and has been sent a notification about this.".format(feed.feedurl, feed.user, len(ids)),
+                                                                                                "The blog at {0} by {1}\nreceived {2} new posts in a single fetch.\nAs this may be incorect, the posts have been marked as hidden.\nThe author may individually mark them as visible depending on\nprevious posts, and has been sent a notification about this.".format(feed.feedurl, feed.user, len(ids)),
                                                                                                 sendername="Planet PostgreSQL",
                                                                                                 receivername="Planet PostgreSQL Moderators",
                                                                )
@@ -164,12 +164,12 @@ class Command(BaseCommand):
                                                                send_simple_mail(settings.EMAIL_SENDER,
                                                                                                 feed.user.email,
                                                                                                 "Posts found at your blog at Planet PostgreSQL",
-                                                                                                u"The blog aggregator at Planet PostgreSQL has just picked up the following\nposts from your blog at {0}:\n\n{1}\n\nIf these entries are correct, you don't have to do anything.\nIf any entry should not be there, head over to\n\nhttps://planet.postgresql.org/register/edit/{2}/\n\nand click the 'Hide' button for those entries as soon\nas possible.\n\nThank you!\n\n".format(
+                                                                                                "The blog aggregator at Planet PostgreSQL has just picked up the following\nposts from your blog at {0}:\n\n{1}\n\nIf these entries are correct, you don't have to do anything.\nIf any entry should not be there, head over to\n\nhttps://planet.postgresql.org/register/edit/{2}/\n\nand click the 'Hide' button for those entries as soon\nas possible.\n\nThank you!\n\n".format(
                                                                                                         feed.blogurl,
                                                                                                         "\n".join(["* " + t for t in titles]),
                                                                                                         feed.id),
                                                                                                 sendername="Planet PostgreSQL",
-                                                                                                receivername=u"{0} {1}".format(feed.user.first_name, feed.user.last_name),
+                                                                                                receivername="{0} {1}".format(feed.user.first_name, feed.user.last_name),
                                                                )
 
                                                if entries > 0 and not had_entries:
@@ -177,7 +177,7 @@ class Command(BaseCommand):
                                                        send_simple_mail(settings.EMAIL_SENDER,
                                                                                         settings.NOTIFICATION_RECEIVER,
                                                                                         "A blog was added to Planet PostgreSQL",
-                                                                                        u"The blog at {0} by {1}\nwas added to Planet PostgreSQL, and has now received entries.\n\nTo moderate: https://planet.postgresql.org/register/moderate/\n\n".format(feed.feedurl, feed.user),
+                                                                                        "The blog at {0} by {1}\nwas added to Planet PostgreSQL, and has now received entries.\n\nTo moderate: https://planet.postgresql.org/register/moderate/\n\n".format(feed.feedurl, feed.user),
                                                                                         sendername="Planet PostgreSQL",
                                                                                         receivername="Planet PostgreSQL Moderators",
                                                        )
@@ -188,20 +188,20 @@ class Command(BaseCommand):
                                                        send_simple_mail(settings.EMAIL_SENDER,
                                                                                         settings.NOTIFICATION_RECEIVER,
                                                                                         "A blog url changed on Planet PostgreSQL",
-                                                                                        u"When checking the blog at {0} by {1}\nthe blog URL was updated to:\n{2}\n(from previous value {3})\n\nTo moderate: https://planet.postgresql.org/register/moderate/\n\n".format(feed.feedurl, feed.user, feed.new_blogurl, feed.blogurl),
+                                                                                        "When checking the blog at {0} by {1}\nthe blog URL was updated to:\n{2}\n(from previous value {3})\n\nTo moderate: https://planet.postgresql.org/register/moderate/\n\n".format(feed.feedurl, feed.user, feed.new_blogurl, feed.blogurl),
                                                                                         sendername="Planet PostgreSQL",
                                                                                         receivername="Planet PostgreSQL Moderators",
                                                        )
                                                        send_simple_mail(settings.EMAIL_SENDER,
                                                                                         feed.user.email,
                                                                                         "URL of your blog at Planet PostgreSQL updated",
-                                                                                        u"The blog aggregator at Planet PostgreSQL has update the URL of your blog\nwith the feed at {0} to:\n{1} (from {2})\nIf this is correct, you don't have to do anything.\nIf not, please contact planet@postgresql.org\n".format(
+                                                                                        "The blog aggregator at Planet PostgreSQL has update the URL of your blog\nwith the feed at {0} to:\n{1} (from {2})\nIf this is correct, you don't have to do anything.\nIf not, please contact planet@postgresql.org\n".format(
                                                                                                 feed.feedurl,
                                                                                                 feed.new_blogurl,
                                                                                                 feed.blogurl,
                                                                                         ),
                                                                                         sendername="Planet PostgreSQL",
-                                                                                        receivername=u"{0} {1}".format(feed.user.first_name, feed.user.last_name),
+                                                                                        receivername="{0} {1}".format(feed.user.first_name, feed.user.last_name),
                                                                                         )
                                                        feed.blogurl = feed.new_blogurl
                                                        feed.save()
index d227a2d16f4ca74d3f4c51c50f3f8229df20a45f..99993d2ea6aeadacb3f179e2a39547761049e086 100644 (file)
@@ -26,7 +26,7 @@ class Command(BaseCommand):
                                send_simple_mail(settings.EMAIL_SENDER,
                                                                 feed.user.email,
                                                                 "Errors retreiving your feed for Planet PostgreSQL",
-                                                                u"""Your blog aggregated to Planet PostgreSQL with feed URL
+                                                                """Your blog aggregated to Planet PostgreSQL with feed URL
 
 {0}
 
@@ -52,5 +52,5 @@ is generating more than {3} errors per day.
                   feed.id,
                   THRESHOLD),
                                                                 sendername="Planet PostgreSQL",
-                                                                receivername=u"{0} {1}".format(feed.user.first_name, feed.user.last_name),
+                                                                receivername="{0} {1}".format(feed.user.first_name, feed.user.last_name),
                                                                 )
index d8a6c877dc96f6e908f424eed15c71e32017681d..f6c0bbe0a781052a311adcc5b6f525a2de81333c 100644 (file)
@@ -9,7 +9,7 @@ class Team(models.Model):
        name = models.CharField(max_length=255, blank=False)
        manager = models.ForeignKey(User, null=True, blank=True)
 
-       def __unicode__(self):
+       def __str__(self):
                return "%s (%s)" % (self.name, self.teamurl)
 
        class Meta:
@@ -35,7 +35,7 @@ class Blog(models.Model):
        # Things that may change
        new_blogurl = None
 
-       def __unicode__(self):
+       def __str__(self):
                return "%s (%s)" % (self.name, self.feedurl)
 
        @property
@@ -80,7 +80,7 @@ class Post(models.Model):
        twittered = models.BooleanField(default=False)
        shortlink = models.CharField(max_length=255)
 
-       def __unicode__(self):
+       def __str__(self):
                return self.title
 
        class Meta:
@@ -115,7 +115,7 @@ class AuditEntry(models.Model):
                self.user = username
                self.logtxt = txt
 
-       def __unicode__(self):
+       def __str__(self):
                return "%s (%s): %s" % (self.logtime, self.user, self.logtxt)
 
        class Meta:
@@ -132,5 +132,5 @@ class AggregatorLog(models.Model):
                db_table = 'aggregatorlog'
                ordering = ['-ts']
 
-       def __unicode__(self):
+       def __str__(self):
                return "Log entry for %s (%s)" % (self.feed.name, self.ts)
index 9d6b4f33888969d524d0d8258ace2fd8fd5ef072..7f411af77c117fbc4c0658f3256904bf2e37edb6 100644 (file)
@@ -67,7 +67,7 @@ def edit(request, id=None):
                else:
                        blog = get_object_or_404(Blog, id=id, user=request.user)
        else:
-               blog = Blog(user=request.user, name = u"{0} {1}".format(request.user.first_name, request.user.last_name))
+               blog = Blog(user=request.user, name = "{0} {1}".format(request.user.first_name, request.user.last_name))
 
        if request.method == 'POST':
                saved_url = blog.feedurl
@@ -87,7 +87,7 @@ def edit(request, id=None):
                                                send_simple_mail(settings.EMAIL_SENDER,
                                                                                 settings.NOTIFICATION_RECEIVER,
                                                                                 "A blog was edited on Planet PostgreSQL",
-                                                                                u"The blog at {0}\nwas edited by {1} in a way that needs new moderation.\n\nTo moderate: https://planet.postgresql.org/register/moderate/\n\n".format(blog.feedurl, blog.user),
+                                                                                "The blog at {0}\nwas edited by {1} in a way that needs new moderation.\n\nTo moderate: https://planet.postgresql.org/register/moderate/\n\n".format(blog.feedurl, blog.user),
                                                                                 sendername="Planet PostgreSQL",
                                                                                 receivername="Planet PostgreSQL Moderators",
                                                                         )
@@ -107,12 +107,12 @@ def edit(request, id=None):
                                send_simple_mail(settings.EMAIL_SENDER,
                                                                 obj.team.manager.email,
                                                                 "A blog joined your team on Planet PostgreSQL",
-                                                                u"The blog at {0} by {1} {2}\nhas been added to yor team {3} on Planet PostgreSQL\n\nIf this is correct, you do not need to do anything.\n\nIf this is incorrect, please go to\n\nhttps://planet.postgresql.org/register/\n\nand click the button to remove the blog from your team.\nWe apologize if this causes work for you.\n\n".format(
+                                                                "The blog at {0} by {1} {2}\nhas been added to yor team {3} on Planet PostgreSQL\n\nIf this is correct, you do not need to do anything.\n\nIf this is incorrect, please go to\n\nhttps://planet.postgresql.org/register/\n\nand click the button to remove the blog from your team.\nWe apologize if this causes work for you.\n\n".format(
                                                                         obj.feedurl,
                                                                         obj.user.first_name, obj.user.last_name,
                                                                         obj.team.name),
                                                                 sendername="Planet PostgreSQL",
-                                                                receivername=u"{0} {1}".format(obj.team.manager.first_name, obj.team.manager.last_name),
+                                                                receivername="{0} {1}".format(obj.team.manager.first_name, obj.team.manager.last_name),
                                                                 )
 
                        return HttpResponseRedirect("/register/edit/{0}/".format(obj.id))
@@ -139,7 +139,7 @@ def delete(request, id):
        send_simple_mail(settings.EMAIL_SENDER,
                                         settings.NOTIFICATION_RECEIVER,
                                         "A blog was deleted on Planet PostgreSQL",
-                                        u"The blog at {0} by {1}\nwas deleted by {2}\n\n".format(blog.feedurl, blog.name, request.user.username),
+                                        "The blog at {0} by {1}\nwas deleted by {2}\n\n".format(blog.feedurl, blog.name, request.user.username),
                                         sendername="Planet PostgreSQL",
                                         receivername="Planet PostgreSQL Moderators",
        )
@@ -160,7 +160,7 @@ def archive(request, id):
        send_simple_mail(settings.EMAIL_SENDER,
                                         settings.NOTIFICATION_RECEIVER,
                                         "A blog was archived on Planet PostgreSQL",
-                                        u"The blog at {0} by {1}\nwas archived by {2}\n\n".format(blog.feedurl, blog.name, request.user.username),
+                                        "The blog at {0} by {1}\nwas archived by {2}\n\n".format(blog.feedurl, blog.name, request.user.username),
                                         sendername="Planet PostgreSQL",
                                         receivername="Planet PostgreSQL Moderators",
        )
@@ -187,7 +187,7 @@ def remove_from_team(request, teamid, blogid):
        send_simple_mail(settings.EMAIL_SENDER,
                                         settings.NOTIFICATION_RECEIVER,
                                         "A blog was removed from a team on Planet PostgreSQL",
-                                        u"The blog at {0} by {1} {2}\nwas removed from team {3} by {4}.\n".format(
+                                        "The blog at {0} by {1} {2}\nwas removed from team {3} by {4}.\n".format(
                                                 blog.feedurl, blog.user.first_name, blog.user.last_name, team.name, request.user.username),
                                         sendername="Planet PostgreSQL",
                                         receivername="Planet PostgreSQL Moderators",
@@ -196,9 +196,9 @@ def remove_from_team(request, teamid, blogid):
        send_simple_mail(settings.EMAIL_SENDER,
                                         blog.user.email,
                                         "Your blog on Planet PostgreSQL was removed from the team",
-                                        u"Your blog at {0} has been removed\nfrom the team {1} on Planet PostgreSQL.\n\nIf you believe this to be in error, please contact\nthe team administrator.\n\n".format(blog.feedurl, team.name),
+                                        "Your blog at {0} has been removed\nfrom the team {1} on Planet PostgreSQL.\n\nIf you believe this to be in error, please contact\nthe team administrator.\n\n".format(blog.feedurl, team.name),
                                         sendername="Planet PostgreSQL",
-                                        receivername=u"{0} {1}".format(blog.user.first_name, blog.user.last_name),
+                                        receivername="{0} {1}".format(blog.user.first_name, blog.user.last_name),
                                         )
 
        messages.info(request, "Blog {0} removed from team {1}".format(blog.feedurl, team.name))
@@ -273,20 +273,20 @@ def moderate_reject(request, blogid):
                        send_simple_mail(settings.EMAIL_SENDER,
                                                         settings.NOTIFICATION_RECEIVER,
                                                         "A blog was rejected on Planet PostgreSQL",
-                                                        u"The blog at {0} by {1} {2}\nwas marked as rejected by {3}. The message given was:\n\n{4}\n\n".format(blog.feedurl, blog.user.first_name, blog.user.last_name, request.user.username, form.cleaned_data['message']),
+                                                        "The blog at {0} by {1} {2}\nwas marked as rejected by {3}. The message given was:\n\n{4}\n\n".format(blog.feedurl, blog.user.first_name, blog.user.last_name, request.user.username, form.cleaned_data['message']),
                                                         sendername="Planet PostgreSQL",
                                                         receivername="Planet PostgreSQL Moderators",
                                                         )
-                       messages.info(request, u"Blog {0} rejected, notification sent to moderators".format(blog.feedurl))
+                       messages.info(request, "Blog {0} rejected, notification sent to moderators".format(blog.feedurl))
                        if not form.cleaned_data['modsonly']:
                                send_simple_mail(settings.EMAIL_SENDER,
                                                                 blog.user.email,
                                                                 "Your blog submission to Planet PostgreSQL",
-                                                                u"The blog at {0} that you submitted to Planet PostgreSQL has\nunfortunately been rejected. The reason given was:\n\n{1}\n\n".format(blog.feedurl, form.cleaned_data['message']),
+                                                                "The blog at {0} that you submitted to Planet PostgreSQL has\nunfortunately been rejected. The reason given was:\n\n{1}\n\n".format(blog.feedurl, form.cleaned_data['message']),
                                                                 sendername="Planet PostgreSQL",
-                                                                receivername = u"{0} {1}".format(blog.user.first_name, blog.user.last_name),
+                                                                receivername = "{0} {1}".format(blog.user.first_name, blog.user.last_name),
                                                                 )
-                               messages.info(request, u"Blog {0} rejected, notification sent to blog owner".format(blog.feedurl))
+                               messages.info(request, "Blog {0} rejected, notification sent to blog owner".format(blog.feedurl))
 
                        blog.delete()
                        return HttpResponseRedirect("/register/moderate/")
@@ -306,13 +306,13 @@ def moderate_approve(request, blogid):
        blog = get_object_or_404(Blog, id=blogid)
 
        if blog.approved:
-               messages.info(request, u"Blog {0} was already approved.".format(blog.feedurl))
+               messages.info(request, "Blog {0} was already approved.".format(blog.feedurl))
                return HttpResponseRedirect("/register/moderate/")
 
        send_simple_mail(settings.EMAIL_SENDER,
                                         settings.NOTIFICATION_RECEIVER,
                                         "A blog was approved on Planet PostgreSQL",
-                                        u"The blog at {0} by {1} {2}\nwas marked as approved by {3}.\n\n".format(blog.feedurl, blog.user.first_name, blog.user.last_name, request.user.username),
+                                        "The blog at {0} by {1} {2}\nwas marked as approved by {3}.\n\n".format(blog.feedurl, blog.user.first_name, blog.user.last_name, request.user.username),
                                         sendername="Planet PostgreSQL",
                                         receivername="Planet PostgreSQL Moderators",
        )
@@ -320,9 +320,9 @@ def moderate_approve(request, blogid):
        send_simple_mail(settings.EMAIL_SENDER,
                                         blog.user.email,
                                         "Your blog submission to Planet PostgreSQL",
-                                        u"The blog at {0} that you submitted to Planet PostgreSQL has\nbeen approved.\n\n".format(blog.feedurl),
+                                        "The blog at {0} that you submitted to Planet PostgreSQL has\nbeen approved.\n\n".format(blog.feedurl),
                                         sendername="Planet PostgreSQL",
-                                        receivername = u"{0} {1}".format(blog.user.first_name, blog.user.last_name),
+                                        receivername = "{0} {1}".format(blog.user.first_name, blog.user.last_name),
        )
 
        blog.approved = True
@@ -330,7 +330,7 @@ def moderate_approve(request, blogid):
 
        AuditEntry(request.user.username, 'Approved blog %s at %s' % (blog.id, blog.feedurl)).save()
 
-       messages.info(request, u"Blog {0} approved, notification sent to moderators and owner.".format(blog.feedurl))
+       messages.info(request, "Blog {0} approved, notification sent to moderators and owner.".format(blog.feedurl))
 
        purge_root_and_feeds()
        purge_url('/feeds.html')
index c1cd6c140793f4c423dc93a3d4e2b25d356de09e..064d2879f72a90f57845e3581c06eba1c87f4592 100644 (file)
@@ -4,7 +4,7 @@ import urllib.request, urllib.error, urllib.parse
 
 def purge_url(url):
        if not settings.VARNISH_URL:
-               print "Not purging {0}".format(url)
+               print("Not purging {0}".format(url))
        else:
                try:
                        request = urllib2.Request(settings.VARNISH_URL, headers={
index 4ec76210407a20677342446eb984ff2fddb1f6ec..8154c56ba7e8103dff6b762b4073a43794be4ff7 100755 (executable)
@@ -52,8 +52,8 @@ class PostToTwitter(TwitterClient):
                                # in case it's needed.
                                try:
                                        short = self.shortid(post[0])
-                               except Exception, e:
-                                       print "Failed to shorten URL %s: %s" % (post[2], e)
+                               except Exception as e:
+                                       print("Failed to shorten URL %s: %s" % (post[2], e))
                                        continue
 
                                c.execute("UPDATE posts SET shortlink=%(short)s WHERE id=%(id)s", {
@@ -65,14 +65,14 @@ class PostToTwitter(TwitterClient):
                        # Set up the string to twitter
                        if post[5] and len(post[5])>1:
                                # Twitter username registered
-                               msg = unicode("%s (@%s): %s %s") % (
+                               msg = "%s (@%s): %s %s" % (
                                        post[4],
                                        post[5],
                                        self.trimpost(post[1],len(post[4])+len(post[5])+len(short)+7),
                                        short,
                                )
                        else:
-                               msg = unicode("%s: %s %s") % (
+                               msg = "%s: %s %s" % (
                                        post[4],
                                        self.trimpost(post[1],len(post[4])+len(short)+3),
                                        short,
@@ -81,8 +81,8 @@ class PostToTwitter(TwitterClient):
                        # Now post it to twitter
                        try:
                                self.do_post(msg)
-                       except Exception, e:
-                               print "Error posting to twitter (post %s): %s" % (post[0], e)
+                       except Exception as e:
+                               print("Error posting to twitter (post %s): %s" % (post[0], e))
                                # We'll just try again with the next one
                                continue
 
@@ -90,7 +90,7 @@ class PostToTwitter(TwitterClient):
                        c.execute("UPDATE posts SET twittered='t' WHERE id=%(id)s", { 'id': post[0] })
                        self.db.commit()
 
-                       print unicode("Twittered: %s" % msg).encode('utf8')
+                       print("Twittered: %s" % msg)
 
 
        # Trim a post to the length required by twitter, so we don't fail to post
index 77e36bac8cca085682c7e71c95f798d10e3bdbcf..95409493a9c7aa85850d9bd120bfd861a79fda7c 100755 (executable)
@@ -17,17 +17,17 @@ cfg = configparser.ConfigParser()
 cfg.read('planet.ini')
 
 if not cfg.has_option('twitter', 'consumer') or not cfg.has_option('twitter', 'consumersecret'):
-       print "Before you can run this, you need to register an application at"
-       print "developer.twitter.com and put the consumer and consumersecret values"
-       print "in the [twitter] section of planet.ini."
+       print("Before you can run this, you need to register an application at")
+       print("developer.twitter.com and put the consumer and consumersecret values")
+       print("in the [twitter] section of planet.ini.")
        sys.exit(1)
 
 oauth = requests_oauthlib.OAuth1Session(cfg.get('twitter', 'consumer'), cfg.get('twitter', 'consumersecret'))
 fetch_response = oauth.fetch_request_token('https://api.twitter.com/oauth/request_token')
 auth_url = oauth.authorization_url('https://api.twitter.com/oauth/authorize')
 
-print "Please go to {0} and log in".format(auth_url)
-pin = raw_input('Enter the PIN received here:')
+print("Please go to {0} and log in".format(auth_url))
+pin = input('Enter the PIN received here:')
 
 oauth2 = requests_oauthlib.OAuth1Session(cfg.get('twitter', 'consumer'),
                                                                                 cfg.get('twitter', 'consumersecret'),
@@ -37,10 +37,10 @@ oauth2 = requests_oauthlib.OAuth1Session(cfg.get('twitter', 'consumer'),
 tokens = oauth2.fetch_access_token('https://api.twitter.com/oauth/access_token')
 
 
-print "Access token received."
-print "Register the following two valuesi n planet.ini under [twitter]:"
-print "token={0}".format(tokens.get('oauth_token'))
-print "secret={0}".format(tokens.get('oauth_token_secret'))
+print("Access token received.")
+print("Register the following two valuesi n planet.ini under [twitter]:")
+print("token={0}".format(tokens.get('oauth_token')))
+print("secret={0}".format(tokens.get('oauth_token_secret')))
 
 
 
index 3b9196138848145a6ce08729433aa6e0f0af265c..697e0a121cb4fd69dcc92dc29e917f6cfa435847 100755 (executable)
@@ -39,7 +39,7 @@ class SyncTwitter(TwitterClient):
                        if not self.add_subscriber(s):
                                # Most likely it's things like it doesn't exist or we don't have permissions
                                # to follow it.
-                               print "Failed to add twitter subscriber {0}, removing from feed record".format(s)
+                               print("Failed to add twitter subscriber {0}, removing from feed record".format(s))
 
                                # To be on the safe side, store the old twitter username. In case the twitter APIs
                                # go bonkers on us and we end up removing too much.
index a47c7581ad0502c394d0e96c04d47c9b8ba984f0..f303c172dec8fca2dba67d25926e2e60e90baedd 100644 (file)
@@ -44,7 +44,7 @@ class TwitterClient(object):
                                'cursor': cursor,
                                })
                        if response.status_code != 200:
-                               print response.json()
+                               print(response.json())
                                raise Exception("Received status {0} when listing users".format(response.status_code))
                        j = response.json()
                        handles.extend([x['screen_name'].lower() for x in j['users']])
@@ -53,7 +53,7 @@ class TwitterClient(object):
                return handles
 
        def remove_subscriber(self, name):
-               print "Removing twitter user %s from list." % name
+               print("Removing twitter user %s from list." % name)
                r = self.tw.post('{0}lists/members/destroy.json'.format(self.twitter_api), data={
                        'owner_screen_name': self.twittername,
                        'slug': self.twitterlist,
@@ -64,12 +64,12 @@ class TwitterClient(object):
                                err = r.json()['errors'][0]['message']
                        except:
                                err = 'Response does not contain error messages with json'
-                       print "Failed to remove subscriber {0}: {1}".format(name, err)
+                       print("Failed to remove subscriber {0}: {1}".format(name, err))
                        return False
                return True
 
        def add_subscriber(self, name):
-               print "Adding twitter user %s to list." % name
+               print("Adding twitter user %s to list." % name)
                r = self.tw.post('{0}lists/members/create.json'.format(self.twitter_api), data={
                        'owner_screen_name': self.twittername,
                        'slug': self.twitterlist,
@@ -80,6 +80,6 @@ class TwitterClient(object):
                                err = r.json()['errors'][0]['message']
                        except:
                                err = 'Response does not contain error messages with json'
-                       print "Failed to add subscriber {0}: {1}".format(name, err)
+                       print("Failed to add subscriber {0}: {1}".format(name, err))
                        return False
                return True