Christian,

How about also putting it into the KVP?  This way it would
survive a 2.2.x read/save.  Obviously if it's none you can
just leave it blank, but if it's NOT "none" then store it
in the KVP..  on READ look both for the XML and the KVP
(where the XML tag takes precedence over the KVP, but the KVP
is a backup just in case you're loading in from 2.2.x)

Then in 2.5/2.6/3.0 we can turn off the KVP and turn on the
XML tag.

I believe Phil did this with some other tags recently; I think
it's the better way to approach the problem.

-derek

Christian Stimming <[email protected]> writes:

> Modified:
>    gnucash/trunk/src/backend/xml/gnc-recurrence-xml-v2.c
> Log:
> Partly fix broken data file backward compatibility where SX recurrence would 
> disappear.
>
> In r17725 and r17751, I introduced the extra XML child element
> <recurrence:weekend_adj>, but this means a gnucash-2.2.x cannot read
> the SX recurrence of a >=2.3.x file anymore, which is bad. In order
> to improve this broken backward compatibility for at least the common
> case, we don't write out this XML element as long as it is only "none".

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       [email protected]                        PGP key available
_______________________________________________
gnucash-devel mailing list
[email protected]
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

Reply via email to