| @@ -435,20 +435,24 @@ | ||
| 435 | 435 | 'auto_complete_on_video_end' |
| 436 | 436 | ]; |
| 437 | 437 | |
| 438 | 438 | if ($lesson->isQuizType()) { |
| 439 | - $quizFields = ['quiz_questions', 'passing_score', 'enable_passing_score', 'enforce_passing_score', 'hide_result']; | |
| 439 | + $quizFields = ['quiz_questions', 'passing_score', 'enable_passing_score', 'enforce_passing_score', 'hide_result', 'shuffle_options', 'shuffle_questions', 'layout']; | |
| 440 | 440 | $validFields = wp_parse_args($validFields, $quizFields); |
| 441 | 441 | } |
| 442 | 442 | |
| 443 | 443 | $meta = Arr::only($meta, $validFields); |
| 444 | 444 | |
| 445 | - $yesNoFields = ['enable_comments', 'enable_media', 'free_preview_lesson', 'require_video_completion', 'auto_complete_on_video_end', 'enable_passing_score', 'enforce_passing_score', 'hide_result']; | |
| 445 | + $yesNoFields = ['enable_comments', 'enable_media', 'free_preview_lesson', 'require_video_completion', 'auto_complete_on_video_end', 'enable_passing_score', 'enforce_passing_score', 'hide_result', 'shuffle_options', 'shuffle_questions']; | |
| 446 | 446 | |
| 447 | 447 | foreach ($yesNoFields as $field) { |
| 448 | 448 | $meta[$field] = Arr::get($meta, $field, 'no') == 'yes' ? 'yes' : 'no'; |
| 449 | 449 | } |
| 450 | 450 | |
| 451 | + if ($lesson->isQuizType()) { | |
| 452 | + $meta['layout'] = Arr::get($meta, 'layout') === 'one_by_one' ? 'one_by_one' : 'all'; | |
| 453 | + } | |
| 454 | + | |
| 451 | 455 | if (Arr::get($meta, 'enable_media') == 'yes') { |
| 452 | 456 | if ($media = Arr::get($meta, 'media', [])) { |
| 453 | 457 | $meta['media'] = self::sanitizeMedia($media); |
| 454 | 458 | } |
| @@ -489,9 +493,24 @@ | ||
| 489 | 493 | public static function getCourseCategories() |
| 490 | 494 | { |
| 491 | 495 | $terms = Term::whereHas('base_spaces', function ($q) { |
| 492 | 496 | $q->where('type', 'course'); |
| 493 | - })->get(); | |
| 497 | + })->orderBy('title', 'ASC')->get(); | |
| 498 | + // Respect the admin-defined order (settings.serial) | |
| 499 | + $terms = $terms->sort(function ($a, $b) { | |
| 500 | + $serialA = Arr::get($a->settings, 'serial'); | |
| 501 | + $serialB = Arr::get($b->settings, 'serial'); | |
| 502 | + if ($serialA === null && $serialB === null) { | |
| 503 | + return 0; | |
| 504 | + } | |
| 505 | + if ($serialA === null) { | |
| 506 | + return 1; | |
| 507 | + } | |
| 508 | + if ($serialB === null) { | |
| 509 | + return -1; | |
| 510 | + } | |
| 511 | + return (int) $serialA <=> (int) $serialB; | |
| 512 | + })->values(); | |
| 494 | 513 | |
| 495 | 514 | $formattedTerms = []; |
| 496 | 515 | |
| 497 | 516 | foreach ($terms as $term) { |
| @@ -540,14 +559,22 @@ | ||
| 540 | 559 | |
| 541 | 560 | $inlineCss = ''; |
| 542 | 561 | if ($parseContent && $canViewLesson) { |
| 543 | 562 | $content = $lesson->message_rendered; |
| 544 | - if (!$content && $lesson->message) { | |
| 545 | - $content = apply_filters('the_content', $lesson->message); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound | |
| 546 | - $core_styles_keys = array('block-supports'); | |
| 547 | - // Adds comment if code is prettified to identify core styles sections in debugging. | |
| 548 | - foreach ($core_styles_keys as $style_key) { | |
| 549 | - $inlineCss .= wp_style_engine_get_stylesheet_from_context($style_key, []); | |
| 563 | + | |
| 564 | + // message_rendered still holding block markup never went through the_content | |
| 565 | + if (!$content || has_blocks($content)) { | |
| 566 | + $source = $lesson->message ?: $content; | |
| 567 | + | |
| 568 | + if ($source) { | |
| 569 | + $content = apply_filters('the_content', $source); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound | |
| 570 | + if (function_exists('wp_style_engine_get_stylesheet_from_context')) { | |
| 571 | + $core_styles_keys = array('block-supports'); | |
| 572 | + // Adds comment if code is prettified to identify core styles sections in debugging. | |
| 573 | + foreach ($core_styles_keys as $style_key) { | |
| 574 | + $inlineCss .= wp_style_engine_get_stylesheet_from_context($style_key, []); | |
| 575 | + } | |
| 576 | + } | |
| 550 | 577 | } |
| 551 | 578 | } |
| 552 | 579 | |
| 553 | 580 | if (!$content) { |
| @@ -592,8 +619,11 @@ | ||
| 592 | 619 | |
| 593 | 620 | $newLessonMeta = $newLesson->meta; |
| 594 | 621 | $newLessonMeta['document_lists'] = []; |
| 595 | 622 | foreach ($lesson->media as $media) { |
| 623 | + if (!$media->is_active) { | |
| 624 | + continue; | |
| 625 | + } | |
| 596 | 626 | $newMedia = $media->replicate(); |
| 597 | 627 | $newMedia->feed_id = $newLesson->id; |
| 598 | 628 | $newMedia->save(); |
| 599 | 629 | if ($media->object_source == 'lesson_document') { |
| @@ -600,12 +630,10 @@ | ||
| 600 | 630 | $newLessonMeta['document_lists'][] = $newMedia->getPrivateFileMeta(); |
| 601 | 631 | } |
| 602 | 632 | } |
| 603 | 633 | |
| 604 | - if (!empty($newLessonMeta['document_lists'])) { | |
| 605 | - $newLesson->meta = $newLessonMeta; | |
| 606 | - $newLesson->save(); | |
| 607 | - } | |
| 634 | + $newLesson->meta = $newLessonMeta; | |
| 635 | + $newLesson->save(); | |
| 608 | 636 | } |
| 609 | 637 | |
| 610 | 638 | public static function getAccessMessage($course, $lesson, $config) |
| 611 | 639 | { |
