fix: Fix copy format (#1091)

* Fix copy formatting problem with empty arrays

* Changelog

Author: zobo

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/) and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## [1.39.1]
+
+- Fix copy formatting on empty array.
+
 ## [1.39.0]
 
 - Xdebug control socket support for pause on Linux and Windows (Xdebug 3.5).

src/varExport.ts

@@ -8,13 +8,20 @@ export async function varExportProperty(property: xdebug.Property, indent: strin
 
     let displayValue: string
     if (property.hasChildren || property.type === 'array' || property.type === 'object') {
-        if (!property.children || property.children.length === 0) {
-            // TODO: also take into account the number of children for pagination
-            property.children = await property.getChildren()
+        let properties: xdebug.Property[]
+        if (property.hasChildren) {
+            if (property.children.length === property.numberOfChildren) {
+                properties = property.children
+            } else {
+                // TODO: also take into account the number of children for pagination
+                properties = await property.getChildren()
+            }
+        } else {
+            properties = []
         }
         displayValue = (
             await Promise.all(
-                property.children.map(async property => {
+                properties.map(async property => {
                     const indent2 = indent + '  '
                     if (property.hasChildren) {
                         return `${indent2}${property.name} => \n${indent2}${await varExportProperty(

src/varJson.ts

@@ -20,13 +20,20 @@ async function _varJsonProperty(property: xdebug.Property, depth: number = 0): P
 
     let displayValue: string
     if (property.hasChildren || property.type === 'array' || property.type === 'object') {
-        if (!property.children || property.children.length === 0) {
-            // TODO: also take into account the number of children for pagination
-            property.children = await property.getChildren()
+        let properties: xdebug.Property[]
+        if (property.hasChildren) {
+            if (property.children.length === property.numberOfChildren) {
+                properties = property.children
+            } else {
+                // TODO: also take into account the number of children for pagination
+                properties = await property.getChildren()
+            }
+        } else {
+            properties = []
         }
 
         const obj = await Promise.all(
-            property.children.map(async property => {
+            properties.map(async property => {
                 return [property.name, <object>await _varJsonProperty(property, depth + 1)]
             })
         )