Issue 1495 - Introduce $sitescriptsini_content in Puppet sitescripts module

modules/sitescripts/manifests/init.pp

 class sitescripts (
     $sitescriptsini_source = '',
     $sitescriptsini_content = '',
   ){
 
   concat {'/etc/sitescripts.ini':
     mode => 644,
     owner => root,
     group => root,
   }
 
   define configfragment($content = '', $source = '')
   {
     concat::fragment {"/etc/sitescripts.ini#$title":
       target  => '/etc/sitescripts.ini',
       content => $content,
-      source  => "$source;$content" ? {
-        ';'     => $title,

[Wladimir Palant, 2014/11/26 22:47:09]

Why not:

  source => "$source$content" ? {
    '' => $title,

Seems more obvious.

[mathias, 2014/11/26 23:34:29]

I would agree on "more common" rather than "more obvious", but also "less
readable". To answer your question: No reason other than personal preference. Do
you insist on the empty string version? ;)

[Wladimir Palant, 2014/12/08 16:44:00]

I do :)

In fact, I asked Sebastian as somebody who isn't familiar with the code - and he
perceived the variant without a semicolon as more readable. In fact, the empty
string indicates that testing for empty strings is what we are doing here.

[mathias, 2014/12/10 12:51:05]

Done.

+      source  => "$source$content" ? {
+        ''     => $title,
         default => $source,
       }
     }
   }
 
   configfragment {'/etc/sitescripts.ini':
     content => $sitescriptsini_content,
     source => $sitescriptsini_source,
   }
 
   exec { "fetch_sitescripts":
     command => "hg clone https://hg.adblockplus.org/sitescripts /opt/sitescripts",
     path => ["/usr/bin/", "/bin/"],
     require => Package['mercurial'],
     onlyif => "test ! -d /opt/sitescripts"
   }
 
   cron {"update_sitescripts":
     ensure => present,
     command => "hg pull -q -u -R /opt/sitescripts",
     environment => ['MAILTO=admins@adblockplus.org,root'],
     user => root,
     require => Exec["fetch_sitescripts"],
   }
 }