From d9ece0d575151e0ef2b8994a19f672f9104afc3a Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Thu, 8 Mar 2007 19:33:55 +0000
Subject: - 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(-)

(limited to 'includes/common.inc')

diff --git a/includes/common.inc b/includes/common.inc
index 6c442e5b3..63973c499 100644
--- a/includes/common.inc
+++ b/includes/common.inc
@@ -1762,14 +1762,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) {
-- 
cgit v1.2.3