72913 - [RFE]need File.isWritable()

Status: VERIFIED FIXED

(Core Graveyard :: Installer: XPInstall Engine, enhancement)

Description

[dbragg]

XPInstall needs to expose a File.isWritable() so script authors doing direct i/o
type stuff can keep themselves out of trouble.

Comment 1

[Syd Logan]

Fix a bug, get a clue.

Comment 2

[Syd Logan]

Attachment: Patch to fix this

Comment 3

[Syd Logan]

Attachment: Patch to fix this

Comment 4

[Syd Logan]

Attachment: Better patch, ignore the previous one

Comment 5

[Syd Logan]

Attachment: install.js for testing this

Comment 6

[Syd Logan]

Dan, samir -- looking for r= and sr= on this.

Comment 7

[Samir Gehani]

Drop the fileIsWritable name (the file*() names are deprecated in favor of
File.*()).  Else looks good.  Contingent upon dropping this name r=sgehani.

Comment 8

[Syd Logan]

samir, if you look at the test JS I wrote, I am using isWritable. I just copied
the existing patter, internally looks like fileIsWritable, fileIsDirectory and
such are used. We should make any deprecation you refer to in one fell swoop,
not just for my code, so as to maintain consistency. When are you planning to do
this change, if ever?

Comment 9

[Daniel Veditz [:dveditz]]

sr=dveditz with comment change discussed via IM. I don't care about the
redundant fileIsWritable, we can nuke that later.

Comment 10

[Syd Logan]

Fixed

Comment 11

[Jimmy Lee]

Build: 2001-05-17-07-trunk(WIN), 2001-05-17-08-trunk(MAC), 
2001-05-17-08-trunk(LINUX)

Looks good on all platforms.  http://jimbob/jars/a_fileop_iswritable.xpi

Marking Verified!

Comment 12

[Brant Gurganus]

[RFE] is deprecated in favor of severity: enhancement.  They have the same meaning.

Comment 13

[Manuel Reimer]

For me this one seems to fail with Mozilla 1.7.11!
An "isWritable" on an existing file always returns "true" and an "isWritable" on
an non existing file always returns "false".

Could someone please reopen this one?