Cleaning up the code and renaming some classes to better reflect what it is
Cleaning up the code and renaming some classes to better reflect what it is

--- a/ckanext/example/controller_plugin.py
+++ b/ckanext/example/controller_plugin.py
@@ -28,38 +28,20 @@
 
 log = logging.getLogger(__name__)
 
-geographic_granularity = [('', ''),
-                          ('national', 'national'),
-                          ('regional', 'regional'),
-                          ('local authority', 'local authority'),
-                          ('ward', 'ward'),
-                          ('point', 'point'),
-                          ('other', 'other - please specify')]
-
-update_frequency = [('', ''),
-                    ('never', 'never'),
-                    ('discontinued', 'discontinued'),
-                    ('annual', 'annual'),
-                    ('quarterly', 'quarterly'),
-                    ('monthly', 'monthly'),
-                    ('other', 'other - please specify')]
-
-temporal_granularity = [("",""),
-                       ("year","year"),
-                       ("quarter","quarter"),
-                       ("month","month"),
-                       ("week","week"),
-                       ("day","day"),
-                       ("hour","hour"),
-                       ("point","point"),
-                       ("other","other - please specify")]
 
 
-class ExamplePackageController(SingletonPlugin):
+class ExampleDatasetForm(SingletonPlugin):
+    """
+
+    """
 
     implements(IDatasetForm, inherit=True)
 
     def package_form(self):
+        """
+        Returns a string representing the location of the template to be
+        rendered.  e.g. "package/new_package_form.html".
+        """        
         return 'controller/package_plugin.html'
 
     def is_fallback(self):
