[katello-devel] 'rescue Exception => e' removed

Ivan Nečas inecas at redhat.com
Mon Aug 27 16:33:40 UTC 2012


On 08/27/2012 05:15 PM, Mike McCune wrote:
> On 08/27/2012 03:46 AM, Petr Chalupa wrote:
>> Hi,
>>
>> I've just merged pull-request [1] removing "rescue Exception =>  e" from
>> katello source code.
>>
>> Please do not use "rescue Exception =>  e" any more, use standard 
>> "rescue
>> =>  e" (which is equivalent with "rescue StandardError => e") or better
>> specify the exception class "rescue ActiveRecord::RecordNotSaved =>  e".
>>
>> See the pull request for why.
>>
>> Thanks, Petr
>>
>> [1] https://github.com/Katello/katello/pull/176
>>
>
> excellent work
+1
>
> _______________________________________________
> katello-devel mailing list
> katello-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/katello-devel


-- 
Ivan




More information about the katello-devel mailing list