dealy14 / LearningFox-LMS

0 stars 0 forks source link

Merge "course already completed" fix into Cosmos content #29

Closed ccalvert closed 12 years ago

ccalvert commented 12 years ago

When launching a course in the Cosmos content version, we were getting an error "course already completed". Murali fixed this in the company version, and we need to backport that change to the content version.

Murali made changes to site_conf/components/content_enrollment.php to fix this. The changes are only in the production copy of the file, not the repository version. We need to:

  1. Merge the production version of content_enrollment.php into the "cosmos" branch.
  2. Merge the changes that Murali made into the same file in the "cosmos_content" branch and then push the file to the server.
ryanammons commented 12 years ago

The backport/merge of the fix was done; however, this did not solve the problem completely. I corrected the student-course registration code within the API module to ensure that the student's data was properly inserted into the DB for successful course registration.