Good catch. I don't think cms commands should take the deprecated form for
input. commands are run only by sysadmins which number a handful. We can
ensure they know.


On Wed, Apr 30, 2014 at 2:25 PM, Nimisha Asthagiri <notifications@github.com
> wrote:

> In cms/djangoapps/contentstore/management/commands/git_export.py:
>
> > @@ -54,7 +55,7 @@ def handle(self, *args, **options):
> >          # Rethrow GitExportError as CommandError for SystemExit
> >          try:
> >              git_export_utils.export_to_git(
> > -                args[0],
> > +                SlashSeparatedCourseKey.from_deprecated_string(args[0]),
> >                  args[1],
>
> @cahrens <https://github.com/cahrens> @dmitchell<https://github.com/dmitchell>Should management commands continue to accept deprecated strings as their
> input values? Or should they now be accepting only new Key formats?
> In other words: This change must have been made to fix a test case. But
> I'm wondering whether the test should have changed here instead of this
> code.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/3537/files#r12158058>
> .
>