FIX: require full name at signup when display is suppressed and required

discourse · Jun 29, 2016 · b4cb2e3 · b4cb2e3
1 parent 8fbcda5
commit b4cb2e3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/app/assets/javascripts/discourse/controllers/create-account.js.es6 b/app/assets/javascripts/discourse/controllers/create-account.js.es6
@@ -67,6 +67,10 @@ export default Ember.Controller.extend(ModalFunctionality, {
 
   usernameRequired: Ember.computed.not('authOptions.omit_username'),
 
+  fullnameRequired: function() {
+    return this.get('siteSettings.full_name_required') || this.get('siteSettings.enable_names');
+  }.property(),
+
   passwordRequired: function() {
     return Ember.isEmpty(this.get('authOptions.auth_provider'));
   }.property('authOptions.auth_provider'),

diff --git a/app/assets/javascripts/discourse/templates/modal/create-account.hbs b/app/assets/javascripts/discourse/templates/modal/create-account.hbs
@@ -35,7 +35,7 @@
           </tr>
         {{/if}}
 
-        {{#if siteSettings.enable_names}}
+        {{#if fullnameRequired}}
           <tr class="input">
             <td style="width:80px" class="label">
               <label for='new-account-name'>{{i18n 'user.name.title'}}</label>