id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
24664	TemporaryFileUploadHandler new_file() does not match its parent class	David Danier	nobody	"TemporaryFileUploadHandler.new_file(self, file_name, *args, **kwargs)
vs.
FileUploadHandler.new_file(self, field_name, file_name, content_type, content_length, charset=None, content_type_extra=None)

TemporaryFileUploadHandler.new_file will take ""file_name"" as its first argument and pass this (still as first argument) to FileUploadHandler.new_file. So the argument will end up FileUploadHandler.field_name, which is wrong.

I think a valid fix would be to remove file_name from TemporaryFileUploadHandler.new_file as the parameter is not used besides for calling the parent class. It may have been used for passing it to TemporaryUploadedFile, but this must be history. I'll attach a patch to change this."	Cleanup/optimization	closed	Core (Other)	1.8	Normal	fixed			Ready for checkin	1	0	0	0	1	0