Added missing checks for whether task/experiment/project is active for experiment level permissions. Minor code refactoring and restyling.

app/permissions/experiment.rb

@@ -43,7 +43,8 @@ Canaid::Permissions.register_for(Experiment) do
      move_experiment)
     .each do |perm|
     can perm do |_, experiment|
-      experiment.project.active?
+      experiment.active? &&
+        experiment.project.active?
     end
   end
 end
@@ -97,7 +98,9 @@ Canaid::Permissions.register_for(MyModule) do
      complete_module
      create_comment_in_module).each do |perm|
     can perm do |_, my_module|
-      my_module.experiment.project.active?
+      my_module.active? &&
+        my_module.experiment.active? &&
+        my_module.experiment.project.active?
     end
   end
 end

app/views/canvas/edit/modal/_move_module.html.erb

@@ -6,7 +6,7 @@
         <h4 class="modal-title" id="modal-move-module-label"><%=t "experiments.canvas.edit.modal_move_module.title" %></h4>
       </div>
       <div class="modal-body">
-        <% experiments = @experiment.project.experiments.is_archived(false) %>
+        <% experiments = @experiment.project.experiments %>
         <% if experiments.count > 1 %>
           <%= bootstrap_form_tag do |f| %>
             <%= f.select :experiment_id, experiments
@@ -14,13 +14,13 @@
                                         .collect { |e| [ e.name, e.id ] }, {},
                 {class: "form-control selectpicker", "data-role" => "clear"} %>
           <% end %>
-      <% else %>
-        <div>
-          <em>
-            <%= t("experiments.canvas.edit.modal_move_module.no_experiments") %>
-          </em>
-        </div>
-      <% end %>
+        <% else %>
+          <div>
+            <em>
+              <%= t("experiments.canvas.edit.modal_move_module.no_experiments") %>
+            </em>
+          </div>
+        <% end %>
       </div>
       <div class="modal-footer">
       <% if experiments.count > 1 %>

config/locales/en.yml

@@ -1907,9 +1907,9 @@ en:
       leave_flash: "Successfuly left team %{team}."
     teams:
       create_permission_error: "You don't have permission to create team."
-      update_permission_error: "You don't have permission to edit team."
+      update_permission_error: "You don't have permission to edit this team."
     user:
       current_password_invalid: "incorrect password"
       password_confirmation_not_match: "doesn't match"
     invite_users:
-      permission_error: "You don't have permission to invite additional users to team. Contact its administrator/s."
+      permission_error: "You don't have permission to invite additional users to this team. Contact its administrator/s."