@@ -85,14 +67,12 @@
         """
         return ["example"]
 
-    def _setup_template_variables(self, context, data_dict=None):
+    def setup_template_variables(self, context, data_dict=None):
+        """
+        Add variables to c just prior to the template being rendered.
+        """        
         c.licences = [('', '')] + model.Package.get_license_options()
-        c.geographic_granularity = geographic_granularity
-        c.update_frequency = update_frequency
-        c.temporal_granularity = temporal_granularity 
-
         c.publishers = self.get_publishers()
-
         c.is_sysadmin = Authorizer().is_sysadmin(c.user)
         c.resource_columns = model.Resource.get_columns()
 
@@ -102,8 +82,11 @@
             c.auth_for_change_state = Authorizer().am_authorized(
                 c, model.Action.CHANGE_STATE, pkg)
 
-    def _form_to_db_schema(self):
-
+    def form_to_db_schema(self):
+        """
+        Returns the schema for mapping package data from a form to a format
+        suitable for the database.
+        """
         schema = {
             'title': [not_empty, unicode],
             'name': [not_empty, unicode, val.name_validator, val.package_name_validator],
@@ -112,16 +95,6 @@
             'date_released': [date_to_db, convert_to_extras],
             'date_updated': [date_to_db, convert_to_extras],
             'date_update_future': [date_to_db, convert_to_extras],
-            'update_frequency': [use_other, unicode, convert_to_extras],
-            'update_frequency-other': [],
-            'precision': [unicode, convert_to_extras],
-            'geographic_granularity': [use_other, unicode, convert_to_extras],
-            'geographic_granularity-other': [],
-            'geographic_coverage': [ignore_missing, convert_geographic_to_db, convert_to_extras],
-            'temporal_granularity': [use_other, unicode, convert_to_extras],
-            'temporal_granularity-other': [],
-            'temporal_coverage-from': [date_to_db, convert_to_extras],
-            'temporal_coverage-to': [date_to_db, convert_to_extras],
             'url': [unicode],
             'taxonomy_url': [unicode, convert_to_extras],
 
@@ -144,18 +117,16 @@
         }
         return schema
     
-    def _db_to_form_schema(data):
+    def db_to_form_schema(data):
+        """
+        Returns the schema for mapping package data from the database into a
+        format suitable for the form (optional)
+        """
         schema = {
             'date_released': [convert_from_extras, ignore_missing, date_to_form],
             'date_updated': [convert_from_extras, ignore_missing, date_to_form],
             'date_update_future': [convert_from_extras, ignore_missing, date_to_form],
-            'update_frequency': [convert_from_extras, ignore_missing, extract_other(update_frequency)],
             'precision': [convert_from_extras, ignore_missing],
-            'geographic_granularity': [convert_from_extras, ignore_missing, extract_other(geographic_granularity)],
-            'geographic_coverage': [convert_from_extras, ignore_missing, convert_geographic_to_form],
-            'temporal_granularity': [convert_from_extras, ignore_missing, extract_other(temporal_granularity)],
-            'temporal_coverage-from': [convert_from_extras, ignore_missing, date_to_form],
-            'temporal_coverage-to': [convert_from_extras, ignore_missing, date_to_form],
             'taxonomy_url': [convert_from_extras, ignore_missing],
 
             'resources': default_schema.default_resource_schema(),
@@ -177,47 +148,16 @@
         }
         return schema
 
-    def _check_data_dict(self, data_dict):
+    def check_data_dict(self, data_dict):
+        """
+        Check if the return data is correct.
+
+        raise a DataError if not.
+        """
         return
 
     def get_publishers(self):
+        """
+        """
         return [('pub1', 'pub2')]
 
-
-def use_other(key, data, errors, context):
-
-    other_key = key[-1] + '-other'
-    other_value = data.get((other_key,), '').strip()
-    if other_value:
-        data[key] = other_value
-
-def extract_other(option_list):
-
-    def other(key, data, errors, context):
-        value = data[key]
-        if value in dict(option_list).keys():
-            return
-        elif value is missing:
-            data[key] = ''
-            return
-        else:
-            data[key] = 'other'
-            other_key = key[-1] + '-other'
-            data[(other_key,)] = value
-    return other
-            
-def convert_geographic_to_db(value, context):
-
-    if isinstance(value, list):
-        regions = value
-    elif value:
-        regions = [value]
-    else:
-        regions = []
-        
-    return GeoCoverageType.get_instance().form_to_db(regions)
-
-def convert_geographic_to_form(value, context):
-
-    return GeoCoverageType.get_instance().db_to_form(value)
-

--- a/ckanext/example/theme/templates/controller/package_plugin.html
+++ b/ckanext/example/theme/templates/controller/package_plugin.html
@@ -43,7 +43,7 @@
     <dt><label class="field_opt" for="date_released">Date released</label></dt>
     <dd><input id="date_released" name="date_released" size="40" type="text" value="${data.get('date_released', '')}" /></dd>
     <dd class="instructions basic">The date of the official release of the initial version of the dataset</dd>
-    <dd class="instructions further">This is probably not the date that it is uploaded to data.gov.uk. Be careful not to confuse a new 'version' of some data with a new dataset covering another time period or geographic area.</dd>
+    <dd class="instructions further">This is the date that the data was officially released</dd>
     <dd class="hints">DD/MM/YYYY</dd>
     <dd class="field_error" py:if="errors.get('date_released', '')">${errors.get('date_released', '')}</dd>
 
@@ -81,73 +81,6 @@
     <dd><input id="precision" name="precision" size="40" type="text" value="${data.get('precision', '')}"/></dd>
     <dd class="instructions basic">Indicate the level of precision in the data, to avoid over-interpretation.</dd>
     <dd class="hints">e.g. 'per cent to two decimal places' or 'as supplied by respondents'</dd>
-
-    <dt><label class="field_opt" for="geographic_granularity">Geographic granularity</label></dt>
-    <dd>
-      <select class="short" id="geographic_granularity" name="geographic_granularity">
-        <py:for each="geog_name, geog_desc in c.geographic_granularity">
-          <option value="${geog_name}" py:attrs="{'selected': 'selected' if data.get('geographic_granularity', '') == geog_name else None}" >
-            ${geog_desc}
-          </option>
-        </py:for>
-      </select>
-      <label class="inline" for="geographic_granularity-other">Other: 
-        <input class="medium-width" id="geographic_granularity-other" 
-        name="geographic_granularity-other" type="text" value="${data.get('geographic_granularity-other', '')}"/>
-      </label>
-    </dd>
-    <dd class="instructions basic">The lowest level of geographic detail</dd>
-    <dd class="instructions further">This should give the lowest level of geographic detail given in the dataset if it is aggregated. If the data is not aggregated, and so the dataset goes down to the level of the entities being reported on (such as school, hospital, or police station), use 'point'. If none of the choices is appropriate or the granularity varies, please specify in the 'other' element.</dd>
-    <dd class="field_error" py:if="errors.get('geographic_granularity', '')">${errors.get('geographic_granularity', '')}</dd>
-
-    <dt><label class="field_opt" for="geographic_coverage">Geographic coverage</label></dt>
-    <dd py:with="geographic_coverage=data.get('geographic_coverage', [])">
-      <label for="england">
-        <input id="england" name="geographic_coverage" size="40" type="checkbox" 
-        value="england" py:attrs="{'checked': 'checked' if 'england' in geographic_coverage else None}"/> England</label>
-      <label for="scotland">
-        <input id="scotland" name="geographic_coverage" size="40" type="checkbox" 
-        value="scotland" py:attrs="{'checked': 'checked' if 'scotland' in geographic_coverage else None}"/> Scotland</label>
-      <label for="wales">
-        <input id="wales" name="geographic_coverage" size="40" type="checkbox" 
-        value="wales" py:attrs="{'checked': 'checked' if 'wales' in geographic_coverage else None}"/> Wales</label>
-      <label for="northern_ireland">
-        <input id="northern_ireland" name="geographic_coverage" size="40" type="checkbox" 
-        value="northern_ireland" py:attrs="{'checked': 'checked' if 'northern_ireland' in geographic_coverage else None}"/> Northern Ireland</label>
-      <label for="overseas">
-        <input id="overseas" name="geographic_coverage" size="40" type="checkbox" 
-        value="overseas" py:attrs="{'checked': 'checked' if 'overseas' in geographic_coverage else None}"/> Overseas</label>
-      <label for="global">
-        <input id="global" name="geographic_coverage" size="40" type="checkbox" 
-        value="global" py:attrs="{'checked': 'checked' if 'global' in geographic_coverage else None}"/> Global</label>
-    </dd>
-    <dd class="instructions basic">The geographic coverage of this dataset.</dd>
-    <dd class="instructions further">Where a dataset covers multiple areas, the system will automatically group these (e.g. 'England', 'Scotland' and 'Wales' all being selected would be shown as 'Great Britain').</dd>
-
-    <dt><label class="field_opt" for="temporal_granularity">Temporal granularity</label></dt>
-    <dd>
-      <select class="short" id="temporal_granularity" name="temporal_granularity">
-        <py:for each="temporal_name, temporal_desc in c.temporal_granularity">
-          <option value="${temporal_name}" py:attrs="{'selected': 'selected' if data.get('temporal_granularity', '') == temporal_name else None}" >
-            ${temporal_name}
-          </option>
-        </py:for>
-      </select>
-      <label class="inline" for="temporal_granularity-other">Other: 
-        <input class="medium-width" id="temporal_granularity-other" name="temporal_granularity-other" type="text" value="${data.get('temporal_granularity-other', '')}"/>
-      </label>
-    </dd>
-    <dd class="instructions basic">The lowest level of temporal detail granularity</dd>
-    <dd class="instructions further">This should give the lowest level of temporal detail given in the dataset if it is aggregated, expressed as an interval of time. If the data is not aggregated over time, and so the dataset goes down to the instants that reported events occurred (such as the timings of high and low tides), use 'point'. If none of the choices is appropriate or the granularity varies, please specify in the 'other' element.</dd>
-
-    <dt><label class="field_opt" for="temporal_coverage">Temporal coverage</label></dt>
-    <dd>
-      <input class="medium-width" id="temporal_coverage-from" name="temporal_coverage-from" type="text" value="${data.get('temporal_coverage-from', '')}" /> - 
-      <input class="medium-width" id="temporal_coverage-to" name="temporal_coverage-to" type="text" value="${data.get('temporal_coverage-to', '')}" />
-    </dd>
-    <dd class="instructions basic">The temporal coverage of this dataset.</dd>
-    <dd class="instructions further">If available, please indicate the time as well as the date. Where data covers only a single day, the 'To' sub-element can be left blank.</dd>
-    <dd class="hints">e.g. 21/03/2007 - 03/10/2009 or 07:45 31/03/2006</dd>
 
     <dt><label class="field_opt" for="url">URL</label></dt>
     <dd><input id="url" name="url" type="text" value="${data.get('url', '')}"/></dd>

file:a/setup.py -> file:b/setup.py
--- a/setup.py
+++ b/setup.py
@@ -26,7 +26,7 @@
 	"""
         [ckan.plugins]
 	    example=ckanext.example.plugin:ExamplePlugin
-        example_package=ckanext.example.controller_plugin:ExamplePackageController
+        example_package=ckanext.example.controller_plugin:ExampleDatasetForm
 
         [ckan.forms]
         example_form = ckanext.example.package_form:get_example_fieldset