puppet bad file array syntax

BAD:

class svnserve::params {
	file { "svnserve_service_folder": 
		path => "/var/svc/network/repository",
		ensure => directory,
		user => "root",
		group => "sys",
		mode => "0755"
	}
	file { "svnserve_service_file":
		path => "/var/svc/network/repository/svnserve",
		user => "root",
		group => "sys",
		mode => "0755",
		source => "puppet:///modules/svnserve/svnserve.xml"
	}
	file { "svnserve_init_file":
		path => "/etc/init.d/svnserve",
		user => "root",
		group => "root",
		mode => "0755",
		source => "puppet:///modules/svnserve/svnserve"
	}
	case $operatingsystem {
		/Solaris|OpenIndiana/: {
			$svnserve_package_name = "developer/versioning/subversion"
			$svnserve_config_location = "/etc/subversion"
			require => [File["svnserve_service_folder"], File["svnserve_service_file"]]
		}
		default: {
			$svnserve_package_name = "subversion"
			$svnserve_config_location = "/etc/subversion"
			require => File["svnserve_init_file"]
		}
	}
}

CORRECT:

class svnserve::params {
		case $operatingsystem {
		/Solaris|OpenIndiana/: {
			$svnserve_package_name = "developer/versioning/subversion"
			$svnserve_config_location = "/etc/subversion"
			file { "svnserve_service_folder": 
				path => "/var/svc/manifest/network/repository",
				ensure => "directory",
				owner => "root",
				group => "sys",
				mode => "0755"
			}
			file { "svnserve_service_file":
				path => "/var/svc/manifest/network/repository/svnserve.xml",
				owner => "root",
				group => "sys",
				mode => "0755",
				source => "puppet:///modules/svnserve/svnserve.xml"
			}
		}
		default: {
			$svnserve_package_name = "subversion"
			$svnserve_config_location = "/etc/subversion"
			file { "svnserve_init_file":
				path => "/etc/init.d/svnserve",
				owner => "root",
				group => "root",
				mode => "0755",
				source => "puppet:///modules/svnserve/svnserve"
			}
		}
	}
}