Closed
Bug 27857
Opened 25 years ago
Closed 24 years ago
nsIFile - Mac not completely implemented
Categories
(Core :: XPCOM, defect, P2)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
mozilla0.9
People
(Reporter: dougt, Assigned: ccarlen)
References
Details
(Whiteboard: [nsbeta2-][nsbeta3-])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
The follow functions were never implemented!
CopyTo
CopyToFollowingLinks
MoveTo
Spawn
GetFileSizeOfLink
IsHidden
IsSymlink
Normalize
GetPermissions
GetPermissionsOfLink
SetPermissions
SetPermissionsOfLink
IsSpecial
Reporter | ||
Comment 1•25 years ago
|
||
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]
Reporter | ||
Comment 3•25 years ago
|
||
assigning to scc per dp. Scott lets talk before you start.
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.
Comment 5•25 years ago
|
||
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.
Reporter | ||
Comment 7•25 years ago
|
||
davidm, please attach diffs for these changes. If you have questions about what
the interface is suppose to do, lets talk.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 8•25 years ago
|
||
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
Reporter | ||
Comment 10•25 years ago
|
||
*** Bug 33116 has been marked as a duplicate of this bug. ***
Comment 11•25 years ago
|
||
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
Comment 12•25 years ago
|
||
Unless I hear otherwise, we will take this up for M16. Davidm has the functions
required for disk cache in.
Target Milestone: M15 → M16
Comment 13•25 years ago
|
||
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
Comment 15•25 years ago
|
||
Upating [nsbeta-] to read [nsbeta2-] in Status Whiteboard.
Whiteboard: [nsbeta-] → [nsbeta2-]
Comment 16•25 years ago
|
||
mass re-assigning to my new bugzilla account
Assignee: scc → scc
Status: ASSIGNED → NEW
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 17•24 years ago
|
||
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
Assignee | ||
Comment 18•24 years ago
|
||
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?
Assignee | ||
Comment 19•24 years ago
|
||
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
Comment 20•24 years ago
|
||
upping
Comment 21•24 years ago
|
||
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-]
Assignee | ||
Updated•24 years ago
|
Target Milestone: --- → mozilla0.9
Updated•24 years ago
|
QA Contact: leger → kandrot
Assignee | ||
Comment 22•24 years ago
|
||
Assignee | ||
Comment 23•24 years ago
|
||
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?
Comment 24•24 years ago
|
||
Other than asking if the dest is created if it doesn't exist, r=sdagley
Comment 25•24 years ago
|
||
sr=sfraser
Summary: nsIFile - MAC not completely implemented → nsIFile - Mac not completely implemented
Assignee | ||
Comment 26•24 years ago
|
||
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.
Description
•