Make iD default editor

Make iD the default editor for all users that have not specified
a preferred editor, except fall back to P2 on IE.

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 0cd850cc689c2469a6a529cc96076a7af4d0ed7d..aec6120808d8f202e4dcf7e6ed231e1ac431e568 100644 (file)
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -422,6 +422,24 @@ class ApplicationController < ActionController::Base
     request.body.rewind
   end
 
     request.body.rewind
   end
 

+  def preferred_editor
+    editor = if params[:editor]
+      params[:editor]
+    elsif @user and @user.preferred_editor
+      @user.preferred_editor
+    else
+      DEFAULT_EDITOR
+    end
+
+    if request.env['HTTP_USER_AGENT'] =~ /MSIE/ and editor == 'id'
+      editor = 'potlatch2'
+    end
+
+    editor
+  end
+
+  helper_method :preferred_editor
+

 private 
 
   # extract authorisation credentials from headers, returns user = nil if none
 private 
 
   # extract authorisation credentials from headers, returns user = nil if none

diff --git a/app/controllers/site_controller.rb b/app/controllers/site_controller.rb
index b2212645c2761d764e03287a23b8dc6f79af0ec1..69ce8f8b1f07ef9d018c6137103d0dcac6e020a7 100644 (file)
--- a/app/controllers/site_controller.rb
+++ b/app/controllers/site_controller.rb
@@ -52,7 +52,7 @@ class SiteController < ApplicationController
   end
 
   def edit
   end
 
   def edit

-    editor = params[:editor] || @user.preferred_editor || DEFAULT_EDITOR
+    editor = preferred_editor

 
     if editor == "remote"
       render :action => :index
 
     if editor == "remote"
       render :action => :index

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 160e25e3581aae47d488974dd49de6d95fe8f644..4765cb35b3b6fe1fbd506bfc248d32454fc8e09a 100644 (file)
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -57,16 +57,6 @@ module ApplicationHelper
     content_tag(tag, capture(&block), :class => "hide_unless_administrator")
   end
 
     content_tag(tag, capture(&block), :class => "hide_unless_administrator")
   end
 

-  def preferred_editor
-    if params[:editor]
-      params[:editor]
-    elsif @user and @user.preferred_editor
-      @user.preferred_editor
-    else
-      DEFAULT_EDITOR
-    end
-  end
-

   def scale_to_zoom(scale)
     Math.log(360.0 / (scale.to_f * 512.0)) / Math.log(2.0)
   end
   def scale_to_zoom(scale)
     Math.log(360.0 / (scale.to_f * 512.0)) / Math.log(2.0)
   end

diff --git a/config/example.application.yml b/config/example.application.yml
index 9a6d83aba9a2671643f65a8ac8c1c0ec30312d71..c6c15bb1d4003eb79409536b53ee19ea320ae499 100644 (file)
--- a/config/example.application.yml
+++ b/config/example.application.yml
@@ -73,7 +73,7 @@ defaults: &defaults
   # URL of Nominatim instance to use for geocoding
   nominatim_url: "http://nominatim.openstreetmap.org/"
   # Default editor
   # URL of Nominatim instance to use for geocoding
   nominatim_url: "http://nominatim.openstreetmap.org/"
   # Default editor

-  default_editor: "potlatch2"
+  default_editor: "id"

   # OAuth consumer key for Potlatch 2
   #potlatch2_key: ""
   # OAuth consumer key for the web site
   # OAuth consumer key for Potlatch 2
   #potlatch2_key: ""
   # OAuth consumer key for the web site