Name fallback is actually a bit confusing

controllers/wizard.rb

@@ -1,7 +1,7 @@
 class CustomWizard::WizardController < ::ApplicationController
   prepend_view_path(Rails.root.join('plugins', 'discourse-custom-wizard', 'views'))
   layout 'wizard'
-  
+
   before_action :ensure_logged_in
   helper_method :wizard_page_title
 

lib/builder.rb

@@ -40,18 +40,8 @@ class CustomWizard::Builder
   def self.build_post(template, user, data)
     post = template.gsub(/u\{(.*?)\}/) do |match|
       result = ''
-
+      result = user.send($1) if USER_FIELDS.include?($1)
-      if USER_FIELDS.include?($1)
+      result = user.user_profile.send($1) if PROFILE_FIELDS.include?($1)
-        result = user.send($1)
-        if result.blank? && $1 === 'name'
-          result = user.send('username')
-        end
-      end
-
-      if PROFILE_FIELDS.include?($1)
-        result = user.user_profile.send($1)
-      end
-
       result
     end
     post.gsub!(/w\{(.*?)\}/) { |match| data[$1.to_sym] }