From 8e4c7c878ebc4ac4c46a0baede2c2c07f1a6b483 Mon Sep 17 00:00:00 2001
From: Calen Pennington <calen.pennington@gmail.com>
Date: Wed, 11 Jul 2012 10:23:06 -0400
Subject: [PATCH] Add comment about location invalid chars

---
 common/lib/xmodule/xmodule/modulestore/__init__.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/common/lib/xmodule/xmodule/modulestore/__init__.py b/common/lib/xmodule/xmodule/modulestore/__init__.py
index 1e8ee838d7f..3930ee9ff33 100644
--- a/common/lib/xmodule/xmodule/modulestore/__init__.py
+++ b/common/lib/xmodule/xmodule/modulestore/__init__.py
@@ -19,6 +19,8 @@ URL_RE = re.compile("""
     (/(?P<revision>[^/]+))?
     """, re.VERBOSE)
 
+# TODO (cpennington): We should decide whether we want to expand the
+# list of valid characters in a location
 INVALID_CHARS = re.compile(r"[^\w.-]")
 
 _LocationBase = namedtuple('LocationBase', 'tag org course category name revision')
-- 
GitLab