From d45c4d8caf17c37cde65cd3d07a053f29fd597aa Mon Sep 17 00:00:00 2001
From: Ayub khan <ayub.khan@arbisoft.com>
Date: Mon, 2 May 2016 14:23:39 +0500
Subject: [PATCH] Revert "Implement 'from_string_or_404' in utils"

---
 cms/djangoapps/contentstore/views/course.py   |  6 ++--
 common/djangoapps/util/course_key_utils.py    | 30 -----------------
 .../util/tests/test_course_key_utils.py       | 32 -------------------
 3 files changed, 4 insertions(+), 64 deletions(-)
 delete mode 100644 common/djangoapps/util/course_key_utils.py
 delete mode 100644 common/djangoapps/util/tests/test_course_key_utils.py

diff --git a/cms/djangoapps/contentstore/views/course.py b/cms/djangoapps/contentstore/views/course.py
index 4450031b62..791db5991e 100644
--- a/cms/djangoapps/contentstore/views/course.py
+++ b/cms/djangoapps/contentstore/views/course.py
@@ -91,7 +91,6 @@ from util.organizations_helpers import (
     organizations_enabled,
 )
 from util.string_utils import _has_non_ascii_characters
-from util.course_key_utils import from_string_or_404
 from xmodule.contentstore.content import StaticContent
 from xmodule.course_module import CourseFields
 from xmodule.course_module import DEFAULT_START_DATE
@@ -875,7 +874,10 @@ def course_info_handler(request, course_key_string):
     GET
         html: return html for editing the course info handouts and updates.
     """
-    course_key = from_string_or_404(course_key_string)
+    try:
+        course_key = CourseKey.from_string(course_key_string)
+    except InvalidKeyError:
+        raise Http404
 
     with modulestore().bulk_operations(course_key):
         course_module = get_course_and_check_access(course_key, request.user)
diff --git a/common/djangoapps/util/course_key_utils.py b/common/djangoapps/util/course_key_utils.py
deleted file mode 100644
index ab58a6558d..0000000000
--- a/common/djangoapps/util/course_key_utils.py
+++ /dev/null
@@ -1,30 +0,0 @@
-"""
-Convenience methods for working with course objects
-"""
-from django.http import Http404
-from opaque_keys import InvalidKeyError
-from opaque_keys.edx.keys import CourseKey
-
-
-def from_string_or_404(course_key_string):
-    """
-    Gets CourseKey from the string passed as parameter.
-
-    Parses course key from string(containing course key) or raises 404 if the string's format is invalid.
-
-    Arguments:
-        course_key_string(str): It is string containing the course key
-
-    Returns:
-        CourseKey: A key that uniquely identifies a course
-
-    Raises:
-        HTTP404: A 404 not found exception will be thrown if course_key_string's format is invalid
-
-    """
-    try:
-        course_key = CourseKey.from_string(course_key_string)
-    except InvalidKeyError:
-        raise Http404
-
-    return course_key
diff --git a/common/djangoapps/util/tests/test_course_key_utils.py b/common/djangoapps/util/tests/test_course_key_utils.py
deleted file mode 100644
index efc88a09ee..0000000000
--- a/common/djangoapps/util/tests/test_course_key_utils.py
+++ /dev/null
@@ -1,32 +0,0 @@
-"""
-Tests for util.course_key_utils
-"""
-from nose.tools import assert_equals, assert_raises  # pylint: disable=no-name-in-module
-from util.course_key_utils import from_string_or_404
-from opaque_keys.edx.keys import CourseKey
-from django.http import Http404
-
-
-def test_from_string_or_404():
-
-    #testing with split style course keys
-    assert_raises(
-        Http404,
-        from_string_or_404,
-        "/some.invalid.key/course-v1:TTT+CS01+2015_T0"
-    )
-    assert_equals(
-        CourseKey.from_string("course-v1:TTT+CS01+2015_T0"),
-        from_string_or_404("course-v1:TTT+CS01+2015_T0")
-    )
-
-    #testing with mongo style course keys
-    assert_raises(
-        Http404,
-        from_string_or_404,
-        "/some.invalid.key/TTT/CS01/2015_T0"
-    )
-    assert_equals(
-        CourseKey.from_string("TTT/CS01/2015_T0"),
-        from_string_or_404("TTT/CS01/2015_T0")
-    )