ok, that makes sense. so much of our code assumes these 2 uses are the same
:-)


On Thu, Aug 8, 2013 at 10:42 AM, Calen Pennington
<notifications@github.com>wrote:

> In common/lib/xmodule/xmodule/modulestore/parsers.py:
>
> >
> >
> >  def parse_url(string):
> >      """
> > -    A url must begin with 'edx://' (case-insensitive match),
> > +    A url can begin with 'edx://' (case-insensitive match),
>
> Yes, I agree. However, I think that really the right thing to do to remove
> the urls-in-urls thing is to make our url maps understand the portions of
> the id. (In otherwords, rather than go through this parse_url function,
> they'd be in our urls as actual first class parameters). This function
> would only be used for parsing urls that appear from the outside as
> monolithic string ids.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/601/files#r5659233>
> .
>