From 79b8aacbb7b347bba9d14b1332666e7263a3a058 Mon Sep 17 00:00:00 2001 From: Jesse Morgan Date: Sun, 8 Oct 2017 17:29:48 -0700 Subject: Attempt to update chapter completion on every request. I'm still seeing occasional users with missing progress information in CCB. Currently, the progress is only updated when the user completes a chapter. With this change, the progress will be updated every time the training page is loaded. CCBProgressReporter does not actually update the user's profile unless the new level is greater than the existing level. This provision will prevent needlessly calling the CCB API. Additionally, this update will execute in a background thread to avoid impacting the page load. --- src/main/java/com/p4square/grow/model/Chapter.java | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) (limited to 'src/main/java/com/p4square/grow/model') diff --git a/src/main/java/com/p4square/grow/model/Chapter.java b/src/main/java/com/p4square/grow/model/Chapter.java index 3a08e4c..ac27de6 100644 --- a/src/main/java/com/p4square/grow/model/Chapter.java +++ b/src/main/java/com/p4square/grow/model/Chapter.java @@ -4,6 +4,7 @@ package com.p4square.grow.model; +import java.util.Date; import java.util.HashMap; import java.util.Map; @@ -97,6 +98,26 @@ public class Chapter implements Cloneable { return complete; } + /** + * @return the completion date for the chapter, or null if it has not been completed. + */ + @JsonIgnore + public Date getCompletionDate() { + Date latest = new Date(0); + for (VideoRecord video : mVideos.values()) { + if (video.getRequired() && !video.getComplete()) { + // Hey, this chapter isn't complete! + return null; + } + + Date completionDate = video.getCompletionDate(); + if (completionDate.after(latest)) { + latest = completionDate; + } + } + return latest; + } + /** * Deeply clone a chapter. * -- cgit v1.2.3