frioux / DBIx-Class-Helpers

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

Update ToJSON helper use accessors if they exist. #33

Closed kbenson closed 10 years ago

kbenson commented 10 years ago

Since ToJSON uses the the column name as the accessor method for getting the json data, it died trying to access an invalid method if an accessor was defined.

coveralls commented 10 years ago

Coverage Status

Coverage decreased (-0.01%) when pulling 3a7b953bba2bac190a02a6799f07fa59ea9cce05 on kbenson:master into 7cdad51003051eee400fe254a71a0988c814c469 on frioux:master.

kbenson commented 10 years ago

Hmm, apparently a test for this would have been a good idea. :( I'll see what I can do.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.02%) when pulling 4086e9226e7e14d2b02845a87396b406eb903085 on kbenson:master into 7cdad51003051eee400fe254a71a0988c814c469 on frioux:master.

frioux commented 10 years ago

Thanks, released!