Closed (outdated)
Project:
CSV Serialization
Version:
8.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Mar 2017 at 11:32 UTC
Updated:
17 Nov 2023 at 00:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
volkerk commentedComment #3
grasmash commentedA few alternatives here: https://stackoverflow.com/questions/262891/is-there-a-way-to-find-out-ho...
I like...
Comment #4
grasmash commentedComment #5
miritas commentedHello.
@Vokerk
Seems there is an inaccuracy in the code. I believe that such changes are required:
+ $copy[$key]['__test'] == true;+ $copy[$key]['__test'] = true;@grasmash
Your example is not resistant to infinity recursion that can appear because of references.
Regards.
Comment #6
michelleThis patch doesn't fix the core issue, so I'm going to leave the status as needs work. What it does is add in a check to ensure the process doesn't crash when it gets the array depth wrong. I think this is a useful addition even once the real patch for this issue is in as it's a good failsafe.
Comment #7
steinmb commentedBumping to major as i totally breaks and prevent us from doing any useful with it when we hit this.
Comment #8
michelleRemoving my patch from the list. I missed doing it when I uploaded it. Since my patch is just a helpful failsafe while we wait on a real fix, I don't want it showing up in the top post and confusing people.
Comment #9
singularoAlternate approach - trim the leading spaces off - Feels hacky though.
Comment #10
markdorisonIs this issue still relevant? If so, please reopen with a merge request against the 4.x branch.