Hi Sam:

On Thu, 03 Jul 2008, Samuele Kaplun wrote:
> Ouch and there are many of them...

It's not too bad, because many of these cases don't have a chance to
manifest themselves due to the context.

But as I said to Lars he should try a simple fix:

| def record_add_field(rec, tag, ind1="", ind2="",
|                      controlfield_value="",
|                      datafield_subfield_code_value_tuples=None,
|                      desired_field_number=-1):
|
|     if datafield_subfield_code_value_tuples is None:
|         datafield_subfield_code_value_tuples = []

to see if Marko's problem goes away.  I suspect it will.

P.S. But yes, we should deploy such a fix for most/all of these cases,
     or else alter the default values from lists to tuples etc.

Best regards
-- 
Tibor Simko ** CERN Document Server ** <http://cds.cern.ch/>

Reply via email to