[9663] causes uploads to be stored in /uploads/0000/00/

[DD32]

[9663] Store uploads according to the parent post date when using year/ month sub-directories

results in this for $uploads when attaching to a draft:

array
  'path' => string 'C:\www\wordpress/wp-content/uploads/0000/00' (length=43)
  'url' => string 'http://localhost/wordpress/wp-content/uploads/0000/00' (length=53)
  'subdir' => string '/0000/00' (length=8)
  'basedir' => string 'C:\www\wordpress/wp-content/uploads' (length=35)
  'baseurl' => string 'http://localhost/wordpress/wp-content/uploads' (length=45)
  'error' => boolean false

It appears that _gmt's are not set for drafts, Not sure if the problem is the drafts, or the uploader for not taking it into consideration.

[ryan]

Drafts purposefully don't have post_date_gmt set. Uploader should take into account.

[azaozz]

(In [9675]) Fix uploads sub-directory for drafts, props DD32, fixes #8205

[azaozz]

It should have been post_date, not post_date_gmt, but having a check there won't hurt.

[DD32]

post_date, not post_date_gmt

In that case, current_time('mysql', true); will need to have its 2nd param removed (else it'll default to the current gmt time)

[azaozz]

Right, local time is used as default in wp_upload_dir() too.

[azaozz]

(In [9676]) Use local time when determining uploads directory, props DD32, see #8205