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