LocalJumpError (блок не указан) в Rails

Question

Этот метод работает на моей локальной машине, но не работает на моем удаленном сервере. Он ищет блок, но я не знаю, «где» или «как» его разместить.

def generate_csv
  if params[:print_envelopes]
    @signups = CardBatch.find(params[:id]).card_signups.each.reject { |a| a.envelope? }
    @filename = 'envelopes.csv'
    csv_string = FasterCSV.generate do |csv|
      csv << ["first name, last name, street address, city, state, zip code"]
      @signups.each do |signup|
        csv << [signup.first_name, signup.last_name, signup.street_address_1, signup.city, signup.state, signup.zip_code]
        signup.envelope = true
        signup.save
      end
    end
  elsif params[:print_cards]
    @signups = CardBatch.find(params[:id]).card_signups.reject { |a| a.card? }
    @filename = 'cards.csv'
    csv_string = FasterCSV.generate do |csv|
      csv << ["name", "expiration date", "card number"]
      @signups.each do |signup|
        csv << [signup.full_name, signup.exp_date.strftime('%m/%y'), signup.formatted_card_number] if card?
        signup.card = true
        signup.save
      end
    end
  end

  send_data(csv_string,
    :type => 'text/csv; charset=utf-8; header=present',
    :filename => @filename)
end

Ошибка:

Processing Admin::CardBatchesController#generate_csv (for 173.161.167.41 at 2010-07-21 07:19:47)     [POST]
  Parameters: {"action"=>"generate_csv",     "authenticity_token"=>"OcDII/t8ZleZxRBpISi+Giw+4MAV2Cjjq8bdixJJ+I8=", "id"=>"4", "controller"=>"admin/card_batches", "print_envelopes"=>"print envelopes"}

LocalJumpError (no block given):
  vendor/gems/mislav-will_paginate-2.3.11/lib/will_paginate/finder.rb:170:in `method_missing'
  app/controllers/admin/card_batches_controller.rb:62:in `generate_csv'
  haml (2.2.2) [v] rails/./lib/sass/plugin/rails.rb:19:in `process'
  lib/flash_session_cookie_middleware.rb:14:in `call'
  vendor/gems/hoptoad_notifier-2.2.2/lib/hoptoad_notifier/rack.rb:27:in `call'

** [Hoptoad] Failure: Net::HTTPClientError
** [Hoptoad] Environment Info: [Ruby: 1.8.6] [Rails: 2.3.3] [Env: production]
** [Hoptoad] Response from Hoptoad: 
<?xml version="1.0" encoding="UTF-8"?> 
<errors>
<error>No project exists with the given API key.</error>
</errors>
Rendering /data/HQ_Channel2/releases/20100721141730/public/500.html (500 Internal Server Error)

Обновление

Если will_paginate вызывается где-нибудь в фокусе этого контроллера, он запустит эту функцию.

Finder.rb вокруг строки 170

  def method_missing_with_paginate(method, *args) #:nodoc:
    # did somebody tried to paginate? if not, let them be
    unless method.to_s.index('paginate') == 0
      if block_given?
        return method_missing_without_paginate(method, *args) { |*a| yield(*a) }
      else
        return method_missing_without_paginate(method, *args) 
      end
    end

    # paginate finders are really just find_* with limit and offset
    finder = method.to_s.sub('paginate', 'find')
    finder.sub!('find', 'find_all') if finder.index('find_by_') == 0

    options = args.pop
    raise ArgumentError, 'parameter hash expected' unless options.respond_to? :symbolize_keys
    options = options.dup
    options[:finder] = finder
    args << options

    paginate(*args) { |*a| yield(*a) if block_given? }
  end

И по какой-то причине эта функция и моя функция, написанная выше, не совпадают. Если я смогу это решить, то этот билет можно будет закрыть.

Answer 1

Метод method_missing_without_paginate происходит от alias_method_chain. will_paginate добавляет поведение перед поведением по умолчанию method_missing в Active Record.

Я предполагаю, что ошибка в этой строке (это строка 62)

@signups = CardBatch.find(params[:id]).card_signups.each.reject { |a| a.envelope? }

бит ключа: card_signups.each.reject {

Возможно, у вас 1.8.7 на вашей локальной машине. В 1.8.7 each без блока возвращает объект Enumerator, который вы можете перебирать. Это было перенесено с 1.9 и это здорово.

Но 1.8.6 этого не делает. В 1.8.6 each требуется блок для работы. Вы в порядке, хотя, потому что вам не нужно each в этом выражении.

Заменить эту строку на

@signups = CardBatch.find(params[:id]).card_signups.reject { |a| a.envelope? }

и оно должно работать.

Еще одна вещь, о которой стоит подумать, - насколько большим будет card_signups. Возможно, вы захотите использовать :conditions, чтобы сообщить БД о необходимости выполнить фильтрацию за вас.

Если вы используете логический столбец с именем envelope, он будет выглядеть так:

@signups = CardBatch.find(params[:id]).card_signups.all :conditions => {:envelope=>false}