From a923e00e636ad8b83bc0a689c9c7e8833aa63219 Mon Sep 17 00:00:00 2001
From: Sergey Motornyuk <sergey.motornyuk@linkdigital.com.au>
Date: Tue, 31 Oct 2017 17:59:34 +0200
Subject: [PATCH] admin blueprint. First step

---
 ckan/config/routing.py                     |   9 -
 ckan/lib/base.py                           |   8 +-
 ckan/logic/__init__.py                     |   8 +-
 ckan/templates-bs2/admin/base.html         |   6 +-
 ckan/templates-bs2/admin/config.html       |  31 ++-
 ckan/templates-bs2/admin/index.html        |   2 +-
 ckan/templates-bs2/admin/trash.html        |   3 +-
 ckan/templates/admin/base.html             |   6 +-
 ckan/templates/admin/config.html           |  73 ++++---
 ckan/templates/admin/index.html            |   2 +-
 ckan/templates/admin/trash.html            |   3 +-
 ckan/tests/legacy/functional/test_admin.py |   3 +-
 ckan/views/admin.py                        | 213 +++++++++++++++++++++
 13 files changed, 309 insertions(+), 58 deletions(-)
 create mode 100644 ckan/views/admin.py

diff --git a/ckan/config/routing.py b/ckan/config/routing.py
index 08b3a3e3bdc..8f0b246a0bb 100644
--- a/ckan/config/routing.py
+++ b/ckan/config/routing.py
@@ -371,15 +371,6 @@ def make_map():
         m.connect('/revision/list', action='list')
         m.connect('/revision/{id}', action='read')
 
-
-    map.connect('ckanadmin_index', '/ckan-admin', controller='admin',
-                action='index', ckan_icon='gavel')
-    map.connect('ckanadmin_config', '/ckan-admin/config', controller='admin',
-                action='config', ckan_icon='check-square-o')
-    map.connect('ckanadmin_trash', '/ckan-admin/trash', controller='admin',
-                action='trash', ckan_icon='trash-o')
-    map.connect('ckanadmin', '/ckan-admin/{action}', controller='admin')
-
     with SubMapper(map, controller='ckan.controllers.storage:StorageController') as m:
         m.connect('storage_file', '/storage/f/{label:.*}',
                   action='file')
diff --git a/ckan/lib/base.py b/ckan/lib/base.py
index 16c5eacd4a6..67df4949a31 100644
--- a/ckan/lib/base.py
+++ b/ckan/lib/base.py
@@ -14,7 +14,10 @@
 from pylons.templating import cached_template, pylons_globals
 from webhelpers.html import literal
 
-from flask import render_template as flask_render_template
+from flask import (
+    render_template as flask_render_template,
+    abort as flask_abort
+)
 import ckan.exceptions
 import ckan
 import ckan.lib.i18n as i18n
@@ -61,6 +64,9 @@ def abort(status_code=None, detail='', headers=None, comment=None):
     # #1267 Convert detail to plain text, since WebOb 0.9.7.1 (which comes
     # with Lucid) causes an exception when unicode is received.
     detail = detail.encode('utf8')
