From 7ba6f6a54d14066f2e49cd45d8390d758198925f Mon Sep 17 00:00:00 2001
From: Martin Edenhofer <me@edenhofer.de>
Date: Tue, 5 Jan 2016 10:38:43 +0100
Subject: [PATCH] Enable default chat in preferences automatically if only one
 exists.

---
 .../javascripts/app/controllers/chat.coffee   | 36 +++++++++++++++----
 .../app/views/customer_chat/setting.jst.eco   | 29 +++++++--------
 2 files changed, 44 insertions(+), 21 deletions(-)

diff --git a/app/assets/javascripts/app/controllers/chat.coffee b/app/assets/javascripts/app/controllers/chat.coffee
index f0349f984..4eaae253b 100644
--- a/app/assets/javascripts/app/controllers/chat.coffee
+++ b/app/assets/javascripts/app/controllers/chat.coffee
@@ -127,11 +127,31 @@ class App.CustomerChat extends App.Controller
     if state
       @startPushState()
       preferences = @Session.get('preferences')
-      if !preferences || !preferences.chat || !preferences.chat.active || _.isEmpty(preferences.chat.active)
-        @notify(
-          type: 'error'
-          msg:  App.i18n.translateContent('To be able to chat you need to select min. one chat topic in settings!')
-        )
+      if App.Chat.first() && !preferences || !preferences.chat || !preferences.chat.active || _.isEmpty(preferences.chat.active)
+
+        # if we only have one chat, avtice it automatically
+        if App.Chat.count() < 2
+          preferences.chat = {}
+          preferences.chat.active = {}
+          preferences.chat.active[App.Chat.first().id] = 'on'
+
+          # update user preferences
+          @ajax(
+            id:          'preferences'
+            type:        'PUT'
+            url:         "#{@apiPath}/users/preferences"
+            data:        JSON.stringify(user: {chat: preferences.chat})
+            processData: true
+            success:     @success
+            error:       @error
+          )
+
+        # if we have more chats, let decide the user
+        else
+          msg = 'To be able to chat you need to select min. one chat topic below!'
+
+          # open modal settings
+          @settings(settings: msg)
     else
       @stopPushState()
       @pushState()
@@ -191,9 +211,10 @@ class App.CustomerChat extends App.Controller
     return if @windowCount() >= @maxChatWindows
     App.WebSocket.send(event:'chat_session_start')
 
-  settings: ->
+  settings: (errors = {}) ->
     new Setting(
       maxChatWindows: @maxChatWindows
+      errors: errors
     )
 
 class CustomerChatRouter extends App.ControllerPermanent
@@ -536,6 +557,7 @@ class Setting extends App.ControllerModal
     App.view('customer_chat/setting')(
       chats: App.Chat.all()
       preferences: preferences
+      errors: @errors
     )
 
   submit: (e) =>
@@ -544,7 +566,7 @@ class Setting extends App.ControllerModal
 
     @formDisable(e)
 
-    # get data
+    # update user preferences
     @ajax(
       id:          'preferences'
       type:        'PUT'
diff --git a/app/assets/javascripts/app/views/customer_chat/setting.jst.eco b/app/assets/javascripts/app/views/customer_chat/setting.jst.eco
index 70d3cc2d6..ed4f22deb 100644
--- a/app/assets/javascripts/app/views/customer_chat/setting.jst.eco
+++ b/app/assets/javascripts/app/views/customer_chat/setting.jst.eco
@@ -1,3 +1,17 @@
+<div class="select form-group">
+  <label for="a"><%- @T('Max. concurrent chats') %></label>
+  <div class="u-positionOrigin">
+    <select id="a" class="form-control" name="chat::max_windows">
+      <% for count in [1..20]: %>
+        <option value="<%- count %>" <% if parseInt(@preferences.chat.max_windows) is count: %>selected<% end %>><%- count %></option>
+      <% end %>
+    </select>
+    <%- @Icon('arrow-down', 'dropdown-arrow') %>
+  </div>
+</div>
+<% if @errors.settings: %>
+<div class="alert alert--danger"><%- @T(@errors.settings) %></div>
+<% end %>
 <table class="settings-list">
   <thead>
     <tr>
@@ -18,17 +32,4 @@
         </label>
 <% end %>
   </tbody>
-
-  <div class="select form-group">
-    <label for="a"><%- @T('Max. concurrent chats') %></label>
-    <div class="u-positionOrigin">
-      <select id="a" class="form-control" name="chat::max_windows">
-        <% for count in [1..20]: %>
-          <option value="<%- count %>" <% if parseInt(@preferences.chat.max_windows) is count: %>selected<% end %>><%- count %></option>
-        <% end %>
-      </select>
-      <%- @Icon('arrow-down', 'dropdown-arrow') %>
-    </div>
-  </div>
-
-</table>
\ No newline at end of file
+</table>