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, 12 Aug 2015 12:30:39 +0000 (14:30 +0200)
commitb47fb9c0c3fee01f3641da23d34b8428ab1e9a70
treef76a9bfabfd599efb77ad71bc1c849d6df22e990
parente187eecba5ad6d6d27b18e471ea8c13d9a4368b3
Remove no longer needed try{} block in related rel instantiation

This try{} is no longer necessary after 3b4c4d72, but was missed due to
its purpose 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