Project

General

Profile

Actions

Defect #4849

closed

Could not upload picture into existing issue

Added by Andrew Rudenko about 14 years ago. Updated over 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Attachments
Target version:
-
Start date:
2010-02-15
Due date:
% Done:

0%

Estimated time:
Resolution:
Invalid
Affected version:

Description

  1. Create an issue
  2. Update the issue but add a picture this time
Actual Results:
  • Uploaded file size is 0 bytes in ./files/ directory.
  • Picture is added twice
  • there is RackMultipart file left in the /tmp directory of my system
    redmine@redmine.org ~ $ ls -l /tmp
    -rw------- 1 redmine redmine 1424 Feb 15 13:19 RackMultipart.15052.0

Note: It is OK if add picture while creating a new issue.

I'm on Redmine 0.9.2.devel.3434 (PostgreSQL)

About your application's environment
Ruby version              1.8.7 (i486-linux)
RubyGems version          1.3.5
Rack version              1.0
Rails version             2.3.5
Active Record version     2.3.5
Active Resource version   2.3.5
Action Mailer version     2.3.5
Active Support version    2.3.5
Edge Rails revision       unknown
Application root          /var/redmine
Environment               production
Database adapter          postgresql
Database schema version   20091227112908

About your Redmine plugins
Redmine Wiki Extensions plugin   0.2.0
Redmine Code Review plugin       0.2.9.3

Actions #1

Updated by Andrew Rudenko about 14 years ago

it is from log file while submitting attachment:

"attachments"=>{"1"=>{"file"=>#<File:/tmp/RackMultipart20100215-27685-6392x6-0>, "description"=>""}}

and it is directory listing:

-rw------- 1 redmine redmine 1424 Feb 15 14:37 RackMultipart20100215-27685-6392x6-0

p.s. Apache run under apache:apache user, but the file in tmp directory has 600 permissions. I am not sure but it can be the reason.

Actions #2

Updated by Andrew Rudenko about 14 years ago

I'm sorry to send this issue.

The problem was because of duplicated lines in app/controllers/issues_controller.rb
+ attachments = attach_files(@issue, params[:attachments])
+ attachments.each {|a| journal.details << JournalDetail.new(:property => 'attachment', :prop_key => a.id, :value => a.filename)}

I've merged it wrongly last time so I got two duplicated file names and zero file size.

Close it please as INVALID.

Actions #3

Updated by Felix Schäfer about 14 years ago

  • Status changed from New to Closed
  • Resolution set to Invalid
Actions #4

Updated by Shon Dunham over 12 years ago

SPAM

Actions

Also available in: Atom PDF