Closed Bug 27857 Opened 25 years ago Closed 24 years ago

nsIFile - Mac not completely implemented

Categories

(Core :: XPCOM, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla0.9

People

(Reporter: dougt, Assigned: ccarlen)

References

Details

(Whiteboard: [nsbeta2-][nsbeta3-])

Attachments

(1 file)

The follow functions were never implemented! CopyTo CopyToFollowingLinks MoveTo Spawn GetFileSizeOfLink IsHidden IsSymlink Normalize GetPermissions GetPermissionsOfLink SetPermissions SetPermissionsOfLink IsSpecial
These are also not implemented! GetLastModificationDate SetLastModificationDate GetLastModificationDateOfLink SetLastModificationDateOfLink IsWritable IsReadable IsExecutable
Keywords: beta1
Hi! PDT needs to know exactly how this blocks beta. Thanks!
Whiteboard: [NEED INFO]
assigning to scc per dp. Scott lets talk before you start.
Assignee: dougt → scc
Keywords: beta1
Whiteboard: [NEED INFO]
Target Milestone: M15
Some of these functions block getting the file:// directory listing working on the mac which is a PDT+ bug. Delete is also not functional which block the disk cache code from working properly.
Can you list which ones. Unless we get a prioritized list, we wont be able to shift effort into this for beta. That is what dougt, scc and myself decided after discussing this.
Disk Cache ---------- CopyTo or MoveTo I can't remember which Delete File Listing ------------ IsHidden ( I have an implementation ) GetLastModificationDate would be nice so all files don't say they were create in 1940 There are also bugs in Exists and ResolveAndStat which I have "fixes" for. I don't think the above will take more than 1 day total to do. The only tough part is figuring out what exactly some of the interfaces are supposed to do. The actual code is just copy and paste with morefiles.
davidm, please attach diffs for these changes. If you have questions about what the interface is suppose to do, lets talk.
Status: NEW → ASSIGNED
davidm said he would be working on this. Reassigning after talking to him... Scott can help with this for M15. So if there are more than functions required for beta that need implementation for M15, you can reassign this back to me.
Assignee: scc → davidm
Status: ASSIGNED → NEW
Blocks: 21100
Marking beta2.
Keywords: beta2
*** Bug 33116 has been marked as a duplicate of this bug. ***
implimented the functions that I am going to. Reassign back to scott if we need to do *ToFollowingLInks and the various permissions stuff
Assignee: davidm → scc
Unless I hear otherwise, we will take this up for M16. Davidm has the functions required for disk cache in.
Target Milestone: M15 → M16
It sounds more like we don't know that we need to implement anything else. I'd like to push this off until we hit somthing that says "Oops. We need method X now."
Status: NEW → ASSIGNED
Target Milestone: M16 → M20
Keywords: nsbeta2
Putting on [nsbeta2-] radar.
Whiteboard: [nsbeta-]
Upating [nsbeta-] to read [nsbeta2-] in Status Whiteboard.
Whiteboard: [nsbeta-] → [nsbeta2-]
mass re-assigning to my new bugzilla account
Assignee: scc → scc
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
The suggestion from macdev was to re-assign this to you, conrad, since you're working on related stuff. You can decide if this bug is still valid.
Assignee: scc → ccarlen
Status: ASSIGNED → NEW
These lists were a bit out of date. As of today, here's what's not implemented: CopyToFollowingLinks Normalize GetPermissions GetPermissionsOfLink SetPermissions SetPermissionsOfLink IsSpecial GetLastModificationDateOfLink SetLastModificationDateOfLink Are there any of these we can't live without?
I recently found that CopyToFollowingLinks was needed. When copying files from defaults to a profile folder, sometimes the file being copied is an alias. Without CopyToFollowingLinks, the alias, not the actual file is copied. If the copied file is then changed, the original is changed, not a unique copy in the profile folder. This is a problem only in development builds, where files in defaults may be aliases, but does deserve to be fixed.
Status: NEW → ASSIGNED
upping
Keywords: nsbeta3
Priority: P3 → P2
Whiteboard: [nsbeta2-] → [nsbeta2-][nsbeta3+]
Not holding PR3 for this; marking nsbeta3-. If some part of this work really really needs to be done for Seamonkey RTM, please explain in detail when nominating for rtm.
Whiteboard: [nsbeta2-][nsbeta3+] → [nsbeta2-][nsbeta3-]
Target Milestone: --- → mozilla0.9
QA Contact: leger → kandrot
Attached patch adds CopyToFollowingLinks (deleted) — Splinter Review
The patch implements CopyToFollowingLinks. Of the list of things not implemented at this point (which is much smaller than when this bug was filed) this one we could actually use. Steve, Simon - can you r/sr?
Other than asking if the dest is created if it doesn't exist, r=sdagley
sr=sfraser
Summary: nsIFile - MAC not completely implemented → nsIFile - Mac not completely implemented
Blocks: 27416
No longer blocks: 21100
This was checked in 02/26/01 but not marked. With CopyToFollowingLinks, it's complete except for a few things which only make sense for Unix and some XXXOfLink methods which are going to be changing anyway based on an API review.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: