search

ironfede / openmcdf

Microsoft Compound File .net component - pure C# - netstandard 2.0
Mozilla Public License 2.0
308 stars 76 forks source link

When saving an OLEProperties container of type 'AppSpecific' which ha… #137

Closed Numpsy closed 2 weeks ago

Numpsy commented 3 months ago

…s custom property names present, write a dictinoary property to the output stream

refs https://github.com/ironfede/openmcdf/issues/134

As it stands now, if you read a property set other than SummaryInformation/DocumentSummaryInformation, and then save that back to the file, and that set contains a DictionaryProperty, then said dictionary will be lost.

So - try always writing it for AppSpecific sets (we don't want to write such a property for Summry/DocSummary).

Numpsy commented 3 weeks ago

@ironfede @farfilli Any thoughts or comments on how useful this is or if it should be completed now? (I can see about adding a unit test if it's useful)

farfilli commented 3 weeks ago

It's sure useful!

Numpsy commented 2 weeks ago

Test added

ironfede commented 2 weeks ago

Many Thanks @Numpsy ! Merging now.