manage_repo and php_version works, php-fpm is excluded, now

It might be not useful in some cases to overwrite config

manifests/init.pp

@@ -29,18 +29,29 @@ class wmdeit_nextcloud(
 	$nc_user = "www-data",
 	$nc_group = "www-data",
 
-	$php_version = "7.2"
+	$php_version = "7.4",
+	$php_manage_repos = true,
+
+	$create_config = false
 )
 
  {
-
 	class { 'apache':
 		mpm_module => 'prefork',
 	} 
 	class { 'apache::mod::php':
-#		php_version=>"7.2"		
+		php_version=>$php_version		
 	}
+
+
+	class { '::php::globals':
+		php_version => $php_version,
+		config_root => "/etc/php/$php_version",
+	}->
 	class { '::php':
+		manage_repos => $php_manage_repos,
+		fpm  => false,
+
 		extensions => {
 			zip => {},
 			mbstring => {},
@@ -75,18 +86,27 @@ class wmdeit_nextcloud(
 		serveraliases => $trusted_domains,
 		port    => '80',
 		docroot => $install_dir,
+		docroot_owner => 'www-data',
+		docroot_group => 'www-data',
 	} ->
 	exec { 'ncperms':
 		command   => "/bin/chown -R www-data:www-data $install_dir",
 		subscribe => Archive["/tmp/$distfile"],
+		refreshonly => true,
 		require => Class["apache"]
-	} ->
+	} 
+
+	if $create_config 
+	{
 		file { "$install_dir/config/config.php":
+			require => Exec['ncperms'],
 			ensure => file,
 			content => template("wmdeit_nextcloud/config.php.erb")
-	} ->
+		} 
+	}
 
 	exec {"/bin/mkdir -p $data_dir":
+		require => Exec['ncperms'],
 		creates => $data_dir
 	} ->
 	file {$data_dir: