Re: [Gnumed-devel] demographics editor

[Karsten Hilbert]

> > 1) in every plugin check for active patient and return if None there
> >    - cumbersome,
> >    - prone to forgetting,
> >    - lot's of checking at every tab change for a rare corner case
> Cumbersome ? Prone to forgetting ? Corner case ?
> I call this appropriate handling of an error condigtion. This is a MUST.
> We need to handle this case anyways (just in case something went wrong and
> left us without a valid current patient).
> This is independent of the question of loading patients at startup or not.
This started to dawn on me (hence the Horst reference). So I
guess I will stop whining and start coding the patient checks
where appopriate. As time goes by a more elegant solution will
emerge.

> > 3) take the design decision to *always* load a patient, even
> >    before the user selected one
> >    - sounds cleanest, technically
> I'd call this an ugly hack.
I'd call it an elegant hack at the expense of the user ;-)

> Again, this is different from handling the case in which no patient was
> specified. IMHO this is definitely no solution to the problem stated above.
Words of Consequence. I shall obey :-)

Karsten
-- 
GPG key ID E4071346 @ wwwkeys.pgp.net
E167 67FD A291 2BEA 73BD  4537 78B9 A9F9 E407 1346