Remove no longer needed try{} block in related rel instantiation
authorPeter Rabbitson <ribasushi@cpan.org>
Wed, 24 Jun 2015 11:04:28 +0000 (13:04 +0200)
committerPeter Rabbitson <ribasushi@cpan.org>
Wed, 24 Jun 2015 12:19:42 +0000 (14:19 +0200)
commit00ba5e6ecf1a6a550b31c913c674cd0cf8f2f879
treea954bf1c05f75b3d9ef3f159bdc206811d8de3bd
parent35ca725ad0d3a592102b69ad602d9471ee24e5a2
Remove no longer needed try{} block in related rel instantiation

This try{} is no longer necessary after 3b4c4d72, but was missed due its
function not being readily apparent. The only known case where this
exception could be raised and be silenced is if an uninserted object had
some filter relations populated directly via {_inflated_column} *before*
related_resultset() has ever been called. This would result in a cascade
of failures during the relationship resolution, and bubble up here.

Starting with 3b4c4d72 the empty related resultset is instantiated ahead
of time, and everything works properly from there on.

In the unlikely case this does cause issues downstream, a way will be
found to address this ( worst case scenario by reintroducing the try{} )
lib/DBIx/Class/Relationship/Base.pm