From b64e200b91bdde0f545223d67d4daec9fed064d2 Mon Sep 17 00:00:00 2001 From: Luka Murn Date: Thu, 28 Jul 2016 16:41:10 +0200 Subject: [PATCH] Refactor canvas a lot. Also refactor localization files concerning canvas. --- app/controllers/canvas_controller.rb | 5 +- app/controllers/my_module_tags_controller.rb | 2 +- app/controllers/my_modules_controller.rb | 12 +- app/controllers/projects_controller.rb | 8 - app/helpers/permission_helper.rb | 8 +- app/helpers/sidebar_helper.rb | 2 +- app/views/canvas/_edit.html.erb | 30 +-- app/views/canvas/edit/_my_module.html.erb | 14 +- .../canvas/edit/modal/_delete_module.html.erb | 6 +- .../edit/modal/_delete_module_group.html.erb | 6 +- .../canvas/edit/modal/_edit_module.html.erb | 12 +- .../edit/modal/_edit_module_group.html.erb | 12 +- .../canvas/edit/modal/_new_module.html.erb | 12 +- .../canvas/full_zoom/_my_module.html.erb | 2 +- app/views/experiments/canvas.html.erb | 6 +- app/views/my_module_comments/_index.html.erb | 8 +- app/views/my_module_tags/_index_edit.html.erb | 34 ++-- app/views/my_modules/_activities.html.erb | 6 +- .../my_modules/_description_label.html.erb | 2 +- app/views/my_modules/_due_date_label.html.erb | 2 +- .../_module_header_due_date_label.html.erb | 2 +- app/views/my_modules/_show.html.erb | 4 +- .../modals/_manage_module_tags_modal.html.erb | 2 +- .../modals/_manage_users_modal.html.erb | 10 +- app/views/projects/canvas.html.erb | 39 ---- app/views/projects/index/_project.html.erb | 2 +- app/views/projects/show.html.erb | 2 + app/views/sample_my_modules/_index.html.erb | 6 +- app/views/user_my_modules/_index.html.erb | 10 +- .../user_my_modules/_index_edit.html.erb | 8 +- config/locales/en.yml | 177 +++++++++--------- config/routes.rb | 4 +- 32 files changed, 205 insertions(+), 250 deletions(-) delete mode 100644 app/views/projects/canvas.html.erb diff --git a/app/controllers/canvas_controller.rb b/app/controllers/canvas_controller.rb index 0cddfc386..c07e59ad3 100644 --- a/app/controllers/canvas_controller.rb +++ b/app/controllers/canvas_controller.rb @@ -232,7 +232,7 @@ class CanvasController < ApplicationController end flash[:success] = t( - "projects.canvas.update.success_flash") + "experiments.canvas.update.success_flash") redirect_to canvas_experiment_path(@experiment) end @@ -254,7 +254,6 @@ class CanvasController < ApplicationController def load_vars @experiment = Experiment.find_by_id(params[:id]) - @project = @experiment.project unless @experiment respond_to do |format| format.html { render_404 and return } @@ -266,7 +265,7 @@ class CanvasController < ApplicationController end def check_edit_canvas - unless can_edit_canvas(@project) + unless can_edit_canvas(@experiment) render_403 and return end end diff --git a/app/controllers/my_module_tags_controller.rb b/app/controllers/my_module_tags_controller.rb index b683034f8..aa1dbf916 100644 --- a/app/controllers/my_module_tags_controller.rb +++ b/app/controllers/my_module_tags_controller.rb @@ -8,7 +8,7 @@ class MyModuleTagsController < ApplicationController @my_module_tags = @my_module.my_module_tags @unassigned_tags = @my_module.unassigned_tags @new_mmt = MyModuleTag.new(my_module: @my_module) - @new_tag = Tag.new(project: @my_module.project) + @new_tag = Tag.new(project: @my_module.experiment.project) respond_to do |format| format.json { diff --git a/app/controllers/my_modules_controller.rb b/app/controllers/my_modules_controller.rb index 58da7d3a9..08f9e2f63 100644 --- a/app/controllers/my_modules_controller.rb +++ b/app/controllers/my_modules_controller.rb @@ -147,7 +147,7 @@ class MyModulesController < ApplicationController # Currently not in use Activity.create( type_of: :archive_module, - project: @my_module.project, + project: @my_module.experiment.project, my_module: @my_module, user: current_user, message: t( @@ -162,7 +162,7 @@ class MyModulesController < ApplicationController if saved Activity.create( type_of: :restore_module, - project: @my_module.project, + project: @my_module.experiment.project, my_module: @my_module, user: current_user, message: t( @@ -178,7 +178,7 @@ class MyModulesController < ApplicationController if saved and description_changed then Activity.create( type_of: :change_module_description, - project: @my_module.project, + project: @my_module.experiment.project, my_module: @my_module, user: current_user, message: t( @@ -240,7 +240,7 @@ class MyModulesController < ApplicationController def samples @samples_index_link = samples_index_my_module_path(@my_module, format: :json) - @organization = @my_module.project.organization + @organization = @my_module.experiment.project.organization end def archive @@ -293,7 +293,7 @@ class MyModulesController < ApplicationController # AJAX actions def samples_index - @organization = @my_module.project.organization + @organization = @my_module.experiment.project.organization respond_to do |format| format.html @@ -309,7 +309,7 @@ class MyModulesController < ApplicationController @direct_upload = ENV['PAPERCLIP_DIRECT_UPLOAD'] == "true" @my_module = MyModule.find_by_id(params[:id]) if @my_module - @project = @my_module.project + @project = @my_module.experiment.project else render_404 end diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb index 62a7deb30..5f3fd4296 100644 --- a/app/controllers/projects_controller.rb +++ b/app/controllers/projects_controller.rb @@ -12,7 +12,6 @@ class ProjectsController < ApplicationController before_action :check_view_notifications_permissions, only: [ :notifications ] before_action :check_edit_permissions, only: [ :edit ] before_action :check_module_archive_permissions, only: [:module_archive] - before_action :check_canvas_permissions, only: [:workflow] filter_by_archived = false @@ -311,13 +310,6 @@ class ProjectsController < ApplicationController end end - def check_canvas_permissions - @project = Project.find_by_id(wf_params[:id]) - unless can_edit_canvas(@project) - render_403 - end - end - def check_module_archive_permissions unless can_restore_archived_modules(@project) render_403 diff --git a/app/helpers/permission_helper.rb b/app/helpers/permission_helper.rb index c134b182c..cd181e8c8 100644 --- a/app/helpers/permission_helper.rb +++ b/app/helpers/permission_helper.rb @@ -323,8 +323,8 @@ module PermissionHelper # ---- WORKFLOW PERMISSIONS ---- - def can_edit_canvas(project) - is_user_or_higher_of_project(project) + def can_edit_canvas(experiment) + is_user_or_higher_of_project(experiment.project) end def can_reposition_modules(experiment) @@ -378,11 +378,11 @@ module PermissionHelper end def can_add_tag_to_module(my_module) - is_user_or_higher_of_project(my_module.project) + is_user_or_higher_of_project(my_module.experiment.project) end def can_remove_tag_from_module(my_module) - is_user_or_higher_of_project(my_module.project) + is_user_or_higher_of_project(my_module.experiment.project) end def can_view_module_info(my_module) diff --git a/app/helpers/sidebar_helper.rb b/app/helpers/sidebar_helper.rb index f23d1a7dd..93afa232d 100644 --- a/app/helpers/sidebar_helper.rb +++ b/app/helpers/sidebar_helper.rb @@ -15,7 +15,7 @@ module SidebarHelper when "archive" return module_archive_project_url(project) else - return canvas_project_path(project) + return project_path(project) end end diff --git a/app/views/canvas/_edit.html.erb b/app/views/canvas/_edit.html.erb index 4cd74b2e9..8c01d4206 100644 --- a/app/views/canvas/_edit.html.erb +++ b/app/views/canvas/_edit.html.erb @@ -6,16 +6,16 @@ data-can-delete-modules="<%= can_archive_modules(@experiment) ? "yes" : "no" %>" data-can-reposition-modules="<%= can_reposition_modules(@experiment) ? "yes" : "no" %>" data-can-edit-connections="<%= can_edit_connections(@experiment) ? "yes" : "no" %>" - data-unsaved-work-text="<%=t "projects.canvas.edit.unsaved_work" %>" + data-unsaved-work-text="<%=t "experiments.canvas.edit.unsaved_work" %>" > <%= bootstrap_form_tag url: canvas_experiment_url, method: "post" do |f| %>
<%= f.submit class: "btn btn-primary", id: "canvas-save" do %> - <%= t("projects.canvas.edit.save_short") %> - + <%= t("experiments.canvas.edit.save_short") %> + <% end %> <%= link_to canvas_experiment_path(@experiment), type: "button", class: "btn btn-default cancel-edit-canvas" do %> - +   <% end %>
@@ -23,12 +23,12 @@ <%=link_to "", type: "button", class: "btn btn-default", id: "canvas-new-module" do %> - <%= t("projects.canvas.edit.new_module") %> + <%= t("experiments.canvas.edit.new_module") %> - - <%= t("projects.canvas.edit.new_module") %> + + <%= t("experiments.canvas.edit.new_module") %> <% end %> <% end %> @@ -42,28 +42,28 @@ <%= hidden_field_tag 'module-groups', '{}' %> <% end %>
diff --git a/app/views/canvas/edit/_my_module.html.erb b/app/views/canvas/edit/_my_module.html.erb index bfaae0f62..c9670ce93 100644 --- a/app/views/canvas/edit/_my_module.html.erb +++ b/app/views/canvas/edit/_my_module.html.erb @@ -23,30 +23,30 @@
  • <%= t('projects.index.options_header') %>
    • <% if can_edit_module(my_module) %>
  • - <%=t "projects.canvas.edit.edit_module" %> + <%=t "experiments.canvas.edit.edit_module" %>
    • <% end %> <% if can_edit_module_groups(my_module.experiment) %>
  • > - <%=t "projects.canvas.edit.edit_module_group" %> + <%=t "experiments.canvas.edit.edit_module_group" %>
    • <% end %> <% if can_clone_modules(my_module.experiment) %>
  • - <%=t "projects.canvas.edit.clone_module" %> + <%=t "experiments.canvas.edit.clone_module" %>
  • > - <%=t "projects.canvas.edit.clone_module_group" %> + <%=t "experiments.canvas.edit.clone_module_group" %>
    • <% end %> <% if can_archive_module(my_module) %>
  • - <%=t "projects.canvas.edit.delete_module" %> + <%=t "experiments.canvas.edit.delete_module" %>
    • <% end %> <% if can_archive_modules(my_module.experiment) %>
  • > - <%=t "projects.canvas.edit.delete_module_group" %> + <%=t "experiments.canvas.edit.delete_module_group" %>
    • <% end %> @@ -56,7 +56,7 @@ <% if can_edit_connections(my_module.experiment) %>
    - <%=t "projects.canvas.edit.drag_connections" %> + <%=t "experiments.canvas.edit.drag_connections" %>
    <% end %> diff --git a/app/views/canvas/edit/modal/_delete_module.html.erb b/app/views/canvas/edit/modal/_delete_module.html.erb index dc9e57043..90a4b58da 100644 --- a/app/views/canvas/edit/modal/_delete_module.html.erb +++ b/app/views/canvas/edit/modal/_delete_module.html.erb @@ -3,15 +3,15 @@
    - +
    - +
    - +
    diff --git a/app/views/canvas/edit/modal/_delete_module_group.html.erb b/app/views/canvas/edit/modal/_delete_module_group.html.erb index 8e8b03b7a..9d6e82f2a 100644 --- a/app/views/canvas/edit/modal/_delete_module_group.html.erb +++ b/app/views/canvas/edit/modal/_delete_module_group.html.erb @@ -3,15 +3,15 @@
    - +
    - +
    - +
    diff --git a/app/views/canvas/edit/modal/_edit_module.html.erb b/app/views/canvas/edit/modal/_edit_module.html.erb index f1ae36479..6f4c6b091 100644 --- a/app/views/canvas/edit/modal/_edit_module.html.erb +++ b/app/views/canvas/edit/modal/_edit_module.html.erb @@ -3,25 +3,25 @@
    - +
    <%= bootstrap_form_tag do |f| %> - <%= f.text_field t("projects.canvas.edit.modal_edit_module.name"), id: "edit-module-name-input" %> + <%= f.text_field t("experiments.canvas.edit.modal_edit_module.name"), id: "edit-module-name-input" %> <% end %>
    - +
    diff --git a/app/views/canvas/edit/modal/_edit_module_group.html.erb b/app/views/canvas/edit/modal/_edit_module_group.html.erb index cbdf51fd6..f8f576453 100644 --- a/app/views/canvas/edit/modal/_edit_module_group.html.erb +++ b/app/views/canvas/edit/modal/_edit_module_group.html.erb @@ -3,25 +3,25 @@
    - +
    <%= bootstrap_form_tag do |f| %> - <%= f.text_field t("projects.canvas.edit.modal_edit_module_group.name"), id: "edit-module-group-name-input" %> + <%= f.text_field t("experiments.canvas.edit.modal_edit_module_group.name"), id: "edit-module-group-name-input" %> <% end %>
    - +
    diff --git a/app/views/canvas/edit/modal/_new_module.html.erb b/app/views/canvas/edit/modal/_new_module.html.erb index 9c70387b4..42059ad01 100644 --- a/app/views/canvas/edit/modal/_new_module.html.erb +++ b/app/views/canvas/edit/modal/_new_module.html.erb @@ -3,25 +3,25 @@
    - +
    <%= bootstrap_form_tag do |f| %> - <%= f.text_field t("projects.canvas.edit.modal_new_module.name"), placeholder: t("projects.canvas.edit.modal_new_module.name_placeholder"), id: "new-module-name-input" %> + <%= f.text_field t("experiments.canvas.edit.modal_new_module.name"), placeholder: t("experiments.canvas.edit.modal_new_module.name_placeholder"), id: "new-module-name-input" %> <% end %>
    - +
    diff --git a/app/views/canvas/full_zoom/_my_module.html.erb b/app/views/canvas/full_zoom/_my_module.html.erb index e9c985506..11c46ec2c 100644 --- a/app/views/canvas/full_zoom/_my_module.html.erb +++ b/app/views/canvas/full_zoom/_my_module.html.erb @@ -32,7 +32,7 @@
    - <%= link_to_if can_edit_module(my_module), t("projects.canvas.full_zoom.due_date"), due_date_my_module_path(my_module, format: :json), remote: true, class: "due-date-link" %> + <%= link_to_if can_edit_module(my_module), t("experiments.canvas.full_zoom.due_date"), due_date_my_module_path(my_module, format: :json), remote: true, class: "due-date-link" %>
    <% if can_edit_module(my_module) %> diff --git a/app/views/experiments/canvas.html.erb b/app/views/experiments/canvas.html.erb index edbad799e..7ba7e19b2 100644 --- a/app/views/experiments/canvas.html.erb +++ b/app/views/experiments/canvas.html.erb @@ -1,10 +1,10 @@ -<% provide(:head_title, raw(t("projects.canvas.head_title", project: @project.name))) %> +<% provide(:head_title, raw(t("experiments.canvas.head_title", project: @project.name))) %> <%= render partial: "shared/sidebar" %> <%= render partial: "shared/secondary_navigation" %>
    - <% if can_edit_canvas(@project) %> - <%=link_to t("projects.canvas.canvas_edit"), canvas_edit_experiment_url(@experiment), remote: true, type: "button", id: "edit-canvas-button", class: "ajax btn btn-default", "data-action" => "edit" %> + <% if can_edit_canvas(@experiment) %> + <%=link_to t("experiments.canvas.canvas_edit"), canvas_edit_experiment_url(@experiment), remote: true, type: "button", id: "edit-canvas-button", class: "ajax btn btn-default", "data-action" => "edit" %> <% end %>
    <%=link_to canvas_full_zoom_experiment_path(@experiment), remote: true, type: "button", class: "ajax btn btn-primary active", "data-action" => "full_zoom", "data-toggle" => "button", "aria-pressed" => true do %> diff --git a/app/views/my_module_comments/_index.html.erb b/app/views/my_module_comments/_index.html.erb index 516dd1ba5..aff54c799 100644 --- a/app/views/my_module_comments/_index.html.erb +++ b/app/views/my_module_comments/_index.html.erb @@ -1,15 +1,15 @@ -
    <%= t('projects.canvas.popups.comments_tab') %>
    +
    <%= t('experiments.canvas.popups.comments_tab') %>
      <% if @comments.size == 0 then %> -
    • <%= t 'projects.canvas.popups.no_comments' %>
      • +
    • <%= t 'experiments.canvas.popups.no_comments' %>
      • <% else %> <%= render 'my_module_comments/list.html.erb', comments: @comments %> <% end %> <% if @comments.length == @per_page %>
    • - <%=t "projects.canvas.popups.more_comments" %> + <%=t "experiments.canvas.popups.more_comments" %>
      • <% end %> @@ -19,7 +19,7 @@
    • <%= bootstrap_form_for :comment, url: { format: :json }, method: :post, remote: true do |f| %> - <%= f.text_field :message, hide_label: true, placeholder: t("projects.canvas.popups.comment_placeholder"), append: f.submit("+"), help: '.' %> + <%= f.text_field :message, hide_label: true, placeholder: t("experiments.canvas.popups.comment_placeholder"), append: f.submit("+"), help: '.' %> <% end %>
    diff --git a/app/views/my_module_tags/_index_edit.html.erb b/app/views/my_module_tags/_index_edit.html.erb index 7167c7acd..0b485a09d 100644 --- a/app/views/my_module_tags/_index_edit.html.erb +++ b/app/views/my_module_tags/_index_edit.html.erb @@ -1,6 +1,6 @@ -
    <%=t "projects.canvas.modal_manage_tags.subtitle", module: @my_module.name %>
    +
    <%=t "experiments.canvas.modal_manage_tags.subtitle", module: @my_module.name %>
    <% if @my_module_tags.size == 0 then %> -
    <%= t 'projects.canvas.modal_manage_tags.no_tags' %>
    +
    <%= t 'experiments.canvas.modal_manage_tags.no_tags' %>
    <% else %>
      <% @my_module_tags.each_with_index do |mmt, i| tag = mmt.tag %> @@ -11,20 +11,20 @@

      <%= tag.name %>

    - <% if can_edit_tag(@my_module.project) then %> - <%= link_to "", remote: true, class: 'btn btn-link edit-tag-link', title: t("projects.canvas.modal_manage_tags.edit_tag") do %> + <% if can_edit_tag(@my_module.experiment.project) then %> + <%= link_to "", remote: true, class: 'btn btn-link edit-tag-link', title: t("experiments.canvas.modal_manage_tags.edit_tag") do %> <% end %> <% end %> <% if can_remove_tag_from_module(@my_module) then %> - <%= link_to my_module_my_module_tag_path(@my_module, mmt, format: :json), method: :delete, remote: true, class: 'btn btn-link remove-tag-link', title: t("projects.canvas.modal_manage_tags.remove_tag", module: @my_module.name) do %> + <%= link_to my_module_my_module_tag_path(@my_module, mmt, format: :json), method: :delete, remote: true, class: 'btn btn-link remove-tag-link', title: t("experiments.canvas.modal_manage_tags.remove_tag", module: @my_module.name) do %> <% end %> <% end %> - <% if can_delete_tag(@my_module.project) then %> - <%= bootstrap_form_for tag, remote: true, url: project_tag_path(@my_module.project, tag, format: :json), method: :delete, html: { class: "delete-tag-form"} do |f| %> + <% if can_delete_tag(@my_module.experiment.project) then %> + <%= bootstrap_form_for tag, remote: true, url: project_tag_path(@my_module.experiment.project, tag, format: :json), method: :delete, html: { class: "delete-tag-form"} do |f| %> <%= hidden_field_tag :my_module_id, @my_module.id %> - <%= f.button class: 'btn btn-link delete-tag-link', title: t("projects.canvas.modal_manage_tags.delete_tag") do %> + <%= f.button class: 'btn btn-link delete-tag-link', title: t("experiments.canvas.modal_manage_tags.delete_tag") do %> <% end %> <% end %> @@ -32,19 +32,19 @@
    - <% if can_edit_tag(@my_module.project) %> + <% if can_edit_tag(@my_module.experiment.project) %>
    <% end %> <% end %> - <% if can_create_new_tag(@my_module.project) then %> + <% if can_create_new_tag(@my_module.experiment.project) then %>
    - <%= bootstrap_form_for [@my_module.project, @new_tag], remote: true, format: :json, html: { class: 'add-tag-form' } do |f| %> + <%= bootstrap_form_for [@my_module.experiment.project, @new_tag], remote: true, format: :json, html: { class: 'add-tag-form' } do |f| %> <%= hidden_field_tag :my_module_id, @my_module.id %> - <%= f.hidden_field :project_id, :value => @my_module.project.id %> + <%= f.hidden_field :project_id, :value => @my_module.experiment.project.id %> <%= f.hidden_field :name, :value => t("tags.create.new_name") %> <%= f.hidden_field :color, :value => TAG_COLORS[0] %> <%= f.button class: "btn btn-primary" do %> - + <% end %> <% end %>
    diff --git a/app/views/my_modules/_activities.html.erb b/app/views/my_modules/_activities.html.erb index 6229f2eb0..d012c9cc6 100644 --- a/app/views/my_modules/_activities.html.erb +++ b/app/views/my_modules/_activities.html.erb @@ -1,8 +1,8 @@ -
    <%= t("projects.canvas.popups.activities_tab") %>
    +
    <%= t("experiments.canvas.popups.activities_tab") %>
      <% if @activities.size == 0 then %> -
    • <%= t 'projects.canvas.popups.no_activities' %>
      • +
    • <%= t 'experiments.canvas.popups.no_activities' %>
      • <% else %> <% @activities.each do |activity| %>
    • <%=l activity.created_at, format: :full %> @@ -11,5 +11,5 @@ <% end %> <% end %>
      -
    • <%= link_to t("projects.canvas.popups.more_activities"), activities_my_module_path(@my_module) %>
      • +
    • <%= link_to t("experiments.canvas.popups.more_activities"), activities_my_module_path(@my_module) %>
    diff --git a/app/views/my_modules/_description_label.html.erb b/app/views/my_modules/_description_label.html.erb index 46f1027da..c1974bbb0 100644 --- a/app/views/my_modules/_description_label.html.erb +++ b/app/views/my_modules/_description_label.html.erb @@ -1,5 +1,5 @@ <% if @my_module.description.blank? %> - <%=t "projects.canvas.popups.no_description" %> + <%=t "experiments.canvas.popups.no_description" %> <% else %> <%= @my_module.description %> <% end %> \ No newline at end of file diff --git a/app/views/my_modules/_due_date_label.html.erb b/app/views/my_modules/_due_date_label.html.erb index 53740adad..20ed19628 100644 --- a/app/views/my_modules/_due_date_label.html.erb +++ b/app/views/my_modules/_due_date_label.html.erb @@ -4,5 +4,5 @@ <% end %> <% else %> - <%=t "projects.canvas.full_zoom.no_due_date" %> + <%=t "experiments.canvas.full_zoom.no_due_date" %> <% end %> diff --git a/app/views/my_modules/_module_header_due_date_label.html.erb b/app/views/my_modules/_module_header_due_date_label.html.erb index c193aae34..975331bbc 100644 --- a/app/views/my_modules/_module_header_due_date_label.html.erb +++ b/app/views/my_modules/_module_header_due_date_label.html.erb @@ -1,5 +1,5 @@ <% if @my_module.due_date.blank? %> - <%=t "projects.canvas.full_zoom.no_due_date" %> + <%=t "experiments.canvas.full_zoom.no_due_date" %> <% else %> <%= l(@my_module.due_date, format: :full) %> <% end %> \ No newline at end of file diff --git a/app/views/my_modules/_show.html.erb b/app/views/my_modules/_show.html.erb index 7212250fd..b497e1756 100644 --- a/app/views/my_modules/_show.html.erb +++ b/app/views/my_modules/_show.html.erb @@ -1,4 +1,4 @@ -
    <%=t "projects.canvas.popups.info_tab" %>
    +
    <%=t "experiments.canvas.popups.info_tab" %>
    • @@ -9,7 +9,7 @@ <% if can_edit_module(@my_module) %>
    • - <%= link_to t("projects.canvas.popups.full_info"), description_my_module_path(@my_module, format: :json), class: "description-link", remote: true %> + <%= link_to t("experiments.canvas.popups.full_info"), description_my_module_path(@my_module, format: :json), class: "description-link", remote: true %>
      • <% end %>
    \ No newline at end of file diff --git a/app/views/my_modules/modals/_manage_module_tags_modal.html.erb b/app/views/my_modules/modals/_manage_module_tags_modal.html.erb index 1c3e2edb1..23e34bfda 100644 --- a/app/views/my_modules/modals/_manage_module_tags_modal.html.erb +++ b/app/views/my_modules/modals/_manage_module_tags_modal.html.erb @@ -3,7 +3,7 @@
    - +
    diff --git a/app/views/my_modules/modals/_manage_users_modal.html.erb b/app/views/my_modules/modals/_manage_users_modal.html.erb index f56a9fd92..74508ab1c 100644 --- a/app/views/my_modules/modals/_manage_users_modal.html.erb +++ b/app/views/my_modules/modals/_manage_users_modal.html.erb @@ -3,16 +3,16 @@
    - +
    - <% if is_admin_of_organization(@project.organization) %> - <%= link_to t("projects.canvas.full_zoom.modal_manage_users.invite_users_link"), organization_path(organization_id: @project.organization.id) %> - <%=t "projects.canvas.full_zoom.modal_manage_users.invite_users_details", organization: @project.organization.name %> + <% if is_admin_of_organization(@experiment.project.organization) %> + <%= link_to t("experiments.canvas.full_zoom.modal_manage_users.invite_users_link"), organization_path(organization_id: @experiment.project.organization.id) %> + <%=t "experiments.canvas.full_zoom.modal_manage_users.invite_users_details", organization: @experiment.project.organization.name %> <% else %> - <%=t "projects.canvas.full_zoom.modal_manage_users.contact_admins", organization: @project.organization.name %> + <%=t "experiments.canvas.full_zoom.modal_manage_users.contact_admins", organization: @experiment.project.organization.name %> <% end %> diff --git a/app/views/projects/canvas.html.erb b/app/views/projects/canvas.html.erb deleted file mode 100644 index 07459fcb7..000000000 --- a/app/views/projects/canvas.html.erb +++ /dev/null @@ -1,39 +0,0 @@ -<% provide(:head_title, raw(t("projects.canvas.head_title", project: @project.name))) %> -<%= render partial: "shared/sidebar" %> -<%= render partial: "shared/secondary_navigation" %> - -
    - <% if can_edit_canvas(@project) %> - <%=link_to t("projects.canvas.canvas_edit"), canvas_edit_project_url(@project), remote: true, type: "button", id: "edit-canvas-button", class: "ajax btn btn-default", "data-action" => "edit" %> - <% end %> -
    - <%=link_to canvas_full_zoom_project_path(@project), remote: true, type: "button", class: "ajax btn btn-primary active", "data-action" => "full_zoom", "data-toggle" => "button", "aria-pressed" => true do %> - - <% end %> - <%=link_to canvas_medium_zoom_project_path(@project), remote: true, type: "button", class: "ajax btn btn-primary", "data-action" => "medium_zoom" do %> - - <% end %> - <%=link_to canvas_small_zoom_project_path(@project), remote: true, type: "button", class: "ajax btn btn-primary", "data-action" => "small_zoom" do %> - - <% end %> -
    -
    -
    - <%= render partial: 'canvas/full_zoom', locals: { project: @project, my_modules: @project.active_modules } %> -
    - - -<%= render partial: "my_modules/modals/manage_module_tags_modal", locals: { my_module: nil } %> - - -<%= javascript_include_tag("jsPlumb-2.0.4-min") %> -<%= javascript_include_tag("jsnetworkx") %> -<%= javascript_include_tag("eventPause-min") %> - -<%= javascript_include_tag("projects/canvas") %> diff --git a/app/views/projects/index/_project.html.erb b/app/views/projects/index/_project.html.erb index d46293f6a..baa6a8446 100644 --- a/app/views/projects/index/_project.html.erb +++ b/app/views/projects/index/_project.html.erb @@ -33,7 +33,7 @@ data-project-users-tab-url="<%= url_for project_user_projects_path(project_id: p <% end %> <% if can_view_project(project) then %> - <%#= link_to project.name, canvas_project_path(project), id: "#{project.id}-project-canvas-link" %> + <%= link_to project.name, project_path(project), id: "#{project.id}-project-canvas-link" %> <% else %> <%= project.name %> <% end %> diff --git a/app/views/projects/show.html.erb b/app/views/projects/show.html.erb index b696f3705..9b228dbe5 100644 --- a/app/views/projects/show.html.erb +++ b/app/views/projects/show.html.erb @@ -1,3 +1,5 @@ <% provide(:head_title, raw(t("projects.show.head_title", project: @project.name))) %> <%= render partial: "shared/sidebar" %> <%= render partial: "shared/secondary_navigation" %> + +TODO \ No newline at end of file diff --git a/app/views/sample_my_modules/_index.html.erb b/app/views/sample_my_modules/_index.html.erb index 3bf2a0704..c4d03088d 100644 --- a/app/views/sample_my_modules/_index.html.erb +++ b/app/views/sample_my_modules/_index.html.erb @@ -1,13 +1,13 @@ -
    <%= t('projects.canvas.popups.samples_tab') %>
    +
    <%= t('experiments.canvas.popups.samples_tab') %>
      <% if @number_of_samples == 0 then %> -
    • <%= t 'projects.canvas.popups.no_samples' %>
      • +
    • <%= t 'experiments.canvas.popups.no_samples' %>
      • <% else %> <% @samples.each do |sample| %>
    • <%= sample.name %>
      • <% end %> <% end %>
      -
    • <%= link_to t("projects.canvas.popups.manage_samples"), samples_my_module_url(id: @my_module.id) %>
      • +
    • <%= link_to t("experiments.canvas.popups.manage_samples"), samples_my_module_url(id: @my_module.id) %>
    diff --git a/app/views/user_my_modules/_index.html.erb b/app/views/user_my_modules/_index.html.erb index 893c16b90..2617d4805 100644 --- a/app/views/user_my_modules/_index.html.erb +++ b/app/views/user_my_modules/_index.html.erb @@ -1,8 +1,8 @@ -
    <%=t "projects.canvas.popups.users_tab" %>
    +
    <%=t "experiments.canvas.popups.users_tab" %>
      <% if @user_my_modules.size == 0 then %> -
    • <%= t "projects.canvas.popups.no_users" %>
      • +
    • <%= t "experiments.canvas.popups.no_users" %>
      • <% else %> <% @user_my_modules.each_with_index do |user_my_module, i| %> <% user = user_my_module.user %> @@ -14,9 +14,9 @@
    <%= user.full_name %>
    - "> + "> - <%=t "projects.canvas.popups.module_user_join", date: l(user_my_module.created_at, format: :full_date) %> + <%=t "experiments.canvas.popups.module_user_join", date: l(user_my_module.created_at, format: :full_date) %>
    @@ -28,6 +28,6 @@ <% if can_edit_users_on_module(@my_module) then %>

    - <%= link_to t('projects.canvas.popups.manage_users'), my_module_users_edit_path(@my_module, format: :json), remote: true, class: "manage-users-link" %> + <%= link_to t('experiments.canvas.popups.manage_users'), my_module_users_edit_path(@my_module, format: :json), remote: true, class: "manage-users-link" %>

    <% end %> \ No newline at end of file diff --git a/app/views/user_my_modules/_index_edit.html.erb b/app/views/user_my_modules/_index_edit.html.erb index efdd4d535..6aa5c6991 100644 --- a/app/views/user_my_modules/_index_edit.html.erb +++ b/app/views/user_my_modules/_index_edit.html.erb @@ -1,7 +1,7 @@
      <% if @user_my_modules.size == 0 then %> -
    • <%= t 'projects.canvas.full_zoom.modal_manage_users.no_users' %>
      • +
    • <%= t 'experiments.canvas.full_zoom.modal_manage_users.no_users' %>
      • <% else %> <% @user_my_modules.each_with_index do |umm, i| user = umm.user %>
    • @@ -14,8 +14,8 @@
      <%= user.full_name %> -
      "> - <%=t "projects.canvas.full_zoom.modal_manage_users.user_join", date: l(umm.created_at, format: :full_date) %> +
      "> + <%=t "experiments.canvas.full_zoom.modal_manage_users.user_join", date: l(umm.created_at, format: :full_date) %>
      @@ -43,7 +43,7 @@
      <%= f.button class: 'btn btn-primary' do %> - + <% end %>
      <% end %> diff --git a/config/locales/en.yml b/config/locales/en.yml index b0ab655cf..12f896e0c 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -376,93 +376,6 @@ en: restore_option: "Restore" archived_on: "Archived on" archived_on_title: "Task archived on %{date} at %{time}." - canvas: - head_title: "%{project} | Overview" - canvas_edit: "Edit workflow" - modal_manage_tags: - head_title: "Manage tags for" - subtitle: "Showing tags of task %{module}" - no_tags: "No tags!" - edit_tag: "Edit tag." - remove_tag: "Remove tag from task %{module}." - delete_tag: "Permanently delete tag from all tasks." - save_tag: "Save tag." - cancel_tag: "Cancel changes to the tag." - create: "Add tag" - create_new: "Create new tag" - edit: - new_module: "New task" - new_module_hover: "Drag me onto canvas" - save: "Save workflow" - save_short: "Save" - cancel: "Cancel" - unsaved_work: "Are you sure you want to leave this page? All unsaved data will be lost." - drag_connections: "Drag connection/s from here" - options_header: "Options" - edit_module: "Rename task" - edit_module_group: "Rename workflow" - clone_module: "Clone task" - clone_module_group: "Clone workflow" - delete_module: "Archive task" - delete_module_group: "Archive workflow" - modal_new_module: - title: "Add new task" - name: "Task name" - name_placeholder: "My task" - confirm: "Add task" - error_length: "Name must contain from 2 to 50 characters." - error_invalid: "Name contains invalid characters ('|')." - error_whitespaces: "Name cannot be blank." - modal_edit_module: - title: "Rename task" - name: "Task name" - confirm: "Rename task" - modal_edit_module_group: - title: "Rename workflow" - name: "Workflow name" - confirm: "Rename workflow" - modal_delete_module: - title: "Archive task" - confirm: "Archive task" - message: "Are you sure you wish to archive task %{module}? Task's samples and position will be removed." - modal_delete_module_group: - title: "Archive workflow" - confirm: "Archive workflow" - message: "Are you sure you wish to archive the workflow task %{module} belongs to? All workflow tasks' samples and positions will be removed." - popups: - info_tab: "Task info" - no_description: "This task has no description." - full_info: "Edit description" - users_tab: "Assigned users" - no_users: "This task has no assigned users." - manage_users: "Manage users" - module_user_join: "Joined on %{date}." - module_user_join_full: "%{user} joined on %{date} at %{time}." - activities_tab: "Activity" - no_activities: "No activities!" - more_activities: "All activities" - comments_tab: "Comments" - no_comments: "No comments!" - more_comments: "More Comments" - comment_placeholder: "Your Message" - new_comment: "New comment" - samples_tab: "Samples" - no_samples: "No samples!" - manage_samples: "Manage samples" - full_zoom: - due_date: "Due date" - no_due_date: "not set" - modal_manage_users: - modal_title: "Manage users for" - no_users: "No users" - user_join: "Joined on %{date}." - user_join_full: "%{user} joined on %{date} at %{time}." - create: "Add" - invite_users_link: "Invite users" - invite_users_details: "to team %{organization}." - contact_admins: "To invite additional users to team %{organization}, contact its administrator/s." - update: - success_flash: "Project successfully updated." user_organizations: enums: @@ -678,7 +591,95 @@ en: error_flash: 'Could not update the experiment.' archive: success_flash: "Successfully archived experiment %{experiment}" - error_flash: 'Could not archive the experiment.' + error_flash: 'Could not archive the experiment.' + canvas: + head_title: "%{project} | Overview" + canvas_edit: "Edit workflow" + modal_manage_tags: + head_title: "Manage tags for" + subtitle: "Showing tags of task %{module}" + no_tags: "No tags!" + edit_tag: "Edit tag." + remove_tag: "Remove tag from task %{module}." + delete_tag: "Permanently delete tag from all tasks." + save_tag: "Save tag." + cancel_tag: "Cancel changes to the tag." + create: "Add tag" + create_new: "Create new tag" + edit: + new_module: "New task" + new_module_hover: "Drag me onto canvas" + save: "Save workflow" + save_short: "Save" + cancel: "Cancel" + unsaved_work: "Are you sure you want to leave this page? All unsaved data will be lost." + drag_connections: "Drag connection/s from here" + options_header: "Options" + edit_module: "Rename task" + edit_module_group: "Rename workflow" + clone_module: "Clone task" + clone_module_group: "Clone workflow" + delete_module: "Archive task" + delete_module_group: "Archive workflow" + modal_new_module: + title: "Add new task" + name: "Task name" + name_placeholder: "My task" + confirm: "Add task" + error_length: "Name must contain from 2 to 50 characters." + error_invalid: "Name contains invalid characters ('|')." + error_whitespaces: "Name cannot be blank." + modal_edit_module: + title: "Rename task" + name: "Task name" + confirm: "Rename task" + modal_edit_module_group: + title: "Rename workflow" + name: "Workflow name" + confirm: "Rename workflow" + modal_delete_module: + title: "Archive task" + confirm: "Archive task" + message: "Are you sure you wish to archive task %{module}? Task's samples and position will be removed." + modal_delete_module_group: + title: "Archive workflow" + confirm: "Archive workflow" + message: "Are you sure you wish to archive the workflow task %{module} belongs to? All workflow tasks' samples and positions will be removed." + popups: + info_tab: "Task info" + no_description: "This task has no description." + full_info: "Edit description" + users_tab: "Assigned users" + no_users: "This task has no assigned users." + manage_users: "Manage users" + module_user_join: "Joined on %{date}." + module_user_join_full: "%{user} joined on %{date} at %{time}." + activities_tab: "Activity" + no_activities: "No activities!" + more_activities: "All activities" + comments_tab: "Comments" + no_comments: "No comments!" + more_comments: "More Comments" + comment_placeholder: "Your Message" + new_comment: "New comment" + samples_tab: "Samples" + no_samples: "No samples!" + manage_samples: "Manage samples" + full_zoom: + due_date: "Due date" + no_due_date: "not set" + modal_manage_users: + modal_title: "Manage users for" + no_users: "No users" + user_join: "Joined on %{date}." + user_join_full: "%{user} joined on %{date} at %{time}." + create: "Add" + invite_users_link: "Invite users" + invite_users_details: "to team %{organization}." + contact_admins: "To invite additional users to team %{organization}, contact its administrator/s." + update: + success_flash: "Project successfully updated." + my_module_tags: new: head_title: "%{project} | %{module} | Add tag" diff --git a/config/routes.rb b/config/routes.rb index e755969e6..b4af37a09 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -103,13 +103,13 @@ Rails.application.routes.draw do resources :experiments, only: [:show] do member do - get 'canvas' # Overview/structure for single project + get 'canvas' # Overview/structure for single experiment get 'canvas/edit', to: 'canvas#edit' # AJAX-loaded canvas edit mode (from canvas) get 'canvas/full_zoom', to: 'canvas#full_zoom' # AJAX-loaded canvas zoom get 'canvas/medium_zoom', to: 'canvas#medium_zoom' # AJAX-loaded canvas zoom get 'canvas/small_zoom', to: 'canvas#small_zoom' # AJAX-loaded canvas zoom post 'canvas', to: 'canvas#update' # Save updated canvas action - get 'module_archive' # Module archive for single project + get 'module_archive' # Module archive for single experiment end end