DriveStatus returns "INVALID" for existing paths

[supersonic]

Hi,

this is strange:

Since AutoIt V3.3.8.0 the function DriveStatus() always returns "INVALID" for any given EXISTING path - at least on my computer (MS Windows 7 X86). The older version 3.3.6.1 works fine...

Here's a reproducer:

MsgBox(0, "", DriveStatus(@TempDir & ""))

Do I something wrong?

Greets,

-supersonic.

Edited January 3, 2012 by supersonic

[Melba23]

supersonic,

It does say the parameter should be "Path of drive" - and it works when you use the drive letter and colon alone:

ConsoleWrite(DriveStatus("C:Temp") & @CRLF)
ConsoleWrite(DriveStatus("C:") & @CRLF)
ConsoleWrite(DriveStatus("C") & @CRLF)

And before you ask you can get the drive from a path like this:

$sPath = "C:Temp"
$sDrive = StringRegExpReplace($sPath, ":.*$", ":")
ConsoleWrite($sDrive & @CRLF)

M23

[BrewManNH]

It appears that the function has been changed in 3.3.8.0 from the way it worked before.

Regardless, I'd use FileExists to see if a path exists rather than DriveGetStatus as it's probably more reliable to use the function designed for such checks.

[supersonic]

melba,

thank you for the reply.

Hmm... The remarks say: "DriveStatus may even work when a complete directory path [...]".

So, DriveStatus("C:TEMP") should work - shouldn't it?

[supersonic]

BrewManNH,

in fact, we had to assume that the function has been changed.

Is this change documented somewhere?

EDIT:

I will give FileExists() a chance...

Edited January 3, 2012 by supersonic

[BrewManNH]

There's a documented bug fix for DriveStatus for using it with a blank string, which probably affected using it with a path other than just a drive path. The remarks say "DriveStatus may even work " which means it may not work, and as you found out, it doesn't.

DriveStatus is to be used to check the status of a drive, FileExists is used to check the existence of a file/folder/drive path, I'd use that to make sure it exists because folders technically don't have statuses.

[supersonic]

BrewManNH,

FileExists() works fine.

Thanks.

Edited January 3, 2012 by supersonic