Remove empty arrays when remove_empty_properties is set

[rmonnerat]

The condition should check if the construct is an Array also, you can easily reproduce in plain javascript:

obj = Array()
obj === Object(obj) // true
Object.keys(obj).length === 0 // true
obj.constructor === Array // true
obj.constructor === Object // false

Otherwise, empty arrays ends up on the getValue result.

Q	A
Is bugfix?	✔️
New feature?	❌
Is backward-compatible?	✔️
Tests pass?	✔️
Fixed issues	Unknown
Updated README/docs?	❌
Added CHANGELOG entry?	❌

[navytux]

@rmonnerat, do you have an idea why tests fail? (see e.g. https://github.com/json-editor/json-editor/actions/runs/6868513133/job/18698471271?pr=1423)

[schmunk42]

@rmonnerat, do you have an idea why tests fail? (see e.g. https://github.com/json-editor/json-editor/actions/runs/6868513133/job/18698471271?pr=1423)

I triggered the failed builds again, let's see if this was just a hickup.

[rmonnerat]

@navytux I don't know, I ran the npm run eslint and npm test and it looked fine. However, I am not familiar with the development in JS using npm very much.

[germanbisurgi]

@schmunk42 related: #1427

[germanbisurgi]

Wait for #1427 to be merged and test again, please

[rmonnerat]

@germanbisurgi I updated the branch, waiting for tests re-rerun.

[germanbisurgi]

The tests show positive results, yet it would be beneficial to create an issue outlining the details along with a description and steps to reproduce the situation. This is to ensure we prevent any potential regression.