Closed Bug 302684 Opened 19 years ago Closed 18 years ago

move code that creates a new attachment from attachment.cgi to Attachment.pm

Categories

(Bugzilla :: Attachments & Requests, enhancement)

Product:
Bugzilla
Component:
Attachments & Requests
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 3.0

People

(Reporter: timello, Assigned: Wurblzap)

References

Details

(Whiteboard: [blocker will fix]) 

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050524 Firefox/1.0 (Ubuntu package 1.0.2 MFSA2005-44)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050524 Firefox/1.0 (Ubuntu package 1.0.2 MFSA2005-44)

<myk> timello: seems like that code could be moved to Attachment.pm so that
attachment.cgi just called the appropriate Attachment.pm function, passing it
the data (probably after validating it first)
<myk> then other scripts that want to create attachments can do the same thing

Reproducible: Always

Steps to Reproduce:
Blocks: 302727
*** Bug 302726 has been marked as a duplicate of this bug. ***
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee: administration → timello
Note bug 302669, attachment 193212 [details] [diff] [review], which improves on Attachment.pm considerably.
Assignee: timello → wurblzap
Component: Administration → Attachments & Requests
Depends on: 5179
Whiteboard: [blocker will fix]
Target Milestone: --- → Bugzilla 2.24
Fixed by bug 5179.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.