Skip to content

Commit

Permalink
Don't make client addressbook source a required configuration parameter.
Browse files Browse the repository at this point in the history
Chicken and egg problem: when turba is first installed, and using a source like
localsql with shares enabled, there will be no existing address books found to list.
If left unpopulated, this will cause client/* api calls to fail, but before we made
this a select list, we made it default to 'localsql' - which would cause the clients/*
api methods to fail since that's not a valid source when using shares.

 So, making this non-required doesn't loose anything we had before.
  • Loading branch information
mrubinsk committed Oct 20, 2010
1 parent f50c729 commit 6b392c4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/conf.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

<configsection name="client">
<configheader>Clients</configheader>
<configenum name="addressbook" desc="Name of client addressbook">
<configenum name="addressbook" desc="Name of client addressbook" required="false">
<values>
<configspecial name="list-contact-sources"/>
</values>
Expand Down

0 comments on commit 6b392c4

Please sign in to comment.