Changeset 5727

Timestamp:

11/19/08 16:39:52 (2 months ago)

Author:

faide

Message:

Applied patch from #1996 Thanks to Peter Russell to have reported this nice implementation from Rick Copeland. See http://blog.pythonisito.com/2008/07/restfulness-in-turbogears.html for more details...

Files:

• branches/1.1/turbogears/controllers.py (modified) (2 diffs)
• branches/1.1/turbogears/tests/test_restmethod.py (added)

branches/1.1/turbogears/controllers.py

@@ -505 +505 @@
-
-
+class ExposedDescriptor(object):
+    """Descriptor used by RESTMethod to tell if it is exposed
+    """
+    def __get__(self, obj, cls=None):
+        """Return True if the object has a method for the HTTP method of the current request
+        """
+        if cls is None: cls = obj
+        cp_methodname = cherrypy.request.method
+        methodname = cp_methodname.lower()
+        method = getattr(cls, methodname, None)
+        if callable(method) and getattr(method, 'exposed', False):
+            return True
+        raise cherrypy.HTTPError(405, '%s not allowed on %s' % (
+            cp_methodname, cherrypy.request.browser_url))
+
+
+class RESTMethod(object):
+    """Allows REST style dispatch based on different HTTP methods
+
+    For an example usage see turbogears.tests.test_restmethod
+    """
+    exposed = ExposedDescriptor()
+
+    def __init__(self, *l, **kw):
+        methodname = cherrypy.request.method.lower()
+        self.result = getattr(self, methodname)(*l, **kw)
+
+    def __iter__(self):
+        return iter(self.result)
+
+
-def url(tgpath, tgparams=None, **kw):
-    """Computes URLs.
@@ -511 +542 @@
-    with a "/"), the server.webpath, SCRIPT_NAME and the approot of the
-    application are prepended to the path. In order for the approot to
+
-    be detected properly, the root object should extend
+
-    controllers.RootController.
-