Update stdlib and concat to 6.1.0 both

[mirror/dsa-puppet.git] / 3rdparty / modules / stdlib / lib / puppet / parser / functions / getvar.rb

diff --git a/3rdparty/modules/stdlib/lib/puppet/parser/functions/getvar.rb b/3rdparty/modules/stdlib/lib/puppet/parser/functions/getvar.rb
index fb336b6..41d3c4f 100644 (file)
--- a/3rdparty/modules/stdlib/lib/puppet/parser/functions/getvar.rb
+++ b/3rdparty/modules/stdlib/lib/puppet/parser/functions/getvar.rb
@@ -1,29 +1,40 @@
+#
+# getvar.rb
+#
 module Puppet::Parser::Functions
+  newfunction(:getvar, :type => :rvalue, :doc => <<-'DOC') do |args|
+    @summary
+      Lookup a variable in a given namespace.
 
-  newfunction(:getvar, :type => :rvalue, :doc => <<-'ENDHEREDOC') do |args|
-    Lookup a variable in a remote namespace.
+    @return
+      undef - if variable does not exist
 
-    For example:
+    @example Example usage
+      $foo = getvar('site::data::foo') # Equivalent to $foo = $site::data::foo
 
-        $foo = getvar('site::data::foo')
-        # Equivalent to $foo = $site::data::foo
+    @example Where namespace is stored in a string
+      $datalocation = 'site::data'
+      $bar = getvar("${datalocation}::bar") # Equivalent to $bar = $site::data::bar
 
-    This is useful if the namespace itself is stored in a string:
-
-        $datalocation = 'site::data'
-        $bar = getvar("${datalocation}::bar")
-        # Equivalent to $bar = $site::data::bar
-    ENDHEREDOC
+    > **Note:** from Puppet 6.0.0, the compatible function with the same name in Puppet core
+    will be used instead of this function. The new function also has support for
+    digging into a structured value. See the built-in
+    [`getvar`](https://puppet.com/docs/puppet/latest/function.html#getvar) function
+    DOC
 
     unless args.length == 1
-      raise Puppet::ParseError, ("getvar(): wrong number of arguments (#{args.length}; must be 1)")
+      raise Puppet::ParseError, "getvar(): wrong number of arguments (#{args.length}; must be 1)"
     end
 
     begin
-      self.lookupvar("#{args[0]}")
-    rescue Puppet::ParseError # Eat the exception if strict_variables = true is set
+      result = nil
+      catch(:undefined_variable) do
+        result = lookupvar((args[0]).to_s)
+      end
+
+      # avoid relying on inconsistent behaviour around ruby return values from catch
+      result
+    rescue Puppet::ParseError # rubocop:disable Lint/HandleExceptions : Eat the exception if strict_variables = true is set
     end
-
   end
-
 end