bb-drummer / node-patternlibrary

lib/handlebars/repeat.js

var handlebars = require('handlebars');

/**
 * Handlebars block helper that repeats the content inside of it n number of times.
 * @param {integer} count - Number of times to repeat.
 * @param {object} options - Handlebars object.
 * @example
 * {{#repeat 5}}<li>List item!</li>{{/repeat}}
 * @returns The content inside of the helper, repeated n times.
 */
function repeat (count, options) {
  var str = '';
  var data;

  if (options.data) {
    data = handlebars.createFrame(options.data);
  }

  for (var i = 0; i < count; i++) {
    if (data) {
      data.index = i;
    }

    str += options.fn(this, { data: data });

The variable data does not seem to be initialized in case options.data on line 15 is false. Are you sure this can never be the case?

  }

  return str;
}

module.exports = repeat;
handlebars.registerHelper('repeat', repeat);