Correct the default values for exploded property in IDEA configuration
diff --git a/CHANGELOG b/CHANGELOG
index 2fe7054..7ebd1fd 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,6 +4,10 @@
much more frequent given the changes to support external annotations that are packaged as jars. This
crash is now avoided by changing guard from `artifact.respond_to?(:to_spec_hash)` to
`artifact.is_a?(Buildr::Artifact)`.
+* Fixed: The `add_glassfish_remote_configuration` method and the `add_glassfish_configuration` method used the
+ incorrect types for the default values for the `exploded` optional property. This is unlikely to have
+ caused problems during normal usage but could lead to unexpected behaviour when the innards of these
+ methods were monkey patched and expected the "correct" type for these properties.
1.5.8 (2019-07-14)
* Fixed: Changed references to `https://repo1.maven.org/maven2` to use https where possible.
diff --git a/lib/buildr/ide/idea.rb b/lib/buildr/ide/idea.rb
index 6aac75f..92e6ec8 100644
--- a/lib/buildr/ide/idea.rb
+++ b/lib/buildr/ide/idea.rb
@@ -1046,7 +1046,7 @@
domain_name = options[:domain] || project.iml.id
domain_port = options[:port] || '9009'
packaged = options[:packaged] || {}
- exploded = options[:exploded] || {}
+ exploded = options[:exploded] || []
add_to_composite_component(self.configurations) do |xml|
xml.configuration(:name => configuration_name, :type => 'GlassfishConfiguration', :factoryName => 'Local', :default => false, :APPLICATION_SERVER_NAME => server_name) do |xml|
@@ -1114,7 +1114,7 @@
configuration_name = options[:configuration_name] || "Remote #{server_name}"
domain_port = options[:port] || '9009'
packaged = options[:packaged] || {}
- exploded = options[:exploded] || {}
+ exploded = options[:exploded] || []
add_to_composite_component(self.configurations) do |xml|
xml.configuration(:name => configuration_name, :type => 'GlassfishConfiguration', :factoryName => 'Remote', :default => false, :APPLICATION_SERVER_NAME => server_name) do |xml|
