Serialization of Spreadsheet objects. Why is it explicitely blocked? #4324
Replies: 3 comments
-
I do not know all of the reasons, but we were getting lots of bug reports (see #1757, #1741, and #932), and possibly others; plus a concern that some of the properties were not serializable (resources, SimpleXMLElement, and maybe others). That said, I believe 1757 is no longer an issue because of the way Dynamic Arrays was implemented, and 1741 can be addressed with any of at least 3 totally trivial changes. I cannot speak to 932; I don't have a good way to test caching. Despite that, It may be time to revisit this decision. I will try to have a PR next week. |
Beta Was this translation helpful? Give feedback.
-
Thanks for the follow-up. If that restriction could be lifted, that would be a really great. Fingers crossed :) |
Beta Was this translation helpful? Give feedback.
-
Release 4.0.0 now allows this. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
There's a commit a7259de that explicitly prevents serialization of the
Spreadsheet
object. I have a code that uses legacy version and successfully serializes instances ofSpreadsheet
for ages, but I'd like to upgrade the package and as serialization is important part of the current flow, I currently stuck.@MarkBaker as you are the author of that commit, could you please cast some light about the reasons?
Beta Was this translation helpful? Give feedback.
All reactions