View Issue Details

IDProjectCategoryView StatusLast Update
0003168JEDI VCL00 JVCL Componentspublic2005-08-31 01:25
ReporterCRoeperAssigned Toobones 
Status resolvedResolutionfixed 
Product Version3.00 
Target VersionFixed in Version3.10 
Summary0003168: TJvCustomAppStorage (unit JvAppStorage) destructor always does a flush.
DescriptionMaybe I am missing something, but every AppStorage component does save all properties and data when the application is closed.
This is because in the TJvCustomAppStorage.Destroy method the Flush method is called.
Is this by design!?
I would have thought that this is what the AutoFlush property is for, when set to TRUE Flush is called on every change and on close; when set to FALSE the developer has to call Flush by himself.
Additional InformationI would suggest to change the first line of the destructor to the following code:

destructor TJvCustomAppStorage.Destroy;
  if FAutoFlush then Flush;
TagsNo tags attached.



2005-08-28 03:29

administrator   ~0007849

Yes, this is by design, for historical reasons. It has been discussed in the newsgroups a while back and left this way to keep compatibility.
Would adding "property FlushOnDestroy: Boolean" be ok with you?


2005-08-29 18:27

developer   ~0007891

Adding "property FlushOnDestroy: Boolean" would be excellent!


2005-08-30 11:17

reporter   ~0007893

Yes, adding a new property like "FlushOnDestroy" is perfect - if it has the same effect. Thanks!


2005-08-31 01:25

administrator   ~0007898

Ok then.
This is now in CVS, the property FlushOnDestroy has been added, its default value being True.

Issue History

Date Modified Username Field Change
2005-08-27 09:06 CRoeper New Issue
2005-08-28 03:29 obones Note Added: 0007849
2005-08-28 03:29 obones Status new => feedback
2005-08-29 18:27 elahn Note Added: 0007891
2005-08-30 11:17 CRoeper Note Added: 0007893
2005-08-31 01:25 obones Status feedback => resolved
2005-08-31 01:25 obones Resolution open => fixed
2005-08-31 01:25 obones Assigned To => obones
2005-08-31 01:25 obones Note Added: 0007898