sdwarwick

@thomasplevy – I’m always grateful for the work you do to make lifter a great product. I see providing specific opportunities for improvement as the most effective way to support those efforts!

I’ll look at a developing a pull request.

so, I have been using something very similar to these concepts for a while now. essentially hid the “mark complete” button and fired the mark_complete form using a jquery “click”

this does two things nicely – registers completion and serves the “next” lesson.

EXCEPT!! the last lesson in the course gets stuck. rather than a default for end-of-course, you just send back the same url. Of course, the second time, the mark complete button is removed since now it is considered a “completed” lesson. but still, users see the last lesson twice, which is a real bummer when it is a final exam!!

you have a very simple opportunity to add a hook in the following code for get next lesson… I am at a loss right now how to deal with this other than literally removing the mark_complete action from init and adding a custom version that provides an end-of-course option. the approach would be to detect a null on next lesson and provide an alternative

	public function get_next_lesson() {

		$parent_section = $this->get_parent_section();
		$current_position = $this->get_order();
		$next_position = $current_position + 1;

		$args = array(
			'posts_per_page' 	=> 1,
			'post_type' 		=> 'lesson',
			'nopaging' 			=> true,
			'post_status'   	=> 'publish',
			'meta_query' 		=> array(
				'relation' => 'AND',
				array(
				    'key' => '_llms_parent_section',
				    'value' => $parent_section,
				    'compare' => '=',
			    ),
			    array(
				    'key' => '_llms_order',
				    'value' => $next_position,
				    'compare' => '=',
			    )
			),
		);
		$lessons = get_posts( $args );

		//return the first one even if there for some crazy reason were more than one.
		if ( $lessons ) {
			return $lessons[0]->ID;
		} else {
			// See if there is another section after this section and get first lesson there
			$parent_course = $this->get_parent_course();
			$cursection = new LLMS_Section( $this->get_parent_section() );
			$current_position = $cursection->get_order();
			$next_position = $current_position + 1;

			$args = array(
				'post_type' 		=> 'section',
				'posts_per_page'	=> 500,
				'meta_key'			=> '_llms_order',
				'order'				=> 'ASC',
				'orderby'			=> 'meta_value_num',
				'meta_query' 		=> array(
					'relation' => 'AND',
					array(
					    'key' => '_llms_parent_course',
					    'value' => $parent_course,
					    'compare' => '=',
				    ),
				    array(
					    'key' => '_llms_order',
					    'value' => $next_position,
					    'compare' => '=',
				    )
				),
			);
			$sections = get_posts( $args );

			if ( $sections ) {
				$newsection = new LLMS_Section( $sections[0]->ID );
				$lessons = $newsection->get_children_lessons();
				if ( $lessons ) {
					return $lessons[0]->ID;
				} else {
					return false;
				}
			} else {
		return false;   // no! no! no!!   
			}
		}
	}

• This reply was modified 9 years, 2 months ago by sdwarwick.

Not sure exactly what you are looking for, however I do believe that if you have the ability to add an additional “lesson” page to your site after the quiz, it is possible to add a very small function that looks for a tag on that page and redirects immediately to another page on the site.

since lifter naturally moves to the next lesson in a course, putting this page at the end would enable the desired behavior.