Merge pull request #1 from brospars/enabled-settings

Fix mit-teaching-systems-lab#33 add enabled settings

config/locales/server.en.yml

@@ -1,5 +1,6 @@
 en:
   site_settings:
+    lti_consumer_enabled: "Enable LTI authentication"
     lti_consumer_key: "LTI consumer key"
     lti_consumer_secret: "LTI consumer secret"
     lti_consumer_authenticate_url: 'EdX authentication URL (eg., course page with link to forums)'

config/settings.yml

@@ -1,4 +1,6 @@
 plugins:
+  lti_consumer_enabled:
+    default: true
   lti_consumer_key:
     default: 'foo'
     client: false

lti_authenticator.rb

@@ -7,6 +7,10 @@ def name
     'lti'
   end
 
+  def enabled?
+    SiteSetting.lti_consumer_enabled
+  end
+
   # override hook
   def register_middleware(omniauth)
     log :info, 'register_middleware'

plugin.rb

@@ -15,6 +15,7 @@
 
 # Enable site settings in admin UI.
 # See descriptions in files in the `config` folder.
+enabled_site_setting :lti_consumer_enabled
 enabled_site_setting :lti_consumer_key
 enabled_site_setting :lti_consumer_secret
 enabled_site_setting :lti_consumer_authenticate_url
@@ -25,9 +26,7 @@
 require_relative 'lti_authenticator.rb'
 auth_provider({
   title: 'Click to login with EdX',
-  message: 'Click to login with EdX',
   authenticator: LTIAuthenticator.new,
-  full_screen_login: true,
   custom_url: '/lti/redirect_to_consumer'
 })