Nitpick: whitespace between do and |.  Also can probably use a map here as well -- @entries = ....map do |entry|.