Merge pull request #85 from projekteuler/admin-set-translations

Allow admins to directly set translations

app/controllers/admin_controller.rb

@@ -1,8 +1,3 @@
 class AdminController < ApplicationController
-  before_action :authenticate!
+  before_action :authenticate_admin!
-
-  def authenticate!
-    authenticate_user!
-    throw(:warden) unless current_user.admin?
-  end
 end

app/controllers/application_controller.rb

@@ -2,4 +2,9 @@ class ApplicationController < ActionController::Base
   # Prevent CSRF attacks by raising an exception.
   # For APIs, you may want to use :null_session instead.
   protect_from_forgery with: :exception
+
+  def authenticate_admin!
+    authenticate_user!
+    throw(:warden) unless current_user.admin?
+  end
 end

app/controllers/translations_controller.rb

@@ -1,5 +1,6 @@
 class TranslationsController < ApplicationController
   before_action :set_problem, only: [:new, :create]
+  before_action :set_accept, only: [:create]
 
   # GET /translations/new
   def new
@@ -17,7 +18,12 @@ class TranslationsController < ApplicationController
       @translation.author = current_user
     end
     if @translation.save
-      redirect_to @problem, notice: t('translations.notice.successfully_created')
+      if @accept
+        @problem.set_translation(@translation)
+        redirect_to @problem, notice: t('translations.notice.successfully_created_and_accepted')
+      else
+        redirect_to @problem, notice: t('translations.notice.successfully_created')
+      end
     else
       render :new
     end
@@ -32,4 +38,12 @@ class TranslationsController < ApplicationController
     def set_problem
       @problem = Problem.find(params[:problem_id])
     end
+
+    def set_accept
+      if user_signed_in? and current_user.admin?
+        @accept = params.fetch(:accept, false)
+      else
+        @accept = false
+      end
+    end
 end

app/views/translations/_form.html.erb

@@ -20,4 +20,7 @@
     <% end %>
   </div>
   <%= f.submit %>
+  <% if user_signed_in? and current_user.admin? %>
+    <%= f.button t('.save_and_accept'), type: :submit, name: "accept", value: true, class: "btn-success" %>
+  <% end %>
 <% end %>

config/locales/views/translations/de.yml

@@ -8,7 +8,9 @@ de:
       translation_source_explanation: Hier kommt der HTML-Quelltext der Übersetzung hin.
       copyright_warning_html: Sie erklären sich einverstanden, dass die Übersetzung unter der Creative Commons Lizenz <a class="alert-link" href="https://creativecommons.org/licenses/by-nc-sa/4.0/deed.de" target="_blank">CC BY-NC-SA 4.0</a> veröffentlicht wird.
       not_logged_in_warning: Wenn Sie wollen, dass die Übersetzung Ihnen zugeordnet werden kann, müssen Sie sich einloggen (oben rechts), andernfalls ist sie anonym.
+      save_and_accept: Übersetzung direkt setzen
     new:
       new_translation: Neue Übersetzung für Problem %{id}
     notice:
-      successfully_created: Der Übersetzungsvorschlag wurde eingereicht. Ein Organisator wird sich Ihren Vorschlag so bald wie möglich anschauen.
+      successfully_created: Der Übersetzungsvorschlag wurde eingereicht. Ein Organisator wird sich Ihren Vorschlag so bald wie möglich anschauen.
+      successfully_created_and_accepted: Die Übersetzung wurde erfolgreich erstellt und akzeptiert.

test/controllers/translations_controller_test.rb

@@ -40,6 +40,28 @@ class TranslationsControllerTest < ActionDispatch::IntegrationTest
     assert_equal users(:translator), Translation.last.author
   end
 
+  test "should create and accept translation from admin" do
+    login_admin
+    assert_difference('Translation.count') do
+      post problem_translations_url(problem_id: 1, translation: @update, accept: true)
+    end
+
+    assert_redirected_to problem_path(id: 1)
+    assert_equal users(:admin), Translation.last.author
+    assert_equal Problem.find(1).translation, Translation.last
+  end
+
+  test "should create but not accept translation from normal user" do
+    login_translator
+    assert_difference('Translation.count') do
+      post problem_translations_url(problem_id: 1, translation: @update, accept: true)
+    end
+
+    assert_redirected_to problem_path(id: 1)
+    assert_equal users(:translator), Translation.last.author
+    assert_not_equal Problem.find(1).translation, Translation.last
+  end
+
   test "should not create incorrect translation" do
     assert_no_difference('Translation.count') do
       post problem_translations_url(problem_id: 1, translation: @incorrect)