View Single Post
Old 08-31-2014, 02:22 PM   #8
chaley
Grand Sorcerer
chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.
 
Posts: 12,471
Karma: 8025600
Join Date: Jan 2010
Location: Notts, England
Device: Kobo Libra 2
Quote:
Originally Posted by Terisa de morgan View Post
Chaley, I understand what you say, I'm quite aware of this way of working. What I get of your message is that it's right to empty the destination field, although it doesn't work the same for integer or date fields (I always order the book so the destination book is the first in that order).

Perhaps @adoby is right, and that is the problem. I don't know.
No, he isn't. Enumerated types have a null (undefined) value.

I just looked more carefully at the merge code. This is a bug. The "if statement" that handles enumerations is not parenthesized correctly, resulting in the source *always* replacing the destination even if the src is empty. I will submit a fix.
chaley is offline   Reply With Quote