frioux / DBIx-Class-Helpers

https://metacpan.org/pod/DBIx::Class::Helpers
20 stars 38 forks source link

Booleans with Helper::Row::ToJSON #88

Open 0x62ash opened 6 years ago

0x62ash commented 6 years ago

ToJSON should return JSON::PP::false, JSON::PP:true for boolean data-types.

0x62ash commented 6 years ago

Or maybe i should somehow bind my boolean column to JSON::PP::true/false objects...

frioux commented 6 years ago

At this point making such a change would be backwards incompatible for anyone who has depended on the existing behavior for very long. I'd accept a PR to make it a flag though.

mat813 commented 5 years ago

I was hit by this issue today, wondering how I could fix it :(

frioux commented 5 years ago

Use the documented interface

mat813 commented 5 years ago

Well, yes, I did that for the table I needed right away, but I was thinking more of a cleaner way in a helper :) Don't worry, I'll figure out something.