Agile611/the-example-app-nodejs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(courses): don't break app on draft courses without lessons

Browse Source
This commit is contained in:
Benedikt Rötsch
2018-02-20 13:10:48 +01:00
committed by Benedikt Rötsch
parent fb979fcb1e
commit 82a9a48d86
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
routes/courses.js
View File

@@ -63,6 +63,9 @@ module.exports.getCourse = async (request, response, next) => {
} }
// Get lessons // Get lessons
if (!course.fields.lessons) {
course.fields.lessons = []
}
const lessons = course.fields.lessons const lessons = course.fields.lessons
let {lesson, lessonIndex} = getNextLesson(lessons, request.params.lslug) let {lesson, lessonIndex} = getNextLesson(lessons, request.params.lslug)

5
views/course.pug
View File

@@ -39,6 +39,7 @@ block content
svg.course__overview-icon svg.course__overview-icon
use(xlink:href='/icons/icons.svg#skill-level') use(xlink:href='/icons/icons.svg#skill-level')
.course__overview-value #{translate('skillLevelLabel', currentLocale.code)}: #{translate(`${course.fields.skillLevel}Label`, currentLocale.code)} .course__overview-value #{translate('skillLevelLabel', currentLocale.code)}: #{translate(`${course.fields.skillLevel}Label`, currentLocale.code)}
.course__overview-cta-wrapper if course.fields.lessons && course.fields.lessons.length
a.course__overview-cta.cta(href=`/courses/${course.fields.slug}/lessons/${course.fields.lessons[0].fields.slug}${queryString}`) #{translate('startCourseLabel', currentLocale.code)} .course__overview-cta-wrapper
a.course__overview-cta.cta(href=`/courses/${course.fields.slug}/lessons/${course.fields.lessons[0].fields.slug}${queryString}`) #{translate('startCourseLabel', currentLocale.code)}
.course__description !{helpers.markdown(course.fields.description)} .course__description !{helpers.markdown(course.fields.description)}