Replaced checkbox with dropdown (#552)

app/assets/javascripts/admins.js

@@ -106,11 +106,6 @@ function changeBrandingImage(path) {
   $.post(path, {url: url})
 }
 
-// Change whether or not user have to be signed in to join a room
-function changeRoomAuthentication(checked, path) {
-  $.post(path, {authenticationRequired: checked})
-}
-
 // Filters by role
 function filterRole(role) {
   search = new URL(location.href).searchParams.get('search')

app/controllers/admins_controller.rb

@@ -131,9 +131,9 @@ class AdminsController < ApplicationController
     redirect_to admins_path
   end
 
-  # POST /admins/meetingAuthentication
+  # POST /admins/room_authentication
   def room_authentication
-    @settings.update_value("Room Authentication", params[:authenticationRequired])
+    @settings.update_value("Room Authentication", params[:value])
     redirect_to admins_path
   end
 

app/controllers/concerns/themer.rb

@@ -17,31 +17,31 @@
 # with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 
 module Themer
-extend ActiveSupport::Concern
+  extend ActiveSupport::Concern
 
-# Lightens a color by 40%
-def color_lighten(color)
-  # Uses the built in Sass Engine to lighten the color
+  # Lightens a color by 40%
+  def color_lighten(color)
+    # Uses the built in Sass Engine to lighten the color
 
-  dummy_scss = "h1 { color: $lighten; }"
-  compiled = Sass::Engine.new("$lighten:lighten(#{color}, 40%);" + dummy_scss, syntax: :scss).render
+    dummy_scss = "h1 { color: $lighten; }"
+    compiled = Sass::Engine.new("$lighten:lighten(#{color}, 40%);" + dummy_scss, syntax: :scss).render
 
-  string_locater = 'color: '
-  color_start = compiled.index(string_locater) + string_locater.length
+    string_locater = 'color: '
+    color_start = compiled.index(string_locater) + string_locater.length
 
-  compiled[color_start..color_start + 6]
-end
-
-# Darkens a color by 10%
-def color_darken(color)
-  # Uses the built in Sass Engine to darken the color
-
-  dummy_scss = "h1 { color: $darken; }"
-  compiled = Sass::Engine.new("$darken:darken(#{color}, 10%);" + dummy_scss, syntax: :scss).render
-
-  string_locater = 'color: '
-  color_start = compiled.index(string_locater) + string_locater.length
-
-  compiled[color_start..color_start + 6]
-end
+    compiled[color_start..color_start + 6]
+  end
+
+  # Darkens a color by 10%
+  def color_darken(color)
+    # Uses the built in Sass Engine to darken the color
+
+    dummy_scss = "h1 { color: $darken; }"
+    compiled = Sass::Engine.new("$darken:darken(#{color}, 10%);" + dummy_scss, syntax: :scss).render
+
+    string_locater = 'color: '
+    color_start = compiled.index(string_locater) + string_locater.length
+
+    compiled[color_start..color_start + 6]
+  end
 end

app/controllers/rooms_controller.rb

@@ -98,8 +98,8 @@ class RoomsController < ApplicationController
 
   # POST /:room_uid
   def join
-    # If this setting is turned on only authenticated users are allowed to join rooms
-    room_authentication_required
+    return redirect_to root_path,
+      flash: { alert: I18n.t("administrator.site_settings.authentication.user-info") } if auth_required
 
     opts = default_meeting_options
     unless @room.owned_by?(current_user)
@@ -275,11 +275,8 @@ class RoomsController < ApplicationController
     redirect_to admins_path if current_user && current_user&.has_role?(:super_admin)
   end
 
-  def room_authentication_required
-    if Setting.find_or_create_by!(provider: user_settings_provider).get_value("Room Authentication") == "true" &&
-       current_user.nil?
-      flash[:alert] = I18n.t("administrator.site_settings.authentication.user-info")
-      redirect_to signin_path
-    end
+  def auth_required
+    Setting.find_or_create_by!(provider: user_settings_provider).get_value("Room Authentication") == "true" &&
+      current_user.nil?
   end
 end

app/helpers/admins_helper.rb

@@ -35,8 +35,12 @@ module AdminsHelper
     registration_method == Rails.configuration.registration_methods[:approval]
   end
 
-  def room_authentication_required
-    Setting.find_or_create_by!(provider: user_settings_provider).get_value("Room Authentication") == "true"
+  def room_authentication_string
+    if Setting.find_or_create_by!(provider: user_settings_provider).get_value("Room Authentication") == "true"
+      I18n.t("administrator.site_settings.authentication.enabled")
+    else
+      I18n.t("administrator.site_settings.authentication.disabled")
+    end
   end
 
   def registration_method_string

app/helpers/rooms_helper.rb

@@ -22,4 +22,9 @@ module RoomsHelper
   def google_calendar_path
     "http://calendar.google.com/calendar/r/eventedit?text=#{@room.name}&location=#{request.base_url + request.fullpath}"
   end
+
+  def room_authentication_required
+    Setting.find_or_create_by!(provider: user_settings_provider).get_value("Room Authentication") == "true" &&
+      current_user.nil?
+  end
 end

app/views/rooms/join.html.erb

@@ -14,7 +14,7 @@
 %>
 
 <%= render 'shared/room_event' do %>
-  <% if room_authentication_required && current_user.nil? %>
+  <% if room_authentication_required %>
     <h2><%= t("administrator.site_settings.authentication.user-info") %></h2>
   <% else %>
     <%= form_for room_path(@room), method: :post do |f| %>

app/views/shared/admin_settings/_site_settings.html.erb

@@ -16,7 +16,7 @@
 <div class="form-group">
   <div class="row">
     <div class="col-12">
-      <div class="mb-7 form-group">
+      <div class="mb-6 form-group">
         <label class="form-label"><%= t("administrator.site_settings.branding.title") %></label>
         <label class="form-label text-muted"><%= t("administrator.site_settings.branding.info") %></label>
         <div class="input-group">
@@ -30,20 +30,7 @@
   </div>
   <div class="row">
     <div class="col-12">
-      <div class="mb-7 form-group">
-        <label class="form-label"><%= t("administrator.site_settings.authentication.title") %></label>
-        <div class="row gutters-xs">
-          <label class="custom-control custom-checkbox ml-1">
-            <%= check_box_tag "room_authentication", '', room_authentication_required, class: 'custom-control-input', onchange: "changeRoomAuthentication(this.checked, '#{admin_room_authentication_path}')"%>
-            <span class="custom-control-label text-muted authentication-required"><%= t("administrator.site_settings.authentication.info") %></span>
-          </label>
-        </div>
-      </div>
-    </div>
-  </div>
-  <div class="row">
-    <div class="col-12">
-      <div class="mb-7 form-group">
+      <div class="mb-6 form-group">
         <label class="form-label"><%= t("administrator.site_settings.color.title") %></label>
         <label class="form-label text-muted"><%= t("administrator.site_settings.color.info") %></label>
           <div class="color-inputs">
@@ -68,7 +55,7 @@
     </div>
   </div>
   <div class="row">
-    <div class="col-12">
+    <div class="mb-6 col-12">
       <div class="form-group">
         <label class="form-label"><%= t("administrator.site_settings.registration.title") %></label>
         <label class="form-label text-muted"><%= t("administrator.site_settings.registration.info") %></label>
@@ -91,4 +78,25 @@
       </div>
     </div>
   </div>
+  <div class="row">
+    <div class="col-12">
+      <div class="form-group">
+        <label class="form-label"><%= t("administrator.site_settings.authentication.title") %></label>
+        <label class="form-label text-muted"><%= t("administrator.site_settings.authentication.info") %></label>
+        <div class="dropdown">
+          <button class="btn btn-primary dropdown-toggle" type="button" id="room-auth" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+            <%= room_authentication_string %>
+          </button>
+          <div class="dropdown-menu" aria-labelledby="room-auth">
+            <%= button_to admin_room_authentication_path(value: "true"), class: "dropdown-item" do %>
+              <%= t("administrator.site_settings.authentication.enabled") %>
+            <% end %>
+            <%= button_to admin_room_authentication_path(value: "false"), class: "dropdown-item" do %>
+              <%= t("administrator.site_settings.authentication.disabled") %>
+            <% end %>
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
 </div>

config/locales/en.yml

@@ -24,6 +24,8 @@ en:
   administrator:
     site_settings:
       authentication:
+        disabled: Disabled
+        enabled: Enabled
         info: Only allow authenticated users to join a room
         title: Require Authentication for Rooms
         user-info: You must sign in to Greenlight to join this room

spec/controllers/admins_controller_spec.rb

@@ -285,13 +285,12 @@ describe AdminsController, type: :controller do
         allow_any_instance_of(User).to receive(:greenlight_account?).and_return(true)
 
         @request.session[:user_id] = @admin.id
-        checked = true
 
-        post :room_authentication, params: { authenticationRequired: checked }
+        post :room_authentication, params: { value: "true" }
 
         feature = Setting.find_by(provider: "provider1").features.find_by(name: "Room Authentication")
 
-        expect(feature[:value]).to eq(checked.to_s)
+        expect(feature[:value]).to eq("true")
         expect(response).to redirect_to(admins_path)
       end
     end

spec/controllers/rooms_controller_spec.rb

@@ -212,7 +212,8 @@ describe RoomsController, type: :controller do
 
       post :join, params: { room_uid: @room }
 
-      expect(response).to redirect_to(signin_path)
+      expect(flash[:alert]).to be_present
+      expect(response).to redirect_to(root_path)
     end
   end