# HG changeset patch # User Jacob Kemme # Date 1469205401 14400 # Fri Jul 22 12:36:41 2016 -0400 # Branch gentex-dev # Node ID c84a03a3c00578ccea6bba3c1e6140bc6fff9023 # Parent 8d668eb6890f23d838b2276262c600545582ddb2 This commit add the REST API for: - getting the list of files for a project and its versions. - creating a new file from an uploaded attachment. diff --git a/app/controllers/files_controller.rb b/app/controllers/files_controller.rb --- a/app/controllers/files_controller.rb +++ b/app/controllers/files_controller.rb @@ -20,7 +20,9 @@ before_action :find_project_by_project_id before_action :authorize + accept_api_auth :index, :create + helper :attachments helper :sort include SortHelper @@ -35,7 +37,10 @@ references(:attachments).reorder(sort_clause).find(@project.id)] @containers += @project.versions.includes(:attachments). references(:attachments).reorder(sort_clause).to_a.sort.reverse - render :layout => !request.xhr? + respond_to do |format| + format.html { render :layout => !request.xhr? } + format.api + end end def new @@ -43,20 +48,29 @@ end def create - container = (params[:version_id].blank? ? @project : @project.versions.find_by_id(params[:version_id])) - attachments = Attachment.attach_files(container, params[:attachments]) + version_id = params[:version_id] || (params[:file] && params[:file][:version_id]) + container = version_id.blank? ? @project : @project.versions.find_by_id(version_id) + attachments = Attachment.attach_files(container, (params[:attachments] || (params[:file] && params[:file][:token] && params))) render_attachment_warning_if_needed(container) if attachments[:files].present? if Setting.notified_events.include?('file_added') Mailer.attachments_added(attachments[:files]).deliver end - flash[:notice] = l(:label_file_added) - redirect_to project_files_path(@project) + respond_to do |format| + format.html { + flash[:notice] = l(:label_file_added) + redirect_to project_files_path(@project) } + format.api { render_api_ok } + end else - flash.now[:error] = l(:label_attachment) + " " + l('activerecord.errors.messages.invalid') - new - render :action => 'new' + respond_to do |format| + format.html { + flash.now[:error] = l(:label_attachment) + " " + l('activerecord.errors.messages.invalid') + new + render :action => 'new' } + format.api { render :status => :bad_request } + end end end end diff --git a/app/helpers/attachments_helper.rb b/app/helpers/attachments_helper.rb --- a/app/helpers/attachments_helper.rb +++ b/app/helpers/attachments_helper.rb @@ -51,7 +51,7 @@ end end - def render_api_attachment(attachment, api) + def render_api_attachment(attachment, api, options={}) api.attachment do api.id attachment.id api.filename attachment.filename @@ -64,6 +64,7 @@ end api.author(:id => attachment.author.id, :name => attachment.author.name) if attachment.author api.created_on attachment.created_on + options.each { |key, value| eval("api.#{key} value") } end end end diff --git a/test/integration/api_test/api_routing_test.rb b/test/integration/api_test/api_routing_test.rb --- a/test/integration/api_test/api_routing_test.rb +++ b/test/integration/api_test/api_routing_test.rb @@ -32,6 +32,11 @@ should_route 'GET /enumerations/issue_priorities' => 'enumerations#index', :type => 'issue_priorities' end + def test_files + should_route 'GET /projects/foo/files' => 'files#index', :project_id => 'foo' + should_route 'POST /projects/foo/files' => 'files#create', :project_id => 'foo' + end + def test_groups should_route 'GET /groups' => 'groups#index' should_route 'POST /groups' => 'groups#create' diff --git a/test/integration/api_test/files_test.rb b/test/integration/api_test/files_test.rb new file mode 100644 --- /dev/null +++ b/test/integration/api_test/files_test.rb @@ -0,0 +1,115 @@ +# Redmine - project management software +# Copyright (C) 2006-2015 Jean-Philippe Lang +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License +# as published by the Free Software Foundation; either version 2 +# of the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + +require File.expand_path('../../../test_helper', __FILE__) + +class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base + fixtures :projects, + :users, + :members, + :roles, + :member_roles, + :enabled_modules, + :attachments, + :versions + + test "GET /projects/:project_id/files.xml should return the list of uploaded files" do + get '/projects/1/files.xml', {}, credentials('jsmith') + assert_response :success + assert_select 'files attachment id', :text => '8' + end + + test "POST /projects/:project_id/files.json should create a file" do + set_tmp_attachments_directory + post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + token = Attachment.last.token + payload = <<-JSON +{ "file": { + "token": "#{token}" + } +} + JSON + post '/projects/1/files.json', payload, {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + assert_response :success + assert_equal 1, Attachment.last.container_id + assert_equal "Project", Attachment.last.container_type + end + + test "POST /projects/:project_id/files.xml should create a file" do + set_tmp_attachments_directory + post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + token = Attachment.last.token + payload = <<-XML + + #{token} + + XML + post '/projects/1/files.xml', payload, {"CONTENT_TYPE" => 'application/xml'}.merge(credentials('jsmith')) + assert_response :success + assert_equal 1, Attachment.last.container_id + assert_equal "Project", Attachment.last.container_type + end + + test "POST /projects/:project_id/files.json should refuse requests without the :token parameter" do + payload = <<-JSON +{ "file": { + "filename": "project_file.zip", + } +} + JSON + post '/projects/1/files.json', payload, {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + assert_response :bad_request + end + + test "POST /projects/:project_id/files.json should accept :filename, :description, :content_type as optional parameters" do + set_tmp_attachments_directory + post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + token = Attachment.last.token + payload = <<-JSON +{ "file": { + "filename": "New filename", + "description": "New description", + "content_type": "application/txt", + "token": "#{token}" + } +} + JSON + post '/projects/1/files.json', payload, {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + assert_response :success + assert_equal "New filename", Attachment.last.filename + assert_equal "New description", Attachment.last.description + assert_equal "application/txt", Attachment.last.content_type + end + + test "POST /projects/:project_id/files.json should accept :version_id to attach the files to a version" do + set_tmp_attachments_directory + post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + token = Attachment.last.token + payload = <<-JSON +{ "file": { + "version_id": 3, + "filename": "New filename", + "description": "New description", + "token": "#{token}" + } +} + JSON + post '/projects/1/files.json', payload, {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + assert_equal 3, Attachment.last.container_id + assert_equal "Version", Attachment.last.container_type + end +end \ No newline at end of file