Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #16247 from edx/pwnage101/avoid_implicit__in_query_PLAT-1525

Browse Source 
Use singular getter function where only one requirement is expected
This commit is contained in:
Troy Sankey
2017-10-17 10:43:32 -04:00
committed by GitHub
parent 77c1ca51d8 2eba1b5122
commit e507dac87c
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
openedx/core/djangoapps/credit/api/eligibility.py
View File

@@ -297,7 +297,7 @@ def remove_credit_requirement_status(username, course_key, req_namespace, req_na
"""
# Find the requirement we're trying to remove
req_to_remove = CreditRequirement.get_course_requirements(course_key, namespace=req_namespace, name=req_name)
req_to_remove = CreditRequirement.get_course_requirement(course_key, req_namespace, req_name)
# If we can't find the requirement, then the most likely explanation
# is that there was a lag removing the credit requirements after the course

2
openedx/core/djangoapps/credit/models.py
View File

@@ -385,7 +385,7 @@ class CreditRequirement(TimeStampedModel):
name(str): Name of credit course requirement
Returns:
CreditRequirement object if exists
CreditRequirement object if exists, None otherwise.
"""
try: