Fix #277 aggregate-add-third-party does not use missingFileUrl parameter

[elballa]

Integration test reproducing the issue

[elballa]

Hello @ppalaga, let me know if fix is as your expectation

[ppalaga]

@elballa thanks, looks quite good. Could you please do the following to make sure that we are not breaking the missingFile param?

Create three separate executions in src/it/aggregate-add-third-party-missing-file/pom.xml (much like we do in

license-maven-plugin/src/it/add-third-party-missing-file/pom.xml

Lines 88 to 109 in 87489be

	<execution>
	<id>by-classpath</id>
	<phase>generate-resources</phase>
	<goals>
	<goal>add-third-party</goal>
	</goals>
	<configuration>
	<missingFileUrl>classpath:missing-licenses.properties</missingFileUrl>
	<thirdPartyFilename>THIRD-PARTY-by-classpath.txt</thirdPartyFilename>
	</configuration>
	</execution>
	<execution>
	<id>by-file</id>
	<phase>generate-resources</phase>
	<goals>
	<goal>add-third-party</goal>
	</goals>
	<configuration>
	<missingFileUrl>file:///${basedir}/missing-licenses.properties</missingFileUrl>
	<thirdPartyFilename>THIRD-PARTY-by-file.txt</thirdPartyFilename>
	</configuration>
	</execution>

)

• First: missingFileUrl set, missingFile not set - this is actually what you have there now
• Second: missingFileUrl not set, missingFile set
• Third: both missingFileUrl and missingFile set -the entries should get merged as documented missingFileUrl 's JavaDoc.

You'll probably need to add one more dependency to have at least one for each of missingFileUrl and missingFile. org.json:json:20070829 might be a good candidate.

[elballa]

We created three integration tests as you asked for.
We found a new problem in using missingFile parameter because it was overrided by default value of deprecated aggregateMissingLicensesFile parameter as explained in the issue #284. So we also provided the solution of that issue by removing the assignment of the default value.

[ppalaga]

Replaced by #286