Merge pull request #538 from mmckinst/bool2str_enhance

add functionality to bool2str function

add functionality to bool2str to return strings of your choice for a boolean

Merge pull request #539 from mentat/MODULES-2696

Add check to ensure regex does not throw for none type.

Add check to ensure regex does not throw for none type.

Add a quick check to ensure puppetversion value is not nil and supporting test.

Merge pull request #537 from cmurphy/fix_load_module_metadata

Fix load module metadata

Let load_module_metadata succeed on empty file

Some modules or module versions don't have a metadata.json file, but we
might still want to use the load_module_metadata function on them. The
lack of a file can still give us important information. For example, it
might tell us that the version of the module installed is "very old"
even if we can't read the version number directly. This patch adds a
parameter to let the user specify if an empty file is acceptable. To
preserve backwards compatibility it does not change the current default
behavior, which is to raise an error if metadata.json does not exist.

Rename load_module_metadata test path

`rake spec` only finds test files that end in _spec.rb, so this test
was not being run. Correct the path name so that the test runs
properly.

Merge pull request #536 from DavidS/improve-file_line-docs

(MODULES-2421) improve description of file_line

(MODULES-2421) improve description of file_line

This mostly needed extraction of the existing doc strings from
the type.

Merge pull request #535 from martinpfeifer/package_provider_fact

prevent deprecation warning about the allow_virtual parameter

prevent deprecation warning about the allow_virtual parameter

Merge pull request #534 from asasfu/feature/package_provider_fact

Add package_provider fact

Merge pull request #533 from HelenCampbell/MODULES-2614-Improved

Modules 2614 improved numeric value handling on empty function

Added acceptance test and updated readme

Check for numeric values as empty fails on those

Merge pull request #532 from DavidS/fix-is_a-readme

(FM-3701) Update README for is_a

(FM-3701) Update README for is_a

Merge pull request #531 from asasfu/improvement/service_provider_fact

fixup-PR#506 Speed improvements in facter resolution

fixup-PR#506 Speed improvements in facter resolution

This is to improve speed on Facter resolution of service_provider fact
that was just introduced in PR# 506.  The improvements go from 280ms
resolution time approx. down to 2ms resolution time approx. by adding
requires statements.

Add package_provider fact

This adds a package_provider fact for situations where we need to be
able to know the client's package provider in a simple way.  Situations
such as: package { 'name': install_options => [] }  As those tend to be
package provider specific options.

Merge pull request #506 from binford2k/feature/service_provider_fact

Add a service_provider fact

Merge pull request #530 from HelenCampbell/MODULES-2614

Adding update to empty function readme

Adding update to empty function readme

Merge pull request #528 from mhaskel/is_a_fixes

Update is_a acceptance tests to only run on puppet4

Update is_a acceptance tests to only run on puppet4

Merge pull request #527 from mhaskel/511_compatibility

Fix backwards compatibility from #511

Fix backwards compatibility from #511

Maintain the old behavior in the case where the optional second
parameter isn't passed. Also, adding arity is backwards incompatible since
stdlib still supports 2.7, so remove that.

Merge pull request #526 from DavidS/improve-validate_re-docs

(MAINT) validate_re: Clarify docs and error message

(MAINT) validate_re: Clarify docs and error message

Merge pull request #524 from binford2k/docs/intersection_truthiness

Clarify what an empty intersection looks like.

Clarify what an empty intersection looks like.

Merge pull request #523 from DavidS/modules-2516-is_a

(MODULES-2561) add is_a function

Merge pull request #511 from dmitryilyin/parseyaml_default

[MODULES-2462] Improve parseyaml function

(MODULES-2516) Adds an is_a() function

The data type system is very hard to understand. Many people don't
understand why

    type_of([1,2,3]) == Array

will fail, but

    type_of([1,2,3]) <= Array

passes. This does a simpler validation that doesn't rely on explicit
data types. Instead, use

    $foo = [1,2,3]
    if $foo.is_a(Array) {
      notify { 'This is an array': }
    }

This is based on code by Ben Ford <ben.ford@puppetlabs.com>.

  * Added acceptance tests
  * Added dispatch
  * Improved unit tests
  * Added docs to README

Merge pull request #518 from logicminds/case_insensitive

accept any case of boolean strings

Merge pull request #519 from dmitryilyin/fetch

[MAINT] Improve 'try_get_value' readme

[MAINT] Improve 'try_get_value' readme

accept any case of boolean strings

  * previously the str2bool function did not accept 'TRUE' as a bool
    type.  This causes the function to now accept TRUE, FALSE strings
    as a boolean type in order to be converted to a proper boolean.
  * This would also cause Y,N, YES, NO to be accepted as boolean types
    as well.

Merge pull request #517 from DavidS/try_get_value-acceptance

(MAINT) fix up try_get_value acceptance test

(MAINT) fix up try_get_value acceptance test

Merge pull request #513 from dmitryilyin/fetch

Add a new function "try_get_value"

Add a new function "try_get_value"

* Extracts a value from a deeply-nested data structure
* Returns default if a value could not be extracted

Merge pull request #515 from jfautley/ticket/MODULES-2478-support_root_home_fact_on_AIX

(MODULES-2478) Make root_home fact work on AIX using native lsuser command

(MODULES-2478) Support root_home fact on AIX through "lsuser" command

Squashed, and amended test for comment lines.

Merge pull request #514 from DavidS/add-convert_base

Adds a convert_base function, which can convert numbers between bases

Adds a convert_base function, which can convert numbers between bases

Squashed, improved docs, updated error handling and unit tests by David S.

[MODULES-2462] Improve parseyaml function

* Add default value support
  Second argument will be returned if yaml cannot be parsed
  instead of false value
