179901 - Defaults to "Desktop Folder" not ~/Desktop/ on first download

Status: VERIFIED FIXED

(Camino Graveyard :: Downloading, defect)

Description

[Norbert Rittel]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:1.0.1) Gecko/20021112 Chimera/0.6+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:1.0.1) Gecko/20021112 Chimera/0.6+

When you create a new user the download location in Internet Preferences seems
to be undefined.
When you do the first download Chimera creates a folder named "Desktop Folder"
in the user's Home folder. Should use "Desktop" instead.

Reproducible: Always

Steps to Reproduce:
1. Create a new user in Mac OS X.
2. Log in under that user.
3. Open Chimera.
4. Download some file, latest Chimera nightly build, e.g.

Actual Results:  
You'll see a folder named "Desktop Folder" created in the user's home folder
which contains the file being downloaded.


Expected Results:  
It should have used ~/Desktop instead.

Does not seem to happen at all if you try this with an existing user where the
download location in Internet Preferences is already defined.
So I never saw this before.

Should be easy to locate the "Desktop Folder" string in the source code.
If Chimera is not the cause this should be an Apple bug. - In that case let me
know and I'll submit it at bugreport.apple.com.

Comment 1

[Steve Dagley]

What version of Mac OS X are you running?  We know the download folder isn't
initialized properly for new users but we'd hoped Apple would fix that in 10.2.2.

As for where the Desktop Folder is being created, we're calling FSFindFolder()
to get kSystemDesktopFolderType which is supposed to be the desktop folder at
the root of the hard drive and never re-directed.  It could behave differently
under OS X though as we're asking for it in kUserDomain. 

Comment 2

[Norbert Rittel]

I'm running 10.2.2. Is the bug with the uninitialized download folder a know bug
(entered in Radar) at Apple?

For getting the current user's Desktop folder you should be calling
kDesktopFolderType not kSystemDesktopFolderType - if I read the Developer
documentation correctly.

Comment 3

[Steve Dagley]

I thought I'd logged that one but I don't see it as one of my open reports on
RadarWeb so I'll re-file it.

And I think you're right on kSystemDesktopFolderType vs. kDesktopFolderType.  I
assumed the comment in the header for kSystemDesktopFolderType of /* the desktop
folder at the root of the hard drive, never the redirected user desktop folder
*/ meant that it was the constant for the desktop itself.  I'll verify that
kDesktopFolder type does what I intended and fix it.

Comment 4

[Steve Dagley]

fixed

Comment 5

[Chris Petersen]

Verified in the 2002-12-03-05 NB under 10.2.2. Tested with a new user account
and file is now saved to ~/Desktop.