[Lazarus] RE : Propose: dont try to open TDBLookup* ListSourceautomatically

Reinier Olislagers reinierolislagers at gmail.com
Mon Jul 9 10:27:38 CEST 2012


On 9-7-2012 10:00, Ludo Brands wrote:
> 
>> Currently the db lookup controls tries to open the dataset linked to 
>> ListSource if is closed at lookup initialization.
>>
> 
> You probably guessed it but I'm voting for removing this feature. Opening of
> datasets and establishing database connections behind the back of the
> programmer is always a bad idea.  
> 
> The try open when loaded (streamed), even in the fixed version, has also the
> nasty side effect of not being deterministic. A programmer can't easily
> influence the loading order of components and a change in the order will
> change the behavior of the program.  The fix in 22337 solves the exception
> but not the unpredictable behavior. If the link dataset and connection are
> loaded before the dblookup it will open the dataset, else it won't. So your
> program can start up with (some) dblookups pre-loaded or not. 
> 
> We shouldn't support users counting on unpredictable behavior. 
> 
Totally agreed.






More information about the Lazarus mailing list