damned-lies r1124 - in branches/djamnedlies: . stats stats/management/commands

[stephaner svn gnome org]

Author: stephaner
Date: Sun Nov  2 14:41:36 2008
New Revision: 1124
URL: http://svn.gnome.org/viewvc/damned-lies?rev=1124&view=rev

Log:
2008-11-02  StÃphane Raimbault  <stephane raimbault gmail com>

	* README: Added information about the ANSI_QUOTES setting
	server-side.
	* stats/management/commands/migrate.py: Removed stupid pdb call
	from my previous commit.
	* stats/urls.py: Added comment.


Modified:
   branches/djamnedlies/ChangeLog
   branches/djamnedlies/README
   branches/djamnedlies/stats/management/commands/migrate.py
   branches/djamnedlies/stats/urls.py

Modified: branches/djamnedlies/README
==============================================================================
--- branches/djamnedlies/README	(original)
+++ branches/djamnedlies/README	Sun Nov  2 14:41:36 2008
@@ -53,6 +53,9 @@
     "init_command": "SET storage_engine=INNODB"
 }
 
-
-
-
+Grep for ANSI_QUOTES in the source code to find the models.py which
+use a hack to workaround the double quotes interpretation in MySQL.
+The best solution is to run the MySQL server with the ANSI_QUOTES
+mode: http://dev.mysql.com/doc/refman/5.0/en/server-sql-mode.html
+(sql-mode="ANSI_QUOTES" in my.cnf) but it can be dangerous for other
+applications.

Modified: branches/djamnedlies/stats/management/commands/migrate.py
==============================================================================
--- branches/djamnedlies/stats/management/commands/migrate.py	(original)
+++ branches/djamnedlies/stats/management/commands/migrate.py	Sun Nov  2 14:41:36 2008
@@ -188,7 +188,6 @@
 
     def migrate_stats(self):
         """ This method migrate statistics if there is an old_statistics table (from previous DL)"""
-        import pdb; pdb.set_trace()
         try:
             mysql_settings = settings.MYSQL_SETTINGS
             # Import just after dict access to avoid a useless import

Modified: branches/djamnedlies/stats/urls.py
==============================================================================
--- branches/djamnedlies/stats/urls.py	(original)
+++ branches/djamnedlies/stats/urls.py	Sun Nov  2 14:41:36 2008
@@ -6,6 +6,8 @@
 from django.contrib import admin
 admin.autodiscover()
 
+# FIXME All these URL must be branched under '/' not '/stats' to
+# maintain an URL compatibility with the old DL
 urlpatterns = patterns('stats.views',
     (r'^$', 'index'),
     url(r'^teams/$', 'teams', name='teams'),
@@ -24,12 +26,14 @@
 people_dict = {
     'queryset': Person.objects.all(),
 }
+
 urlpatterns += patterns('django.views.generic',
     url(r'^people/(?P<object_id>\d+)/$', 'list_detail.object_detail',
         dict(people_dict), 'person_view'),
+    
 )
 
-    # Static files (should be replaced by web server redirection in production
+# Static files (should be replaced by web server redirection in production
 urlpatterns += patterns('',
     (r'^POT/(?P<path>.*)$', 'django.views.static.serve',
      {'document_root': settings.POTDIR}),