[kforge-dev] Changeset [1127]: [major]: change to new url layout system using new Routes based UrlScheme ...

[10: Timeline]

[major]: change to new url layout system using new Routes based [UrlScheme?][1] object. 

Rather than list all changes we note the major ones and special cases: 

  * Completely overhaul kforge.url and replace UrlBuilder* ([UrlBuilderAdmin?][2], [UrlBuilderProject?][3]) with new [UrlScheme?][1] object that uses Routes 

    * We ensure new scheme is backwards compatible 

  * Update all users of [UrlBuilderProject?][3] to use [UrlScheme?][1]
  * Update apacheconfig to use [UrlScheme?][1] when generating apache config 

    * This necessitates a major change to apache config generation: in new system just generate file that will be included in virtual host rather than full virtual host 

  * WARNING: have not yet updated media vhost links as these are hardcoded in templates to use a distinct media host location 
  * Update all of django.apps.kui.test code to use [UrlScheme?][1].url_for (except admin items) 
  * WARNING: however have not updated most of django system because: 

    * (minor) django uses its own regex based system that is hard to integrate with new approach 
    * (major and main reason) currently a very large number of url pathes are hard-coded in templates and in views. 

  * 8 tests currently failing all for reasons unrelated to this change (4 in kforge.handlers, one due to lack of local smtp support, one due to bad local apache config and 2 from django.apps.kui.test.admin.domainObject which are yet to be resolved). 
  * misc: 

    * remove unused tests in kforge/django/apps/kui/test/service.py 

    [1]: http://project.knowledgeforge.net/kforge/trac/wiki/UrlScheme
    [2]: http://project.knowledgeforge.net/kforge/trac/wiki/UrlBuilderAdmin
    [3]: http://project.knowledgeforge.net/kforge/trac/wiki/UrlBuilderProject

URL: http://project.knowledgeforge.net/kforge/trac/changeset/1127