Latest Interactive Video broke xAPI code

Hi spazfox!

It's not that complicated ... This should do (it sure looks like a lot, but remove all the comments and the two extra constants that I assigned ...).

// Since I don't know your environment, let's stick with ES5.
const that = this;

/*
 * Get first instance of H5P.interactiveVideo -- will not work
 * properly if there are more instances of IV.
 */
this.interactiveVideo = H5P.instances.filter(function(instance) {
  return instance.libraryInfo &&
    instance.libraryInfo.machineName === 'H5P.InteractiveVideo';
})[0];

// Determine which interactions are scored and get their instances
this.numberScoredInteractions = interactiveVideo.interactions
  .filter(function(interaction) {
    return typeof interaction.getInstance().getScore === 'function' &&
      typeof interaction.getInstance().getMaxScore === 'function'
  }).length;

// Here's where the action takes place ...
interactiveVideo.video.on('stateChange', function(event) {
  /*
   * H5P.Video should be accessible here, but H5P.Video.ENDED could also
   * be replaced with 0 currently.
   */ 

  // Listen for video ended events only (not play or pause or others that might be introduced)
  if (event.data !== H5P.Video.ENDED) {
    return;
  }

  /* Check if user got maximum score (without retries)
   * >= because you never know ...
   */
  const completed = that.interactiveVideo.getScore() >=
    that.interactiveVideo.getMaxScore();

  /*
   * The function getAnsweredTotal() is not part of a so called "contract"
   * and might change, but this is not likely.
   */

  // Check if user answered at least one question
  const answeredOne = that.interactiveVideo.getAnsweredTotal() > 0;

  // Check if user answered all scored questions, not necessarily correct
  const answeredAll = that.interactiveVideo.getAnsweredTotal() ===
    that.numberScoredInteractions;

  /*
   * Here you can insert the constant that you need (completed or answeredOne
   * or answeredAll), so replace "answered" with one of the other two if
   * you prefer them. You could delete the others, too, of course.
   */
  if (answeredOne) {
    /*
     * Show button, you may have to set the context correctly if showNext()
     * is not accessible from this function.
     */
    showNext();
  }
});

I just checked our previous release note, and it contains a link to the release overview which lists changes (e. g. "Stop sending xAPI completed on end of the video. This is now done only by the submit screen." for Interactive Video). There's a lot we could improve however, that's true. In fact, the feature you described is supposed to be in the update of the Hub -- when we get around to it.

It's good practice not to update on a production system before testing everything on an identical (fairly) system. You should be quite safe if you have a standard install, but the more you customize your system, the more can break -- that's a downside of H5P being so open.

Best,
Oliver