diff options
author | Frédéric Buclin <LpSolit@gmail.com> | 2011-10-01 13:19:55 +0200 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2011-10-01 13:19:55 +0200 |
commit | 613d9ce787fd65d6c186bb76716b3e1a48e1521e (patch) | |
tree | 3834b2b3f6f8c160cdaee1e304a8734e5327f3f8 /Bugzilla | |
parent | e4f8a784dea705fcc2d29183e5c899e4e6d8d6d1 (diff) | |
download | bugs-613d9ce787fd65d6c186bb76716b3e1a48e1521e.tar bugs-613d9ce787fd65d6c186bb76716b3e1a48e1521e.tar.gz bugs-613d9ce787fd65d6c186bb76716b3e1a48e1521e.tar.bz2 bugs-613d9ce787fd65d6c186bb76716b3e1a48e1521e.tar.xz bugs-613d9ce787fd65d6c186bb76716b3e1a48e1521e.zip |
Bug 582529: Ambiguous error message "You did not specify a file to attach" when deleting an existing attachment filename
a=LpSolit
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/Attachment.pm | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm index a39dc3af4..3a8e7d5d5 100644 --- a/Bugzilla/Attachment.pm +++ b/Bugzilla/Attachment.pm @@ -565,7 +565,14 @@ sub _check_filename { my ($invocant, $filename) = @_; $filename = clean_text($filename); - $filename || ThrowUserError('file_not_specified'); + if (!$filename) { + if (ref $invocant) { + ThrowUserError('filename_not_specified'); + } + else { + ThrowUserError('file_not_specified'); + } + } # Remove path info (if any) from the file name. The browser should do this # for us, but some are buggy. This may not work on Mac file names and could |