Closed
Bug 388251
Opened 18 years ago
Closed 17 years ago
Implement 'new Bugzilla::Attachment'
Categories
(Bugzilla :: Attachments & Requests, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.4
People
(Reporter: LpSolit, Assigned: LpSolit)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
11.57 KB,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
... instead of the current Bugzilla::Attachment->get(). Same for new_from_list() instead of get_list(). Said otherwise: use Object.pm! :)
Updated•18 years ago
|
Blocks: bz-email_in-attach
![]() |
Assignee | |
Updated•18 years ago
|
Severity: normal → enhancement
Target Milestone: Bugzilla 3.2 → Bugzilla 4.0
![]() |
Assignee | |
Comment 1•17 years ago
|
||
This bug is specific to |new Bugzilla::Attachment|. I also renamed ->insert_attachment_for_bug() to ->create(). I will update Attachment->create() to match how Object->create() works in a separate bug.
Target Milestone: Bugzilla 4.0 → Bugzilla 3.4
![]() |
Assignee | |
Comment 2•17 years ago
|
||
In a separate bug, I will implement ->create and ->update in a clean way.
Attachment #337486 -
Flags: review+
![]() |
Assignee | |
Comment 3•17 years ago
|
||
Checking in attachment.cgi;
/cvsroot/mozilla/webtools/bugzilla/attachment.cgi,v <-- attachment.cgi
new revision: 1.148; previous revision: 1.147
done
Checking in post_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/post_bug.cgi,v <-- post_bug.cgi
new revision: 1.197; previous revision: 1.196
done
Checking in Bugzilla/Attachment.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Attachment.pm,v <-- Attachment.pm
new revision: 1.58; previous revision: 1.57
done
Checking in Bugzilla/Flag.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Flag.pm,v <-- Flag.pm
new revision: 1.99; previous revision: 1.98
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: approval+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•