Replace improper use of drupal_to_js() with drupal_json()

Confirmed that using echo drupal_to_js($var) is not the intended use. This bug is still present in 7.x and is also should be fixed in system_date_time_lookup.

Setting this to 7.x and will need to backport to 6.x

Looks like upload.module might need to be updated too?

No, it looks like there's a later comment explanation in upload_js, but the explanation is still kinda weird...

  // We send the updated file attachments form.
  // Don't call drupal_json(). ahah.js uses an iframe and
  // the header output by drupal_json() causes problems in some browsers.

Failed due to #74645: modify file_scan_directory to include a regex for the nomask.. Setting back to code needs review.

patch still applies and everything seems to work. ran through the install and the timezone was selected correctly (i'm assuming that's what that bit of js does). created a profile text field and tested that the auto complete suggested properly. i'm assuming that's all there is to test.

Committed to CVS HEAD. Thanks.

Patch for 6.x ready for review.

There are also two occurrences in upload.module.

Oops, sorry about that. In that case I guess it's RTBC.

Thanks, committed to Drupal 6.