ref: refs/heads/master commit 9098066b3d75a2766953ffdaefe5ce60e5dc9b98 Author: Ladislav Slezak Date: Mon Aug 3 15:37:40 2009 +0200 - use Array#find instead of an ugly loop... --- .../patches/app/controllers/patches_controller.rb | 10 ++-------- 1 files changed, 2 insertions(+), 8 deletions(-) diff --git a/plugins/patches/app/controllers/patches_controller.rb b/plugins/patches/app/controllers/patches_controller.rb index ad11971..c0a3d9a 100644 --- a/plugins/patches/app/controllers/patches_controller.rb +++ b/plugins/patches/app/controllers/patches_controller.rb @@ -96,17 +96,11 @@ class PatchesController < ApplicationController end def get_update (id) - patch_update = nil if @patch_updates.nil? || @patch_updates.empty? @patch_updates = get_updateList end - @patch_updates.each do |p| - if p.resolvable_id.to_s == id.to_s - patch_update = p - break - end - end - return patch_update + + @patch_updates.find { |p| p.resolvable_id.to_s == id.to_s } end def install_update (id) -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org