Contain child classes

- found that when used with other modules ordering was difficult, mostly due to:

```
  File {
    require => Package['artifactory'],
  }
```
  This commit removes the blanket precedence statement and replaces it with
  explicit ordering and containment.

manifests/init.pp

@@ -35,10 +35,15 @@
   $backup_path      = undef,
 ) {
 
-  include ::java
+  Class['::java']
+  -> Class['::artifactory::install']
+  -> Class['::artifactory::config']
+  -> Class['::artifactory::service']
+
+  contain '::java'
+  contain '::artifactory::install'
+  contain '::artifactory::config'
+  contain '::artifactory::service'
 
-  class { '::artifactory::install': } ->
-  class { '::artifactory::config': } ->
-  class { '::artifactory::service': }
 
 }

manifests/install.pp

@@ -13,10 +13,6 @@
     fail("Use of private class ${name} by ${caller_module_name}")
   }
 
-  File {
-    require => Package['artifactory'],
-  }
-
   user { 'artifactory':
     ensure => 'present',
     system => true,
@@ -45,11 +41,16 @@
       mode   => '0775',
       owner  => artifactory,
       group  => artifactory,
+      before => Package['artifactory'],
     }
 
     file { '/var/opt/jfrog/artifactory/data':
       ensure => link,
       target => $::artifactory::data_path,
+      before => [
+        Package['artifactory'],
+        File[$::artifactory::data_path],
+      ]
     }
   }
 
@@ -59,7 +60,7 @@
       mode   => '0775',
       owner  => artifactory,
       group  => artifactory,
+      before => Package['artifactory'],
     }
   }
-
 }