Show redirects on separate lines

Author: stsewd

readthedocs/redirects/models.py

@@ -78,32 +78,31 @@ class Meta(object):
         ordering = ('-update_dt',)
 
     def __str__(self):
-        redirect_text = '{type}: {from_url} -> {to_url}'
-        if self.redirect_type == 'prefix':
+        redirect_text = '{type}: {from_to_url}'
+        if self.redirect_type in ['prefix', 'page', 'exact']:
             return redirect_text.format(
-                type=ugettext('Prefix Redirect'),
-                from_url=self.from_url,
-                to_url='/{lang}/{version}/'.format(
-                    lang=self.project.language,
-                    version=self.project.default_version
-                )
-            )
-        elif self.redirect_type == 'page':
-            return redirect_text.format(
-                type=ugettext('Page Redirect'),
-                from_url=self.from_url,
-                to_url=self.to_url
-            )
-        elif self.redirect_type == 'exact':
-            return redirect_text.format(
-                type=ugettext('Exact Redirect'),
-                from_url=self.from_url,
-                to_url=self.to_url
+                type=self.get_redirect_type_display(),
+                from_to_url=self.get_from_to_url_display()
             )
         return ugettext('Redirect: {}'.format(
             self.get_redirect_type_display())
         )
 
+    def get_from_to_url_display(self):
+        if self.redirect_type in ['prefix', 'page', 'exact']:
+            from_url = self.from_url
+            to_url = self.to_url
+            if self.redirect_type == 'prefix':
+                to_url = '/{lang}/{version}/'.format(
+                    lang=self.project.language,
+                    version=self.project.default_version
+                )
+            return '{from_url} -> {to_url}'.format(
+                from_url=from_url,
+                to_url=to_url
+            )
+        return ''
+
     def get_full_path(self, filename, language=None, version_slug=None):
         """
         Return a full path for a given filename.

readthedocs/templates/projects/project_redirects.html

@@ -82,9 +82,14 @@ <h3>{% trans "Redirects" %}</h3>
           <ul>
             {% for redirect in redirects %}
               <li class="module-item">
-                <span>
-                  {{ redirect }}
-                </span>
+                <div>
+                  {{ redirect.get_redirect_type_display }}
+                </div>
+                {% if redirect.get_from_to_url_display %}
+                  <div>
+                    {{ redirect.get_from_to_url_display }}
+                  </div>
+                {% endif %}
                 <ul class="module-item-menu">
                   <li>
                     <form method="post" action="{% url "projects_redirects_delete" project.slug %}">