Permite que questões em uso sejam editadas

app/assets/javascripts/questions.js.erb

@@ -175,6 +175,7 @@ function back(){
   $.fancybox.update();
   $('.block_content').css('width', 'auto');
 }
+
 function back_medias(){
   go_to_medias();
 }
@@ -245,56 +246,106 @@ function edit_question(icon, replace_list){
    return false;
   }
 
+  var questions_ids;
+  var ids;
+
   if(replace_list != '.list_exams'){
-    var questions_ids = $('.ckb_question:checked', $(icon).parents("div.list_questions")).map(function() { return this.value; }).get();
-    var ids = questions_ids;
+      questions_ids = $('.ckb_question:checked', $(icon).parents("div.list_questions")).map(function() { return this.value; }).get();
+      ids = questions_ids
   }else{
-    var ids = $('.ckb_question:checked', $(icon).parents("[id^='exam_']")).map(function() { return this.value; }).get();
-    var questions_ids = $('.ckb_question:checked', $(icon).parents("[id^='exam_']")).map(function() { return $(this).data('question-id'); }).get();
-    var exam_id = $('.ckb_question:checked', $(icon).parents("[id^='exam_']")).map(function() { return $(this).data('exam-id'); }).get();
+      ids = $('.ckb_question:checked', $(icon).parents("[id^='exam_']")).map(function() { return this.value; }).get();
+      questions_ids = $('.ckb_question:checked', $(icon).parents("[id^='exam_']")).map(function() { return $(this).data('question-id'); }).get();
+      var exam_id = $('.ckb_question:checked', $(icon).parents("[id^='exam_']")).map(function() { return $(this).data('exam-id'); }).get();
   }
 
   if (!(!ids.length || ids.length > 1)) {
-      var base_verify_url = "<%=Rails.application.routes.url_helpers.verify_owners_question_path(':id')%>".replace(':id', questions_ids);
+    var base_verify_url = "<%=Rails.application.routes.url_helpers.verify_edit_permissions_question_path(':id')%>".replace(':id', questions_ids);
 
     $.get(exam_id ? (base_verify_url + `/${exam_id}`) : base_verify_url, function(data){
-      var url_edit = $(icon).data('link-to-edit').replace(':id', ids);
-      $(icon).call_fancybox({
-          href : url_edit,
-          open: true,
-          'autoDimensions': false,
-          'autoSize': false,
-          width: '700',
-          height: 'auto',
-          //maxHeight: '70%',
-          beforeClose: function() { clear_ckeditor(); }
-      });
-    }).fail(function(data){
 
-      var data = $.parseJSON(data.responseText);
-        if (!confirm(data.alert))
-          return this;
-
-        $.get("<%=Rails.application.routes.url_helpers.copy_verify_owners_question_path(':id')%>".replace(':id', questions_ids), function(data){
-            var url_copy = $(icon).prev().data('link-to-copy').replace(':id', ids);
-            $(icon).prev().call_fancybox({ href : url_copy, open: true, maxHeight: '70%', 'autoDimensions': false, 'autoSize': false, width: '700', height: 'auto', });
-            if(replace_list != ''){
-                $.get($(replace_list).data("link-list"), function(data2){
-                    $(replace_list).replaceWith(data2);
-                    flash_message(data.notice, 'notice');
-                    update_nice_checkbox();
-                });
+        if (data.action === 'only_points') {
+            if (!confirm(data.alert))
+                return this;
+        } else if (data.action === 'can_fork_copy') {
+            if (confirm(data.alert)) {
+                data.action = 'only_points';
+            } else {
+                if (confirm("<%=I18n.t('questions.error.fork_copy')%>")) {
+                    data.action = 'fork_copy';
+                }
+                return this;
             }
-        }).fail(function(data){
-            var data = $.parseJSON(data.responseText);
-            if (typeof(data.alert) != "undefined")
-                flash_message(data.alert, 'alert');
-        });
+        }
+
+      if (['copy_repository', 'copy', 'fork_copy'].includes(data.action)) {
+
+          if (data.action !== 'fork_copy'){
+              if (!confirm(data.alert))
+                  return this;
+          }
+
+          var url_copy;
+
+          if (data.action === 'copy_repository') {
+              url_copy = "<%=Rails.application.routes.url_helpers.copy_question_path(':id')%>".replace(':id', questions_ids);
+          } else if (data.action === 'fork_copy') {
+              url_copy = "<%=Rails.application.routes.url_helpers.fork_copy_exam_question_path(':id')%>".replace(':id', ids);
+          } else {
+              url_copy = $(icon).prev().data('link-to-copy').replace(':id', ids);
+          }
+
+          $(icon).prev().call_fancybox({
+              href : url_copy,
+              open: true,
+              maxHeight: '70%',
+              'autoDimensions': false,
+              'autoSize': false,
+              width: '700',
+              height: 'auto',
+              beforeClose: function() { clear_ckeditor(); }
+          });
+
+          if(replace_list !== ''){
+              $.get($(replace_list).data("link-list"), function(data2){
+                  $(replace_list).replaceWith(data2);
+                  flash_message(data.notice, 'notice');
+                  update_nice_checkbox();
+              });
+          }
+      } else {
+
+          var url_edit = $(icon).data('link-to-edit').replace(':id', ids) + (data.action === 'only_points' ? "?only_points=true" : "");
+
+          if (data.notify) {
+              if (!confirm("<%=I18n.t('questions.questions.notify_editors_alert')%>")) {
+                  return this;
+              }
+              url_edit += "?notify_responsibles=true";
+          }
+
+          $(icon).call_fancybox({
+              href : url_edit,
+              open: true,
+              'autoDimensions': false,
+              'autoSize': false,
+              width: '700',
+              height: 'auto',
+              beforeClose: function() { clear_ckeditor(); }
+          });
+      }
+    }).fail(function(data){
+        var data = $.parseJSON(data.responseText);
+        if (typeof(data.alert) != "undefined")
+        flash_message(data.alert, 'alert');
     });
+  }else{
+      flash_message("<%=I18n.t('questions.error.choose_one')%>", "alert");
+      return false;
   }
 }
 
 function copy_question(icon, replace_list){
+
   if ($(icon).attr('disabled') == 'disabled'){
    flash_message("<%=I18n.t('questions.error.choose_one')%>", "alert");
    return false;
@@ -410,6 +461,24 @@ function expand_or_compress_icon(icon){
   }
 }
 
+function toggle_alt_info(link){
+    var parent = $(link).parent();
+    parent.siblings().focus();
+    var information = parent.siblings('.alt_info_text')
+
+    if(information.hasClass('invisible')){
+        information.removeClass('invisible')
+        parent.find('i.icon-arrow-up-triangle').removeClass('invisible');
+        parent.find('i.icon-arrow-down-triangle').addClass('invisible');
+    }else{
+        information.addClass('invisible')
+        parent.find('i.icon-arrow-up-triangle').addClass('invisible');
+        parent.find('i.icon-arrow-down-triangle').removeClass('invisible');
+    }
+
+    $(link).focus();
+}
+
 function annul(icon){
   if ($(icon).prop('disabled'))
     return false;

app/controllers/exam_questions_controller.rb

@@ -24,7 +24,15 @@ def new
   def create
     @exam_question = ExamQuestion.new exam_question_params
     authorize! :create, Question, { on: at_ids = @exam_question.exam.allocation_tags.map(&:id) }
-    @exam_question.question.user_id = current_user.id
+
+    if params[:forked_question_id].present?
+      @forked_question = ExamQuestion.find(params[:forked_question_id])
+      @exam_question.question.user_id = @forked_question.question.user_id
+      @exam_question.question.updated_by_user_id = current_user.id unless current_user.id == @forked_question.question.user_id
+      @exam_question.question.status = true
+    else
+      @exam_question.question.user_id = current_user.id
+    end
 
     ActiveRecord::Base.transaction do
       if !params['question_texts']['text'].strip.blank?
@@ -37,6 +45,12 @@ def create
       end
 
       if @exam_question.save
+
+        if @forked_question.present?
+          log(@forked_question.exam, "question: #{@forked_question.question.id} [moved/deleted after copy] exam: #{@forked_question.exam.id}", LogAction::TYPE[:update]) rescue nil
+          @forked_question.destroy
+        end
+
         if @exam_question.exam.questions.size > 1
           render partial: 'question', locals: { question: @exam_question.question, exam_question: @exam_question, exam: @exam_question.exam, hide_columns: false, can_see_preview: true }
         else
@@ -60,6 +74,7 @@ def create
 
   def edit
     @exam_question = ExamQuestion.find(params[:id])
+    @only_points = params[:only_points].nil? ? false : true
     @question_text = QuestionText.find(@exam_question.question.question_text_id) unless @exam_question.question.question_text_id.blank?
     build_exam_question
   end
@@ -89,6 +104,13 @@ def update
     end
 
     if @exam_question.update_attributes exam_question_params
+      if params[:notify_responsibles] == 'true'
+        message = t('questions.questions.notification', question_enunciation: @exam_question.question.enunciation)
+        @exam_question.question.exams.each do |exam|
+          exam.notify_responsibles(message)
+        end
+      end
+
       if !params['question_texts_id'].blank? && params['question_texts']['media_question'].to_i == 0
         if Question.where(:question_text_id => params['question_texts_id']).count == 0
           QuestionText.find(params['question_texts_id']).destroy
@@ -111,6 +133,28 @@ def update
     render_json_error(error, 'questions.error')
   end
 
+  def fork_copy
+    authorize! :copy, Question
+
+    original_exam_question = ExamQuestion.find params[:id]
+    original_question = original_exam_question.question
+    original_question.can_copy?
+
+    @new_question = original_question.dup
+
+    copy_exam_question_associations(original_question, @new_question)
+
+    @exam_question = ExamQuestion.new(original_exam_question.attributes.except('id', 'question_id', 'annulled').merge({ annulled: false }))
+    @exam_question.question = @new_question
+    @forked_question = original_exam_question
+
+    render :new
+  rescue CanCan::AccessDenied
+    render text: t(:no_permission)
+  rescue => error
+    render text: t("exam_questions.errors.#{error}")
+  end
+
   def remove_file_item
     authorize! :update, Question
 
@@ -401,6 +445,24 @@ def log(object, message, type=LogAction::TYPE[:update])
     end
   end
 
+  def copy_exam_question_associations(source_question, target_question)
+    source_question.question_items.each do |item|
+      target_question.question_items << item.dup
+    end
+
+    source_question.question_images.each do |img|
+      target_question.question_images << img.dup
+    end
+
+    source_question.question_labels.each do |label|
+      target_question.question_labels << label.dup
+    end
+
+    source_question.question_audios.each do |audio|
+      target_question.question_audios << audio.dup
+    end
+  end
+
   def build_exam_question
     if @exam_question.question.nil?
       @exam_question.build_question do |q|

app/controllers/questions_controller.rb

@@ -2,7 +2,7 @@ class QuestionsController < ApplicationController
 
   include SysLog::Actions
 
-  before_action :set_current_user, only: [:destroy, :change_status, :verify_owners, :copy, :index]
+  before_action :set_current_user, only: [:destroy, :change_status, :verify_owners, :verify_edit_permissions, :copy, :index]
 
   layout false, except: :index
 
@@ -93,6 +93,13 @@ def update
     end
 
     if @question.update_attributes question_params
+      if params[:notify_responsibles] == 'true' && @question.previous_changes.present?
+          message = t('questions.questions.notification', question_enunciation: @question.enunciation)
+          @question.exams.each do |exam|
+            exam.notify_responsibles(message)
+          end
+      end
+
       render partial: 'question', locals: { question: @question }
     else
       @errors = []
@@ -108,6 +115,29 @@ def update
     render_json_error(error, 'questions.error')
   end
 
+  def verify_edit_permissions
+    question = Question.find params[:id]
+    exam_id = params[:exam_id]
+
+    edition_status = question.verify_edit_permission(current_user, exam_id)
+
+    if [:edit_full, :edit_and_notify].include?(edition_status)
+      authorize! :update, Question
+      render json: { action: 'edit', notify: edition_status == :edit_and_notify }, status: :ok
+    else
+      repository_action =  [:copy_blocked_exam, :copy_ownership].include?(edition_status) ? :copy : edition_status
+      render json: { action: repository_action, alert: I18n.t(get_edit_permissions_alert_key(edition_status), scope: [:questions, :error]) }, status: :ok
+    end
+
+  rescue RuntimeError => error
+    if error.message == 'in_use'
+      error_message = I18n.translate!("#{error.message}", scope: [:questions, :error], :raise => true) rescue t('general_message', scope: [:questions, :error])
+      render json: { success: false, alert: error_message }, status: :forbidden
+    end
+  rescue => error
+    render_json_error(error, 'questions.error')
+  end
+
   def destroy
     authorize! :destroy, Question
     ActiveRecord::Base.transaction do
@@ -135,10 +165,6 @@ def show
 
   def verify_owners
     question = Question.find params[:id]
-    if params[:update]
-      question.can_change?(params[:exam_id])
-      authorize! :update, Question
-    end
     if params[:copy]
       question.can_copy?
       authorize! :copy, Question
@@ -154,6 +180,7 @@ def verify_owners
   end
 
   def copy
+
     authorize! :copy, Question
 
     question  = Question.find params[:id]
@@ -189,4 +216,21 @@ def log(object, message, type=LogAction::TYPE[:update])
     LogAction.create(params_to_log.merge!(description: message, log_type: type))
   end
 
+  def get_edit_permissions_alert_key(status)
+    case status
+    when :copy_blocked_exam
+      'in_use_question'
+    when :copy_ownership
+      'copy_ownership'
+    when :copy_repository
+      'copy_repository'
+    when :only_points
+      'only_points'
+    when :can_fork_copy
+      'can_fork_copy'
+    else
+      'in_use'
+    end
+  end
+
 end

app/models/allocation.rb

@@ -475,6 +475,19 @@ def calculate_grade_and_hours
     end
   end
 
+  def self.send_email_to_responsible_users(allocation_tag_ids ,subject, message_body)
+    responsibles_list = responsibles(allocation_tag_ids)
+    emails = responsibles_list.each do |responsible|
+      responsible.user.email
+    end
+    clean_emails = emails.compact.uniq
+
+    if emails.any?
+      Job.send_mass_email(clean_emails, subject, message_body)
+    end
+
+  end
+
   private
 
     def verify_profile