Deserialize PHP associative arrays as new Map()#295
Deserialize PHP associative arrays as new Map()#295miso-belica wants to merge 1 commit intosteelbrain:mainfrom
new Map()#295Conversation
This way the order of the original PHP array is always preserved.
|
Hello @steelbrain I know it's not easy to find a time for PRs sometimes but this one is not so big so if you find some it would be great 🙂 |
|
Hi @miso-belica! Sorry it took so long. I'll review this as soon as I can and get back to you. Appreciate the contribution |
|
@miso-belica There's some changes we're going to have to make to this before we can ship it, lmk if you want to make them on your own. I'd also be down to making them on my end. Anyway, here's the review requests:
|
|
@steelbrain thank you for the article and other hints. Recently I was blessed and was able to remove WP from the project so I hope I will remove this library soon :) But, thank you for it. It helped us to maintain the project for years. Feel free to make any modification you find useful to merge the code. I hope it helps prevents some bugs to people stuck with WP in their project. |
This way the order of the original PHP array is always preserved. It took me a while, sorry 😅
closes #293