+    if is_flask_request():
+        flask_abort(status_code, detail)
+
     return _abort(status_code=status_code,
                   detail=detail,
                   headers=headers,
diff --git a/ckan/logic/__init__.py b/ckan/logic/__init__.py
index f0833f316b1..ec8ec67a76c 100644
--- a/ckan/logic/__init__.py
+++ b/ckan/logic/__init__.py
@@ -132,7 +132,13 @@ def parse_params(params, ignore_keys=None):
     for key in params:
         if ignore_keys and key in ignore_keys:
             continue
-        value = params.getall(key)
+        # flask request has `getlist` instead of pylons' `getall`
+
+        if hasattr(params, 'getall'):
+            value = params.getall(key)
+        else:
+            value = params.getlist(key)
+
         # Blank values become ''
         if not value:
             value = ''
diff --git a/ckan/templates-bs2/admin/base.html b/ckan/templates-bs2/admin/base.html
index fc45c38eb90..741c229c324 100644
--- a/ckan/templates-bs2/admin/base.html
+++ b/ckan/templates-bs2/admin/base.html
@@ -5,8 +5,8 @@
 {% block breadcrumb_content %}{% endblock %}
 
 {% block content_primary_nav %}
-  {{ h.build_nav_icon('ckanadmin_index', _('Sysadmins')) }}
-  {{ h.build_nav_icon('ckanadmin_config', _('Config')) }}
-  {{ h.build_nav_icon('ckanadmin_trash', _('Trash')) }}
+  {{ h.build_nav_icon('admin.index', _('Sysadmins'), icon='gavel') }}
+  {{ h.build_nav_icon('admin.config', _('Config'), icon='check-square-o') }}
+  {{ h.build_nav_icon('admin.trash', _('Trash'), icon='trash-o') }}
   {{ h.build_extra_admin_nav() }}
 {% endblock %}
diff --git a/ckan/templates-bs2/admin/config.html b/ckan/templates-bs2/admin/config.html
index 840783a9fb3..0d247f4bca4 100644
--- a/ckan/templates-bs2/admin/config.html
+++ b/ckan/templates-bs2/admin/config.html
@@ -6,21 +6,40 @@
 
 {% block primary_content_inner %}
 
-  {{ form.errors(error_summary) }}
+{{ form.errors(error_summary) }}
 
   <form method='post' action="" class="form-horizontal" id="admin-config-form" enctype="multipart/form-data">
     {% block admin_form %}
-      {{ autoform.generate(form_items, data, errors) }}
+
+    {{ form.input('ckan.site_title', id='field-ckan-site-title', label=_('Site Title'), value=data['ckan.site_title'], error=error, classes=['control-medium']) }}
+
+    {{ form.select('ckan.main_css', id='field-ckan-main-css', label=_('Style'), options=styles, selected=data['ckan.main_css'], error=error) }}
+
+    {{ form.input('ckan.site_description', id='field-ckan-site-description', label=_('Site Tag Line'), value=data['ckan.site_description'], error=error, classes=['control-medium']) }}
+
+    {% set field_url = 'ckan.site_logo' %}
+    {% set is_upload = data[field_url] and not data[field_url].startswith('http') %}
+    {% set is_url = data[field_url] and data[field_url].startswith('http') %}
+    {{ form.image_upload(data, errors, is_upload_enabled=h.uploads_enabled(), is_url=is_url, is_upload=is_upload, upload_label = _('Site logo'), url_label=_('Site logo'),  field_url=field_url, field_upload='logo_upload', field_clear='clear_logo_upload' )}}
+
+    {{ form.markdown('ckan.site_about', id='field-ckan-site-about', label=_('About'), value=data['ckan.site_about'], error=error, placeholder=_('About page text')) }}
+
+    {{ form.markdown('ckan.site_intro_text', id='field-ckan-site-intro-text', label=_('Intro Text'), value=data['ckan.site_intro_text'], error=error, placeholder=_('Text on home page')) }}
+
+    {{ form.textarea('ckan.site_custom_css', id='field-ckan-site-custom-css', label=_('Custom CSS'), value=data['ckan.site_custom_css'], error=error, placeholder=_('Customisable css inserted into the page header')) }}
+
+    {{ form.select('ckan.homepage_style', id='field-homepage-style', label=_('Homepage'), options=homepages, selected=data['ckan.homepage_style'], error=error) }}
+
     {% endblock %}
     <div class="form-actions">
-      <a href="{% url_for controller='admin', action='reset_config' %}" class="btn btn-danger pull-left" data-module="confirm-action" data-module-content="{{ _('Are you sure you want to reset the config?') }}">{{ _('Reset') }}</a>
-      <button type="submit" class="btn btn-primary" name="save">{{ _('Update Config') }}</button>
+        <a href="{% url_for controller='admin', action='reset_config' %}" class="btn btn-danger pull-left" data-module="confirm-action" data-module-content="{{ _('Are you sure you want to reset the config?') }}">{{ _('Reset') }}</a>
+        <button type="submit" class="btn btn-primary" name="save">{{ _('Update Config') }}</button>
     </div>
-  </form>
+</form>
 {% endblock %}
 
 {% block secondary_content %}
-  <div class="module module-narrow module-shallow">
+<div class="module module-narrow module-shallow">
     <h2 class="module-heading">
       <i class="fa fa-info-circle"></i>
       {{ _('CKAN config options') }}
diff --git a/ckan/templates-bs2/admin/index.html b/ckan/templates-bs2/admin/index.html
index 498da63d440..c313bf32573 100644
--- a/ckan/templates-bs2/admin/index.html
+++ b/ckan/templates-bs2/admin/index.html
@@ -2,7 +2,7 @@
 
 {% block primary_content_inner %}
   <ul class="user-list">
-    {% for user in c.sysadmins %}
+    {% for user in sysadmins %}
       <li>{{ h.linked_user(user) }}</li>
     {% endfor %}
   </ul>
diff --git a/ckan/templates-bs2/admin/trash.html b/ckan/templates-bs2/admin/trash.html
index 2484eb7cefd..108ce9e6b35 100644
--- a/ckan/templates-bs2/admin/trash.html
+++ b/ckan/templates-bs2/admin/trash.html
@@ -4,7 +4,7 @@
   {% set truncate = truncate or 180 %}
   {% set truncate_title = truncate_title or 80 %}
   <ul class="user-list">
-    {% for pkg in c.deleted_packages %}
+    {% for pkg in deleted_packages %}
       {% set title = pkg.title or pkg.name %}
       <li>{{ h.link_to(h.truncate(title, truncate_title), h.url_for(controller='package', action='read', id=pkg.name)) }}</li>
     {% endfor %}
@@ -35,4 +35,3 @@ <h2 class="module-heading">
     </div>
   </div>
 {% endblock %}
-
diff --git a/ckan/templates/admin/base.html b/ckan/templates/admin/base.html
index fc45c38eb90..741c229c324 100644
--- a/ckan/templates/admin/base.html
+++ b/ckan/templates/admin/base.html
@@ -5,8 +5,8 @@
 {% block breadcrumb_content %}{% endblock %}
 
 {% block content_primary_nav %}
-  {{ h.build_nav_icon('ckanadmin_index', _('Sysadmins')) }}
-  {{ h.build_nav_icon('ckanadmin_config', _('Config')) }}
-  {{ h.build_nav_icon('ckanadmin_trash', _('Trash')) }}
+  {{ h.build_nav_icon('admin.index', _('Sysadmins'), icon='gavel') }}
+  {{ h.build_nav_icon('admin.config', _('Config'), icon='check-square-o') }}
+  {{ h.build_nav_icon('admin.trash', _('Trash'), icon='trash-o') }}
   {{ h.build_extra_admin_nav() }}
 {% endblock %}
diff --git a/ckan/templates/admin/config.html b/ckan/templates/admin/config.html
index 6ac7d26c072..37e09caeda9 100644
--- a/ckan/templates/admin/config.html
+++ b/ckan/templates/admin/config.html
@@ -10,12 +10,31 @@
 
   <form method='post' action="" id="admin-config-form" enctype="multipart/form-data">
     {% block admin_form %}
-      {{ autoform.generate(form_items, data, errors) }}
-    {% endblock %}
-    <div class="form-actions">
-      <a href="{% url_for controller='admin', action='reset_config' %}" class="btn btn-danger pull-left" data-module="confirm-action" data-module-content="{{ _('Are you sure you want to reset the config?') }}">{{ _('Reset') }}</a>
-      <button type="submit" class="btn btn-primary" name="save">{{ _('Update Config') }}</button>
-    </div>
+
+      {{ form.input('ckan.site_title', id='field-ckan-site-title', label=_('Site Title'), value=data['ckan.site_title'], error=error, classes=['control-medium']) }}
+
+      {{ form.select('ckan.main_css', id='field-ckan-main-css', label=_('Style'), options=styles, selected=data['ckan.main_css'], error=error) }}
+
+      {{ form.input('ckan.site_description', id='field-ckan-site-description', label=_('Site Tag Line'), value=data['ckan.site_description'], error=error, classes=['control-medium']) }}
+
+      {% set field_url = 'ckan.site_logo' %}
+      {% set is_upload = data[field_url] and not data[field_url].startswith('http') %}
+      {% set is_url = data[field_url] and data[field_url].startswith('http') %}
+      {{ form.image_upload(data, errors, is_upload_enabled=h.uploads_enabled(), is_url=is_url, is_upload=is_upload, upload_label = _('Site logo'), url_label=_('Site logo'),  field_url=field_url, field_upload='logo_upload', field_clear='clear_logo_upload' )}}
+
+      {{ form.markdown('ckan.site_about', id='field-ckan-site-about', label=_('About'), value=data['ckan.site_about'], error=error, placeholder=_('About page text')) }}
+
+      {{ form.markdown('ckan.site_intro_text', id='field-ckan-site-intro-text', label=_('Intro Text'), value=data['ckan.site_intro_text'], error=error, placeholder=_('Text on home page')) }}
+
+      {{ form.textarea('ckan.site_custom_css', id='field-ckan-site-custom-css', label=_('Custom CSS'), value=data['ckan.site_custom_css'], error=error, placeholder=_('Customisable css inserted into the page header')) }}
+
+      {{ form.select('ckan.homepage_style', id='field-homepage-style', label=_('Homepage'), options=homepages, selected=data['ckan.homepage_style'], error=error) }}
+
+      {% endblock %}
+      <div class="form-actions">
+        <a href="{% url_for controller='admin', action='reset_config' %}" class="btn btn-danger pull-left" data-module="confirm-action" data-module-content="{{ _('Are you sure you want to reset the config?') }}">{{ _('Reset') }}</a>
+        <button type="submit" class="btn btn-primary" name="save">{{ _('Update Config') }}</button>
+      </div>
   </form>
 {% endblock %}
 
@@ -27,28 +46,28 @@ <h2 class="module-heading">
     </h2>
     <div class="module-content">
       {% block admin_form_help %}
-      {% set about_url = h.url_for(controller='home', action='about') %}
-      {% set home_url = h.url_for(controller='home', action='index') %}
-      {% set docs_url = "http://docs.ckan.org/en/{0}/theming".format(g.ckan_doc_version) %}
-      {% trans %}
-        <p><strong>Site Title:</strong> This is the title of this CKAN instance
-          It appears in various places throughout CKAN.</p>
-        <p><strong>Style:</strong> Choose from a list of simple variations of
-          the main colour scheme to get a very quick custom theme working.</p>
-        <p><strong>Site Tag Logo:</strong> This is the logo that appears in the
-          header of all the CKAN instance templates.</p>
-        <p><strong>About:</strong> This text will appear on this CKAN instances
-          <a href="{{ about_url }}">about page</a>.</p>
-        <p><strong>Intro Text:</strong> This text will appear on this CKAN instances
-          <a href="{{ home_url }}">home page</a> as a welcome to visitors.</p>
-        <p><strong>Custom CSS:</strong> This is a block of CSS that appears in
-          <code>&lt;head&gt;</code> tag of every page. If you wish to customize
-          the templates more fully we recommend
-          <a href="{{ docs_url }}" target="_blank">reading the documentation</a>.</p>
-        <p><strong>Homepage:</strong> This is for choosing a predefined layout for
-           the modules that appear on your homepage.</p>
+        {% set about_url = h.url_for(controller='home', action='about') %}
+        {% set home_url = h.url_for(controller='home', action='index') %}
+        {% set docs_url = "http://docs.ckan.org/en/{0}/theming".format(g.ckan_doc_version) %}
+        {% trans %}
+          <p><strong>Site Title:</strong> This is the title of this CKAN instance
+            It appears in various places throughout CKAN.</p>
+          <p><strong>Style:</strong> Choose from a list of simple variations of
+            the main colour scheme to get a very quick custom theme working.</p>
+          <p><strong>Site Tag Logo:</strong> This is the logo that appears in the
+            header of all the CKAN instance templates.</p>
+          <p><strong>About:</strong> This text will appear on this CKAN instances
+            <a href="{{ about_url }}">about page</a>.</p>
+          <p><strong>Intro Text:</strong> This text will appear on this CKAN instances
+            <a href="{{ home_url }}">home page</a> as a welcome to visitors.</p>
+          <p><strong>Custom CSS:</strong> This is a block of CSS that appears in
+            <code>&lt;head&gt;</code> tag of every page. If you wish to customize
+            the templates more fully we recommend
+            <a href="{{ docs_url }}" target="_blank">reading the documentation</a>.</p>
+          <p><strong>Homepage:</strong> This is for choosing a predefined layout for
+            the modules that appear on your homepage.</p>
       {% endtrans %}
-      {% endblock %}
+    {% endblock %}
     </div>
   </div>
 {% endblock %}
diff --git a/ckan/templates/admin/index.html b/ckan/templates/admin/index.html
index 498da63d440..c313bf32573 100644
--- a/ckan/templates/admin/index.html
+++ b/ckan/templates/admin/index.html
@@ -2,7 +2,7 @@
 
 {% block primary_content_inner %}
   <ul class="user-list">
-    {% for user in c.sysadmins %}
+    {% for user in sysadmins %}
       <li>{{ h.linked_user(user) }}</li>
     {% endfor %}
   </ul>
diff --git a/ckan/templates/admin/trash.html b/ckan/templates/admin/trash.html
index 2484eb7cefd..108ce9e6b35 100644
--- a/ckan/templates/admin/trash.html
+++ b/ckan/templates/admin/trash.html
@@ -4,7 +4,7 @@
   {% set truncate = truncate or 180 %}
   {% set truncate_title = truncate_title or 80 %}
   <ul class="user-list">
-    {% for pkg in c.deleted_packages %}
+    {% for pkg in deleted_packages %}
       {% set title = pkg.title or pkg.name %}
       <li>{{ h.link_to(h.truncate(title, truncate_title), h.url_for(controller='package', action='read', id=pkg.name)) }}</li>
     {% endfor %}
@@ -35,4 +35,3 @@ <h2 class="module-heading">
     </div>
   </div>
 {% endblock %}
-
diff --git a/ckan/tests/legacy/functional/test_admin.py b/ckan/tests/legacy/functional/test_admin.py
index 7737f036650..74c79477653 100644
--- a/ckan/tests/legacy/functional/test_admin.py
+++ b/ckan/tests/legacy/functional/test_admin.py
@@ -15,7 +15,7 @@ def teardown_class(self):
 
     #test that only sysadmins can access the /ckan-admin page
     def test_index(self):
-        url = url_for('ckanadmin', action='index')
+        url = url_for('admin.index')
         response = self.app.get(url, status=[403])
         # random username
         response = self.app.get(url, status=[403],
@@ -25,4 +25,3 @@ def test_index(self):
         response = self.app.get(url,
                 extra_environ={'REMOTE_USER': username})
         assert 'Administration' in response, response
-
diff --git a/ckan/views/admin.py b/ckan/views/admin.py
new file mode 100644
index 00000000000..9b86e8c5d22
--- /dev/null
+++ b/ckan/views/admin.py
@@ -0,0 +1,213 @@
+# encoding: utf-8
+
+import logging
+
+from ckan.controllers.home import CACHE_PARAMETERS
+from flask import Blueprint
+from flask.views import MethodView
+
+import ckan.lib.app_globals as app_globals
+import ckan.lib.base as base
+import ckan.lib.helpers as h
+import ckan.lib.navl.dictization_functions as dict_fns
+import ckan.logic as logic
+import ckan.model as model
+from ckan.common import g, _, config, request
+
+log = logging.getLogger(__name__)
+
+admin = Blueprint(u'admin', __name__, url_prefix=u'/ckan-admin')
+
+
+def _get_sysadmins():
+    q = model.Session.query(model.User).filter(model.User.sysadmin.is_(True),
+                                               model.User.state == u'active')
+    return q
+
+
+def _get_config_options():
+    styles = [{
+        u'text': u'Default',
+        u'value': u'/base/css/main.css'
+    }, {
+        u'text': u'Red',
+        u'value': u'/base/css/red.css'
+    }, {
+        u'text': u'Green',
+        u'value': u'/base/css/green.css'
+    }, {
+        u'text': u'Maroon',
+        u'value': u'/base/css/maroon.css'
+    }, {
+        u'text': u'Fuchsia',
+        u'value': u'/base/css/fuchsia.css'
+    }]
+
+    homepages = [{
+        u'value': u'1',
+        u'text': (u'Introductory area, search, featured'
+                  u' group and featured organization')
+    }, {
+        u'value': u'2',
+        u'text': (u'Search, stats, introductory area, '
+                  u'featured organization and featured group')
+    }, {
+        u'value': u'3',
+        u'text': u'Search, introductory area and stats'
+    }]
+
+    return dict(styles=styles, homepages=homepages)
+
+
+def _get_config_items():
+    return [
+        u'ckan.site_title', u'ckan.main_css', u'ckan.site_description',
+        u'ckan.site_logo', u'ckan.site_about', u'ckan.site_intro_text',
+        u'ckan.site_custom_css', u'ckan.homepage_style'
+    ]
+
+
+@admin.before_request
+def before_request():
+    try:
+        context = dict(model=model, user=g.user, auth_user_obj=g.userobj)
+        logic.check_access(u'sysadmin', context)
+    except logic.NotAuthorized:
+        base.abort(403, _(u'Need to be system administrator to administer'))
+
+
+def index():
+    data = dict(sysadmins=[a.name for a in _get_sysadmins()])
+    return base.render(u'admin/index.html', extra_vars=data)
+
+
+class ResetConfigView(MethodView):
+    def get(self):
+        if u'cancel' in request.args:
+            return h.redirect_to(u'admin.config')
+        return base.render(u'admin/confirm_reset.html', extra_vars={})
+
+    def post(self):
+        # remove sys info items
+        for item in _get_config_items():
+            model.delete_system_info(item)
+        # reset to values in config
+        app_globals.reset()
+        return h.redirect_to(u'admin.config')
+
+
+class ConfigView(MethodView):
+    def get(self):
+        items = _get_config_options()
+        schema = logic.schema.update_configuration_schema()
+        data = {}
+        for key in schema:
+            data[key] = config.get(key)
+
+        vars = dict(data=data, errors={}, **items)
+
+        return base.render(u'admin/config.html', extra_vars=vars)
+
+    def post(self):
+        try:
+            # really?
+            data_dict = logic.clean_dict(
+                dict_fns.unflatten(
+                    logic.tuplize_dict(
+                        logic.parse_params(
+                            request.form, ignore_keys=CACHE_PARAMETERS))))
+            del data_dict['save']
+            data = logic.get_action(u'config_option_update')({
+                u'user': g.user
+            }, data_dict)
+
+        except logic.ValidationError as e:
+            items = _get_config_options()
+            data = request.form
+            errors = e.error_dict
+            error_summary = e.error_summary
+            vars = dict(
+                data=data,
+                errors=errors,
+                error_summary=error_summary,
+                form_items=items,
+                **items)
+            return base.render(u'admin/config.html', extra_vars=vars)
+
+        return h.redirect_to(u'admin.config')
+
+
+class TrashView(MethodView):
+    def __init__(self):
+        self.deleted_packages = model.Session.query(
+            model.Package).filter_by(state=model.State.DELETED)
+
+    def get(self):
+        data = dict(deleted_packages=self.deleted_packages)
+        return base.render(u'admin/trash.html', extra_vars=data)
+
+    def post(self):
+        deleted_revisions = model.Session.query(
+            model.Revision).filter_by(state=model.State.DELETED)
+        # NB: we repeat retrieval of of revisions
+        # this is obviously inefficient (but probably not *that* bad)
+        # but has to be done to avoid (odd) sqlalchemy errors (when doing
+        # purge packages) of form: "this object already exists in the
+        # session"
+        msgs = []
+        if (u'purge-packages' in request.form) or (
+                u'purge-revisions' in request.form):
+            if u'purge-packages' in request.form:
+                revs_to_purge = []
+                for pkg in self.deleted_packages:
+                    revisions = [x[0] for x in pkg.all_related_revisions]
+                    # ensure no accidental purging of other(non-deleted)
+                    # packages initially just avoided purging revisions
+                    # where non-deleted packages were affected
+                    # however this lead to confusing outcomes e.g.
+                    # we succesfully deleted revision in which package
+                    # was deleted (so package now active again) but no
+                    # other revisions
+                    problem = False
+                    for r in revisions:
+                        affected_pkgs = set(r.packages).\
+                            difference(set(self.deleted_packages))
+                        if affected_pkgs:
+                            msg = _(u'Cannot purge package %s as '
+                                    u'associated revision %s includes '
+                                    u'non-deleted packages %s')
+                            msg = msg % (pkg.id, r.id,
+                                         [pkg.id for r in affected_pkgs])
+                            msgs.append(msg)
+                            problem = True
+                            break
+                    if not problem:
+                        revs_to_purge += [r.id for r in revisions]
+                model.Session.remove()
+            else:
+                revs_to_purge = [rev.id for rev in deleted_revisions]
+            revs_to_purge = list(set(revs_to_purge))
+            for id in revs_to_purge:
+                revision = model.Session.query(model.Revision).get(id)
+                try:
+                    # TODO deleting the head revision corrupts the edit
+                    # page Ensure that whatever 'head' pointer is used
+                    # gets moved down to the next revision
+                    model.repo.purge_revision(revision, leave_record=False)
+                except Exception, inst:
+                    msg = _(u'Problem purging revision %s: %s') % (id, inst)
+                    msgs.append(msg)
+            h.flash_success(_(u'Purge complete'))
+        else:
+            msgs.append(_(u'Action not implemented.'))
+
+        for msg in msgs:
+            h.flash_error(msg)
+        return h.redirect_to(u'admin.trash')
+
+
+admin.add_url_rule(u'/', view_func=index, strict_slashes=False)
+admin.add_url_rule(
+    u'/reset_config', view_func=ResetConfigView.as_view(str(u'reset_config')))
+admin.add_url_rule(u'/config', view_func=ConfigView.as_view(str(u'config')))
+admin.add_url_rule(u'/trash', view_func=TrashView.as_view(str(u'trash')))