Using lodash's wrap method is prevented in grunt because of underscore.string

[mazesoul]

When using lodash wrap method in a Gruntfile, there are errors because grunt-legacy-log mixes in underscore.string.

Reproduction

At the top of a Gruntfile.js add the following

const _ = require('lodash');

if (_.isString(_.wrap(_.noop, _.noop))) {
  console.info('grunt-legacy-log messes up lodash wrap function by mixing in underscore.string');
}

Where

The problem arises from here https://github.com/gruntjs/grunt-legacy-log/blob/master/index.js#L20-L22.

[axten]

@shama
please see wikimedia/grunt-stylelint#49
We have massive problems with that overrides because of lines:

grunt-legacy-log/index.js

Lines 19 to 22 in 359ada7

	// The upcoming lodash 2.5+ should remove the need for underscore.string.
	var _ = require('lodash');
	_.str = require('underscore.string');
	_.mixin(_.str.exports());

It modifies lodash functions for all npm packages using the same lodash version!!
This was not a problem until you updated to newest lodash version. There are now many many potentially failing packages out there when using lodash with grunt.

please fix that as soon as possible!

[jdforrester]

Brilliant. Is there going to be a new release of grunt pulling in grunt-legacy-log@1.0.2 soon?

[shama]

@jdforrester Grunt has it tagged "grunt-legacy-log": "~1.0.0", so fresh installs will get the latest patch version, 1.0.2.

That should fix the issue for those using lodash@4.17.5. We'll have a fix to prevent this also when using lodash@4.3.0 and for all future versions soon too: gruntjs/grunt-legacy-util#18

[axten]

thank you @shama for the quick response!

[ntwb]

Awesome, thanks for speedy fix and release, greatly appreciated :)