From 663ea78caafc057ac4b120bea64750acfcc2e4cd Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Fri, 8 Nov 2013 16:47:56 +0100
Subject: [PATCH] [ADD] handling of slugs in ModelConverter (generation &
 parsing)

bzr revid: xmo@openerp.com-20131108154756-znkcyd32csd7vj1u
---
 openerp/http.py | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/openerp/http.py b/openerp/http.py
index b3b01277d64..e0681f0e7d1 100644
--- a/openerp/http.py
+++ b/openerp/http.py
@@ -3,10 +3,8 @@
 # OpenERP HTTP layer
 #----------------------------------------------------------
 import ast
-import cgi
 import contextlib
 import errno
-import functools
 import getpass
 import inspect
 import logging
@@ -33,7 +31,15 @@ import werkzeug.routing as routing
 
 import openerp
 from openerp.service import security, model as service_model
-from openerp.tools import config
+
+try:
+    # use python-slugify (https://github.com/un33k/python-slugify) if available
+    from slugify import slugify
+except ImportError:
+    def slugify(s, max_length=None):
+        spaceless = re.sub(r'\s+', '-', s)
+        specialless = re.sub(r'[^-_A-Za-z0-9]', '', spaceless)
+        return specialless[:max_length]
 
 _logger = logging.getLogger(__name__)
 
@@ -850,16 +856,18 @@ class ModelConverter(routing.BaseConverter):
         super(ModelConverter, self).__init__(url_map)
         self.model = model
         # TODO add support for slug in the form [A-Za-z0-9-] bla-bla-89 -> id 89
-        self.regex = '([0-9]+)'
+        self.regex = r'[A-Za-z0-9-_]*?(\d+)'
 
     def to_python(self, value):
         # TODO:
         # - raise routing.ValidationError() if no browse record can be createdm
-        # - support slug 
-        return request.registry[self.model].browse(request.cr, request.uid, int(value), context=request.context)
+        m = re.match(self.regex, value)
+        return request.registry[self.model].browse(
+            request.cr, request.uid, int(m.group(1)), context=request.context)
 
     def to_url(self, value):
-        return str(value.id)
+        [(_, name)] = value.name_get()
+        return "%s-%d" % (slugify(name), value.id)
 
     def generate(self):
         for id in request.registry[self.model].search(request.cr, request.uid, [], context=request.context):