From: Ciaran Gultnieks <ciaran@ciarang.com>
Date: Wed, 9 Oct 2013 12:23:17 +0000 (+0100)
Subject: A few more wiki fixes
X-Git-Tag: 0.1~374
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=d2867a319276a6dcd39821f5d4544b19354d17ab;p=fdroidserver.git

A few more wiki fixes
---

diff --git a/fdroidserver/update.py b/fdroidserver/update.py
index 03865e2c..9a6b5ba3 100644
--- a/fdroidserver/update.py
+++ b/fdroidserver/update.py
@@ -146,7 +146,7 @@ def update_wiki(apps, apks, verbose=False):
             wikidata += '\n[[Category:Apps with no packages]]\n'
         elif cantupdate and not app['Disabled']:
             wikidata += "\n[[Category:Apps we can't update]]\n"
-        elif cantupdate and not app['Disabled']:
+        elif buildfails and not app['Disabled']:
             wikidata += "\n[[Category:Apps with failing builds]]\n"
         elif not gotcurrentver and not app['Disabled']:
             wikidata += '\n[[Category:Apps to Update]]\n'
@@ -171,8 +171,11 @@ def update_wiki(apps, apks, verbose=False):
         apppagename = apppagename.replace('{', '')
         apppagename = apppagename.replace('}', ' ')
         apppagename = apppagename.replace(':', ' ')
-        for page in site.allpages(prefix=apppagename, filterredir='nonredirects'):
-            if page.name == apppagename:
+        # Drop double spaces caused mostly by replacing ':' above
+        apppagename = apppagename.replace('  ', ' ')
+        for expagename in site.allpages(prefix=apppagename,
+                filterredir='nonredirects', generator=False):
+            if expagename == apppagename:
                 noclobber = True
         # Another reason not to make the redirect page is if the app name
         # is the same as it's ID, because that will overwrite the real page