* Update tests

Merge pull request #512 from Jetroid/consistentreadme

Consistent Readme

Add consistent *Type* information

Remove trailing whitespace

Two functions had not been given any *Type* information. This commit fixes that.

Merge pull request #510 from DavidS/base64-unit-tests

(MAINT) improve base64 unit tests

(MAINT) improve base64 unit tests

Merge pull request #507 from Jetroid/mod2456

(MODULES-2456) Modify union to accept more than two arrays

(MODULE-2456) Modify union to accept more than two arrays

Add spec tests to test the new functionality:
 *Case for 3 arrays.
 *Case for 4 arrays.
Modify README to note new functionality.

This is for issue MODULE-2456, follow the precedent of MODULE-444.

This change allows union to be much more useful, unioning many arrays
in one line rather than in n lines. Additionally, as this is only added
functionality, and does not affect the 2 array case that all modules
currently using array are using, it should not affect any existing
modules utilizing union.

This is now useful, for example, for merging many arrays of resources
(eg: packages.) to generate just one list with no duplicates, to avoid
duplicate resource declarations.

Add a service_provider fact

This returns the default provider Puppet will choose to manage services
on this system by instantiating a dummy service resource type and
returning the provider chosen.

Co-Authored-By: Simon Fraser University <asa188@sfu.ca>

Merge pull request #505 from gibbsoft/dos2unix

(MODULES-2410) Add new functions dos2unix and unix2dos

(MODULES-2410) Add new functions dos2unix and unix2dos

Merge pull request #499 from jearls/2370-use-match-for-ensure-absent

[MODULES-2370] allow `match` parameter to influence `ensure => absent` behavior.

Merge pull request #503 from puppetlabs/4.8.x

Mergeback 4.8.x

Add puppet_version back to spec_helper

Merge pull request #502 from hunner/fix_after

Sometimes this exits 1

Sometimes this exits 1

Merge pull request #501 from hunner/bug_helper

Fix extraneous end

Fix extraneous end

Merge pull request #500 from hunner/prep_4.8.0

Prep 4.8.0

Prep 4.8.0

allow `match` parameter to influence `ensure => absent` behavior.

Split the `destroy` method of the file_type::ruby provider into two
private methods:  `handle_destroy_line` which is the same as the previous
`destroy` method, and `handle_destroy_with_match` which will destroy any
line which matches the `match` parameter, raising an error if multiple
lines match and the `multiple` parameter is not `true`.  This new
behavior is only used if the new boolean parameter `match_for_absence`
is `true` (it defaults to `false`).

Merge pull request #497 from domcleal/tickets/master/MODULES-2316

(MODULES-2316) Change file_type boolean parameter to symbols

(MODULES-2316) Change file_type boolean parameter to symbols

Puppet's boolean parameter type is only available in Puppet 3.3 and
higher, so change file_type's new "replace" parameter to a regular
parameter with true and false as possible values.  This matches the
existing "multiple" parameter.

Merge pull request #483 from nibalizer/load_metadata_json

Add load_metadata_json function

Add load_metadata_json function

This function loads the metadata.json into a puppet variable. This enables a number of neat things such as:

* Which version of the module am I using? 2.x? 3.x?
* Which author of the module am I using? puppetlabs? example42?

Merge pull request #496 from pizzaops/master

Remove colorful language from module.

Remove colorful language from module.

Merge pull request #494 from CENGN/fix/master/file_line_replace

[#puppethack] Adding replace attribute to file_line

Merge pull request #495 from puppetlabs/4.7.x

4.7.x

(MODULES-2024) Adding replace attribute to file_line

Merge pull request #493 from DavidS/fixup-acceptance-testing

(maint) use puppet's utility function instead of API that's not avail…

(maint) use puppet's utility function instead of API that's not available on all rubies

Merge pull request #492 from DavidS/fixup-acceptance-testing

Fixup acceptance testing

Merge pull request #491 from ekohl/style-fixes

Style fixes

Style fixes

(main) clean up fqdn_rand acceptance tests to work on windows

(maint) Remove failing acceptance test for ensure_packages

This only duplicates what's already being tested in ensure_packages_spec.rb
but doesn't work on all our supported platforms.

Merge pull request #489 from gcmalloc/master

adding support for hash in the size function

adding support for hash in the size function

Merge pull request #490 from hunner/fix_specifier

disable pw_hash test on sles, as it only supports md5

disable pw_hash test on sles, as it only supports md5

Merge pull request #487 from liv3d/fix_docs_in_upcase

Fix documentation error in upcase

Merge pull request #488 from cyberious/master

(maint) Fix test to not assume is_pe fact on > 4.0.0 puppet

(maint) Fix test to not assume is_pe fact on > 4.0.0 puppet

Fix documentation error in upcase

The documentation example shows an incorrect response when using the
function, this PR corrects the example to agree with what the function
actually does.

Merge pull request #481 from tphoney/release_4.7.0

prep work for 4.7.0

prep work for 4.7.0

add new improvements

puppet4 changes for testing

Merge pull request #485 from ghoneycutt/patch-2

Clarify that third argument to ensure_resource() is a hash

Clarify that third argument to ensure_resource() is a hash

Merge pull request #484 from mhaskel/use_puppet_install_helper

Use puppet_install_helper

Use puppet_install_helper

Merge pull request #475 from DavidS/document-puppet4-compat

Document puppet 4 compatability in 4.6

Merge pull request #482 from DavidS/document_validate_slength_3rd_arg

Add validate_slength's optional 3rd arg to README

Merge pull request #479 from mklette/master

catch and rescue from looking up non-existent facts

Add validate_slength's optional 3rd arg to README