From cbfa047a71f7920d3cbed9bd963ad33ee1551e01 Mon Sep 17 00:00:00 2001
From: Jerome Charaoui <jcharaoui@cmaisonneuve.qc.ca>
Date: Thu, 21 May 2015 13:12:18 -0400
Subject: Revert "Simplify ssh_authorized_key"

puppet-lint complains about "selector inside resource"

This reverts commit f3c0115743cab9d4e6c08b654b67631566572d41.
---
 manifests/ssh_authorized_key.pp | 55 +++++++++++++++++++++++++----------------
 1 file changed, 34 insertions(+), 21 deletions(-)

diff --git a/manifests/ssh_authorized_key.pp b/manifests/ssh_authorized_key.pp
index 7702914..f942d62 100644
--- a/manifests/ssh_authorized_key.pp
+++ b/manifests/ssh_authorized_key.pp
@@ -48,32 +48,45 @@ define sshd::ssh_authorized_key(
   # nothing a user can't already do by writing their own file resources,
   # we still depend on the filesystem permissions to keep things safe.
   if $override_builtin {
+    case $options {
+      'absent': {
+        info("not setting any option for ssh_authorized_key: ${name}")
+        $header = "# HEADER: This file is managed by Puppet.\n"
 
-    $header = '# HEADER: This file is managed by Puppet.'
-
-    file { $real_target:
-      ensure  => $ensure,
-      owner   => $real_user,
-      mode    => '0600',
-      content => $options ? {
-        'absent' => "${header}\n${type} ${key}",
-        default  => "${header}\n${options} ${type} ${key}",
-      },
+        file { $real_target:
+          ensure => $ensure,
+          content => "${header}${type} ${key}",
+          owner => $real_user,
+          mode => '0600';
+        }
+      }
+      default: {
+        file { $real_target:
+          ensure => $ensure,
+          content => "${header}${options} ${type} ${key}",
+          owner => $real_user,
+          mode => '0600';
+        }
+      }
     }
-
   } else {
-
     ssh_authorized_key{$name:
-      ensure  => $ensure,
-      type    => $type,
-      key     => $key,
-      user    => $real_user,
-      target  => $real_target,
-      options => $options ? {
-        'absent' => undef,
-        default  => $options,
-      },
+      ensure => $ensure,
+      type   => $type,
+      key    => $key,
+      user   => $real_user,
+      target => $real_target,
     }
 
+    case $options {
+      'absent': {
+        info("not setting any option for ssh_authorized_key: ${name}")
+      }
+      default: {
+        Ssh_authorized_key[$name]{
+          options => $options,
+        }
+      }
+    }
   }
 }
-- 
cgit v1.2.3