From 935e6c4b0f4ecb9e55ceac39ebd6f3885a90a218 Mon Sep 17 00:00:00 2001
From: "Edward Z. Yang" <ezyang@mit.edu>
Date: Sat, 6 Feb 2010 19:26:35 -0500
Subject: [PATCH] Check for old style URLs.

Signed-off-by: Edward Z. Yang <ezyang@mit.edu>
---
 wizard/app/wordpress.py | 12 +++++++++---
 wizard/deploy.py        |  6 ++++++
 wizard/scripts.py       | 19 +++++++++++++------
 3 files changed, 28 insertions(+), 9 deletions(-)

diff --git a/wizard/app/wordpress.py b/wizard/app/wordpress.py
index 0035209..546827f 100644
--- a/wizard/app/wordpress.py
+++ b/wizard/app/wordpress.py
@@ -41,9 +41,15 @@ class Application(app.Application):
         return os.path.isfile("wp-config.php")
     def checkWeb(self, deployment):
         # XXX: this sucks pretty hard
-        return self.checkWebPage(deployment, "",
-                outputs=["<html", "WordPress", "feed"],
-                exclude=["Error establishing a database connection"])
+        def doCheck():
+            return self.checkWebPage(deployment, "",
+                    outputs=["<html", "WordPress", "feed"],
+                    exclude=["Error establishing a database connection"])
+        if not doCheck():
+            deployment.enableOldStyleUrls()
+            return doCheck()
+        else:
+            return True
     def detectVersion(self, deployment):
         return self.detectVersionFromFile("wp-includes/version.php", php.re_var("wp_version"))
     def install(self, version, options):
diff --git a/wizard/deploy.py b/wizard/deploy.py
index 500f89d..4fdff7d 100644
--- a/wizard/deploy.py
+++ b/wizard/deploy.py
@@ -302,6 +302,12 @@ class Deployment(object):
         if not self._url:
             raise UnknownWebPath
         return self._url
+    def enableOldStyleUrls(self):
+        """
+        Switches to using http://user.scripts.mit.edu/~user/app URLs.
+        No effect if they have an explicit .scripts/url override.
+        """
+        self._url = scripts.fill_url(self.location, self.application.url(self), old_style = True)
     @staticmethod
     def parse(line):
         """
diff --git a/wizard/scripts.py b/wizard/scripts.py
index 0834b25..50af4af 100644
--- a/wizard/scripts.py
+++ b/wizard/scripts.py
@@ -13,7 +13,7 @@ import errno
 import wizard
 from wizard import shell, util
 
-def fill_url(dir, url=None):
+def fill_url(dir, url=None, old_style=False):
     """
     Attempts to determine the URL a directory would be web-accessible at.
     If ``url`` is specified, automatically use it.
@@ -26,11 +26,18 @@ def fill_url(dir, url=None):
     # try the directory
     homedir, _, web_path = dir.partition("/web_scripts")
     if web_path:
-        return urlparse.ParseResult(
-                "http",
-                util.get_dir_owner(homedir) + ".scripts.mit.edu",
-                web_path.rstrip('/'),
-                "", "", "")
+        if old_style:
+            return urlparse.ParseResult(
+                    "http",
+                    "scripts.mit.edu",
+                    "/~" + util.get_dir_owner(homedir) + web_path.rstrip('/'),
+                    "", "", "")
+        else:
+            return urlparse.ParseResult(
+                    "http",
+                    util.get_dir_owner(homedir) + ".scripts.mit.edu",
+                    web_path.rstrip('/'),
+                    "", "", "")
 
     # try the environment
     host = os.getenv("WIZARD_WEB_HOST")
-- 
2.45.2