Version 2.4.7 broke compatibility about nunjucks-loader

[linsolas]

Hello,

Not perfectly sure my problem is related to this incompatibility, but with 2.4.7 the loader does not get the query options, while with 2.4.5 everything is OK. In the loaderUtils.getOptions, it expect an object with the query parameter (code here) :

function getOptions(loaderContext) {
	const query = loaderContext.query;
	if(typeof query === "string" && query !== "") {
		return parseQuery(loaderContext.query);
	}
	if(!query || typeof query !== "object") {
		// Not object-like queries are not supported.
		return null;
	}
	return query;
}

However, the nunjucks-loader is giving a string:

   if (!hasRun){
        var query = loaderUtils.getOptions(this.query);

and thus, the query object is null everytime. If I change the previous line to:

   if (!hasRun){
        var query = loaderUtils.getOptions({query: this.query});

then it works as expected.

note: I'm using Webpack 1 in my project and my Webpack configuration is like that:

{
  test: /\.(njk|nunjucks)$/,
  loader: 'nunjucks-loader',
  query: {
    root: __dirname + '/pages',
    opts: {
      tags: {
        variableStart: '{$',
        variableEnd: '$}'
      }
    }
  }

[at0g]

v2.4.8 reverts the changes in PR #48 and fixes this mess.
Will look at a 3.x branch for the move to loader utils 1.x