Error in xml builder - array with nil values create empty tags

[kristianmandrup]

Gyoko.xml "attribute" => [nil, nil, {}]

IMO should output only one empty <attribute/> element. In my case it outputted 3.
I think all nils should be ignored (f.ex by using compact internally). Just my take.
Cheers!

[tjarratt]

Interesting issue -- was this issue driven by some case where you passed the results of some method to gyoku, @kristianmandrup?

eg:

def slot_machine
  return rand > 0.5 ? nil : "JACKPOT"
end

def craftSomeXML
  Gyoku.xml "attribute" => [slot_machine, slot_machine, {}]
end

Mostly, I'm just curious how this issue came up. It's not immediately obvious to me how handling nil in this manner is an improvement to end users. FWIW, this behavior is actually documented in the README:

NilClass objects are converted to xsi:nil tags

[tjarratt]

Closing as Won't Fix for now. I'm open to changing this behavior in the next version of Gyoku, however, so if you're reading this and would like this behavior, please speak up.