jenkins-bot has submitted this change and it was merged.

Change subject: Correctly capitalize only first letter of username; leave rest 
alone
......................................................................


Correctly capitalize only first letter of username; leave rest alone

Minor style change

Follows-up: 290e200ec44b014cf8c279d71f02609a783f0d48

Change-Id: I03d5ec1253505e45f6ba9f2de2999c87e68e5763
---
M puppet/modules/mediawiki/manifests/user.pp
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  Dduvall: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/puppet/modules/mediawiki/manifests/user.pp 
b/puppet/modules/mediawiki/manifests/user.pp
index 0c5902e..cb6900b 100644
--- a/puppet/modules/mediawiki/manifests/user.pp
+++ b/puppet/modules/mediawiki/manifests/user.pp
@@ -42,7 +42,7 @@
     $username = $title,
     $email    = undef,
     $wiki     = $::mediawiki::db_name,
-    $groups   = []
+    $groups   = [],
 ) {
     include ::mediawiki
 
@@ -50,7 +50,10 @@
     # In the meantime, just address one common issue.
 
     # Capitalize first character
-    $canonical_username = capitalize($username)
+    # Ruby's capitalize has a bug/feature where it *lower-cases* every
+    # character except the first even when they were already
+    # capitalized.  Puppet inherits this.
+    $canonical_username = inline_template('<%= @username[0].capitalize + 
@username[1..-1] %>')
 
     exec { "mediawiki_user_${canonical_username}_${wiki}":
         command => "/usr/local/bin/mwscript createAndPromote.php \

-- 
To view, visit https://gerrit.wikimedia.org/r/254369
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I03d5ec1253505e45f6ba9f2de2999c87e68e5763
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <[email protected]>
Gerrit-Reviewer: BryanDavis <[email protected]>
Gerrit-Reviewer: Dduvall <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to