frioux / DBIx-Class-Helpers

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

Stop leaking an isa() into the inheritance chain #70

Closed ribasushi closed 8 years ago

ribasushi commented 8 years ago

As per https://metacpan.org/pod/Test::Deep#isa-class-Isa-class isa() is a default export (wtf?!). While it doesn't seem to cause issues, specific 5.8 configurations issue 'redefined' warnings due to this.

Using namespace::clean won't cut it due to how things interact in the stack, but the proposed change passes tests just fine here.

frioux commented 8 years ago

Merged (and fixed by adding bag to the import list.)