Analysis “Date Completed” field blank for Course

  • Resolved marcusjwilson

    (@marcusjwilson)


    5 years, 1 month ago

    Hi –

    We’re currently testing Sensei LMS on our site. Each of our courses only has one Lesson/Quiz. When a User on the site completes the single Lesson/Quiz for the Course successfully, the Lesson gets a “Passed” status and “Date Completed” date for the User.

    However, on the Analysis page (i.e. /wp-admin/admin.php?page=sensei_analysis&user_id=10) we would expect the Course associated with the Lesson/Quiz to also show a “Date Completed” date when all Lessons/Quizzes for the Course are completed, but this remains blank.

    What needs to happen in Sensei LMS for the Course to show a “Completed Date”? We have the Courses set to complete “When all Lessons for the Course are complete” in Settings.

    The only thing I can think is that our Lessons and Courses are published as “Private” on the site so only logged in members can see them. Could that be causing Courses to not get a Completed date? Where would the “Date completed” date be stored in the WP database for Courses?

    Any advice gratefully received!
    Marcus

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter marcusjwilson

    (@marcusjwilson)

    Having reviewed the Sensei plugin code, I can confirm that Lessons that are Privately Published are not taken into account when marking Courses as “Completed”.

    This relates to the code in:
    wp-content/plugins/woothemes-sensei/includes/class-sensei-utils.php:1584

    $lesson_ids = Sensei()->course->course_lessons( $course_id, 'publish', 'ids' );

    When the plugin starts to count completed Lessons it looks only for Published ones. However, it seems to me that Lessons or Courses should be able to be published Privately, and still be marked as “Completed” by Sensei.

    Is this something you would consider changing in a future version, so Course Lessons with both publish and private status are considered here?

    Thanks
    Marcus

    Plugin Author Donna Peplinskie (a11n)

    (@donnapep)

    Hi Marcus,

    Is this something you would consider changing in a future version, so Course Lessons with both publish and private status are considered here?

    I would actually consider this a bug, as I can see that elsewhere in our codebase we are looking for lessons with a status of both publish and private.

    I don’t mean to pass the buck, but would you be interested in submitting a pull request for this on GitHub? It’s the best way to ensure this goes out in the next release.

    Cheers!

    Plugin Support Dan (a11n)

    (@drawmyface)

    There is a Github issue open for this here: https://github.com/Automattic/sensei/issues/2802

    So I’m going to mark this thread resolved and suggest subscribing to the Github issue instead.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Analysis “Date Completed” field blank for Course’ is closed to new replies.