utility to combine PaymentTransactionInfoPlugins

Maybe the pull request can be simplified to hide internal changes that aren't interesting after the fact
It can't if PaymentTransactionInfoPlugin is used as expected and perhaps I
should assume that it is?

-Kevin


On Fri, Aug 15, 2014 at 3:06 AM, Pierre-Alexandre Meyer <
notifications@github.com> wrote:

> In lib/killbill/helpers/utils.rb:
>
> > +        t3.properties=[]
> > +
> > +        # Find the transaction count for t1 if it exists and clone the properties from t1 into t3
> > +        t1_trx_count=nil
> > +        unless defined?(t1.properties)==nil || t1.properties.nil?
> > +          t1.properties.each do |property|
> > +            if property.key=='transaction_count'
> > +              t1_trx_count=property.value
> > +           else
> > +             t3.properties << property.clone
> > +            end
> > +          end
> > +        end
> > +
> > +        # If the other transaction info is nil, we intentionally to not update transaction count
> > +        return t3 if t2.nil?
>
> t2 cannot be nil?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/killbill/killbill-plugin-framework-ruby/pull/4/files#r16286526>
> .
>



-- 
Kevin Postlewaite