When I was reviewing this new feature I failed to test the changing of the file name. The documentation states to use 'file_name' as the parameter but internally it is 'filename'. So I get the error: The requested method 'Bugzilla::Attachment::set_file_name' was not found. I will create a patch that converts 'file_name' to 'filename' before set_all() so that we can keep the param naming for attachments consistent. dkl
+1 for this. This should be the same name for add_attachment and update_attachment. Please change this or correct the POD!
Created attachment 806225 [details] [diff] [review] 917483_1.patch
Assignee: webservice → dkl
Status: NEW → ASSIGNED
Attachment #806225 - Flags: review?(simon)
Committing to: bzr+ssh://email@example.com/bugzilla/trunk modified Bugzilla/WebService/Bug.pm Committed revision 8762.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.