From a5a16bde240b757f87f1e712953c4f67cba57d2f Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Thu, 8 Mar 2007 19:34:17 +0000
Subject: [PATCH] - Patch #121876 by Darren, Nedjo et al: drupal_to_js converts
 empty arrays to objects.

---
 includes/common.inc | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/includes/common.inc b/includes/common.inc
index caf4560b92c..7cd66792d68 100644
--- a/includes/common.inc
+++ b/includes/common.inc
@@ -1699,14 +1699,17 @@ function drupal_to_js($var) {
                               array('\r', '\n', '\x3c', '\x3e', '\x26'),
                               addslashes($var)) .'"';
     case 'array':
-      if (array_keys($var) === range(0, sizeof($var) - 1)) {
+      // Arrays in JSON can't be associative. If the array is empty or if it
+      // has sequential whole number keys starting with 0, it's not associative
+      // so we can go ahead and convert it as an array.
+      if (empty ($var) || array_keys($var) === range(0, sizeof($var) - 1)) {
         $output = array();
         foreach ($var as $v) {
           $output[] = drupal_to_js($v);
         }
         return '[ '. implode(', ', $output) .' ]';
       }
-      // Fall through
+      // Otherwise, fall through to convert the array as an object.
     case 'object':
       $output = array();
       foreach ($var as $k => $v) {