Support for old package.json license structure

[0xs]

I know that according to the current package.json format specification the license info should always be a string. Unfortunately many packages still use the old notation of license.type shown here. Since I should show these licenses too it would be great if there was support for listing them too.

[xz64]

I have pushed a change to the dev branch but haven't tested it. Please try the dev branch and see if that works out for you.

[grauwoelfchen]

Hi,

Thank you for nice tool. I like it!
I tried your dev branch. But I think that this line also should be changed for moduleInfo.license.

license-webpack-plugin/index.js

Line 52 in a760765

var isMatching = this.pattern.test(moduleInfo.license) || !moduleInfo.license && this.includeUndefined;

So, I've just tried to extract license before this pattern matching, and updated to support following old 2 styles.

license: {
  type: '...'
, url: '...'
}

Array but it contains only one license.

e.g. respond.js

licenses: [
  { 'type': '...'
  , 'url': '...'
  }
]

I'll attach a link of a small patch. could you please check this :)
https://gist.github.com/grauwoelfchen/d01899aa46c10f079dfdb8833a57df56

If license is also cached like packagejson, it might be better.

[xz64]

Thanks @grauwoelfchen . I've incorporated your patch and force pushed a new change to dev branch. Caching the license would be ideal, but it should be OK to not cache it for now. Please check out the dev branch and let me know if you have any other thoughts.

[grauwoelfchen]

Thank you @xz64 for quick response . You've also added improvements and tests :)
I will try dev branch!

If this plugin could support user to choose a license from packages which have muliple licenses in this old style licenses in package.json, it would be perfect to me!

[xz64]

@grauwoelfchen I have pushed a change to the dev branch which allows you to use a licenseTypeOverrides option. You can check the README to see how to use it. Please let me know if this works out for you.

[grauwoelfchen]

@xz64 Thank you su much! licenseTypeOverrides and licenseTypeCache look very nice :) I will try it.

[xz64]

Published this feature in v0.4.0.

[grauwoelfchen]

🎉