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

Mon Jul 30 10:15:07 UTC 2007

[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?][2]* ([UrlBuilderAdmin?][3], [UrlBuilderProject?][4]) with new [UrlScheme?][1] object that uses Routes 

    * We ensure new scheme is backwards compatible 

  * Update all users of [UrlBuilderProject?][4] 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/UrlBuilder
   [3]: http://project.knowledgeforge.net/kforge/trac/wiki/UrlBuilderAdmin
   [4]: http://project.knowledgeforge.net/kforge/trac/wiki/UrlBuilderProject

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