cloudformation-tool 1.5.14 → 1.5.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9db3672b97a22828e57d6c906a78831153a620bca1632f9f9748f72e27c9db7
-  data.tar.gz: ad0eca7cc6ae46eaf3b0a55c57fd8e85695a27b8c97f71e55492e0a95bd208d2
+  metadata.gz: 32f485059a283b0a34a74ac687c52fe01b6d9522616cc00beb572b46e9a13e1d
+  data.tar.gz: 9533caf5496a471432cadfbf007dd20a6789829d353642538052bde818ed6bfe
 SHA512:
-  metadata.gz: 0452d98f9e5426f6924cc8caf53430b8936fc9ac38187abda7ff9d780cf921330f0f85b592cab7be9b26918e3f54585892e61548e747721dfc46973ec64e04aa
-  data.tar.gz: 6d50f5dd121f968799fd5754a6f54726cd0c64925ce9bd79a27f4360ee710c65a914fc36f3d740536664c4ffc1c1a03b5f055c21848deb2e976900c3f2060dc1
+  metadata.gz: cdd70a1186411fa08ddca90d687c53ad4d5113d503fd02068e6b84e63a3080a11d4dfc9e58f6a8231a90d05ef4ddb5212ece90cae208562a4caef2d54d1eb42c
+  data.tar.gz: f1d758e208b657d5f8e829e4e1dd079850ec4eaa8c1a9593ed09036da9cfe91fc5447edb305d6ed68005a3eda5f2e43df37c7ed4678b03ec8cac745e321af5eb

data/lib/cloud_formation_tool/cloud_formation.rb

@@ -102,50 +102,36 @@ module CloudFormationTool
           # some categories are meta-data that we can ignore from includes
           next if %w(AWSTemplateFormatVersion Description).include? category
           
-          case category
-          when "Parameters"
-            (@data[category]||={}).each do |name, param|
-              if catdata.has_key? name
-                next if param['Default'] == catdata[name]['Default']
-                 
-                if catdata[name].has_key?('Override') and catdata[name]['Override'] == false
-                  catdata.delete(name)
-                else
-                  newname = "#{cfile_key}z#{name}"
-                  log "Rewriting conflicting parameter #{name} (='#{catdata[name]['Default']}') to #{newname}"
-                  catdata[newname] = catdata.delete name
-                  rewrites[name] = newname
-                end
-              else
-                @data[category][name] = param
-              end
+          if category == "Parameters"
+            rewriteParameters catdata, cfile_key, rewrites
+            @data["Parameters"].merge! catdata
+            next
+          end
+          
+          case catdata
+          when Hash
+            # warn against duplicate entities, resources or outputs
+            (@data[category] ||= {}).keys.each do |key|
+              if catdata.has_key? key
+                raise CloudFormationTool::Errors::AppError, "Error compiling #{path} - duplicate '#{category}' item: #{key}"
+              end 
+            end
+            catdata = fixrefs(catdata, rewrites)
+            # add included properties
+            @data[category].merge! catdata
+          when Array
+            if @data[category].nil?
+              @data[category] = catdata
+            elsif @data[category].is_a? Array
+              @data[category] += catdata
+            else
+              raise CloudFormationTool::Errors::AppError, "Error compiling #{path} - conflicting types for '#{category}'"
             end
           else
-            case catdata
-            when Hash
-              # warn against duplicate entities, resources or outputs
-              (@data[category] ||= {}).keys.each do |key|
-                if catdata.has_key? key
-                  raise CloudFormationTool::Errors::AppError, "Error compiling #{path} - duplicate '#{category}' item: #{key}"
-                end 
-              end
-              catdata = fixrefs(catdata, rewrites)
-              # add included properties
-              @data[category].merge! catdata
-            when Array
-              if @data[category].nil?
-                @data[category] = catdata
-              elsif @data[category].is_a? Array
-                @data[category] += catdata
-              else
-                raise CloudFormationTool::Errors::AppError, "Error compiling #{path} - conflicting types for '#{category}'"
-              end
+            if @data[category].nil?
+              @data[category] = catdata
             else
-              if @data[category].nil?
-                @data[category] = catdata
-              else
-                raise CloudFormationTool::Errors::AppError, "Error compiling #{path} - I do not know how to merge non-list non-dictionary '#{category}'!"
-              end
+              raise CloudFormationTool::Errors::AppError, "Error compiling #{path} - I do not know how to merge non-list non-dictionary '#{category}'!"
             end
           end
           
@@ -153,6 +139,27 @@ module CloudFormationTool
       end
     end
     
+    def rewriteParameters data, key, rewrites
+      (@data["Parameters"]||={}).each do |name, param|
+        unless data.has_key? name
+          @data["Parameters"][name] = param
+          next
+        end
+        
+        next if param['Default'] == data[name]['Default']
+        
+        if data[name].has_key?('Override') and data[name]['Override'] == false
+          data.delete(name)
+          next
+        end
+        
+        newname = "#{key}z#{name}"
+        log "Rewriting conflicting parameter #{name} (='#{data[name]['Default']}') to #{newname}"
+        data[newname] = data.delete name
+        rewrites[name] = newname
+      end
+    end
+    
     def resolveVal(value)
       case value
       when Hash

data/lib/cloud_formation_tool/version.rb

@@ -1,3 +1,3 @@
 module CloudFormationTool
-  VERSION = '1.5.14'
+  VERSION = '1.5.15'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cloudformation-tool
 version: !ruby/object:Gem::Version
-  version: 1.5.14
+  version: 1.5.15
 platform: ruby
 authors:
 - Oded Arbel
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-10 00:00:00.000000000 Z
+date: 2023-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake