r18376 - gnucash/trunk/src/backend/xml - Partly fix broken data file backward compatibility where SX recurrence would disappear.

[Derek Atkins]

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 <cstim at code.gnucash.org> 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
       warlord at MIT.EDU                        PGP key available