Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enforce array<key, value> to be array<key, value>|\stdClass to account for empty arrays #172

Closed
provokateurin opened this issue Oct 7, 2024 · 1 comment
Closed

Enforce array<key, value> to be array<key, value>|\stdClass to account for empty arrays #172

provokateurin opened this issue Oct 7, 2024 · 1 comment
Assignees
@provokateurin
Labels
enhancement New feature or request

Comments

@provokateurin
Copy link
Member

Related to #33.

Many apps don't consider this case but it needs to be handled as otherwise it will end up as [] in the JSON output while it should be {} instead.
@provokateurin provokateurin added the enhancement New feature or request label Oct 7, 2024
@provokateurin provokateurin mentioned this issue Oct 28, 2024
Merged
@provokateurin provokateurin self-assigned this Oct 30, 2024
@provokateurin provokateurin moved this to 📄 To do (~10 entries) in 📁 Files team Oct 30, 2024
@provokateurin
Copy link
Member Author

An exception to this is the use of non-empty-array<key, value>, since we know this problem can't appear there.

@github-project-automation github-project-automation bot moved this from 📄 To do (~10 entries) to ☑️ Done in 📁 Files team Dec 17, 2024
@provokateurin provokateurin mentioned this issue Dec 17, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@provokateurin provokateurin

Labels
enhancement New feature or request
Projects
📁 Files team
Status: ☑️ Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@provokateurin