[dev] Re: [cvs] commit: horde/lib/Prefs sql.php

Jan Schneider jan@horde.org
Wed, 15 May 2002 10:59:42 +0200


Zitat von Jon Parise <jon@horde.org>:

> jon         2002/05/14 11:30:06 PDT
> 
>   Modified files:
>     lib/Prefs            sql.php 
>   Log:
>   Instead of testing for a row's existing by attempting an insertion,
>   explicitly perform a select query to test for a row's existence.  Based
> on
>   the result of that query, we know whether to perform an update or an
>   insertion.

> http://cvs.horde.org/diff.php/horde/lib/Prefs/sql.php?r1=1.35&r2=1.36&ty=u

Any reason why you also check for the existance of a shared preference? I 
guess it might happen that a user has a shared preference but this 
preference different for a certain application.

Perhaps an administrator want's to allow his users to change their shared 
language preference in the Horde options screen but let them additionally 
change their prefered language for each application.

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft