Неопределенный метод существует?для Draper :: CollectionDecorator - PullRequest
0 голосов
/ 12 октября 2018

Я пытаюсь реализовать гем Draper для моего проекта Rails и не могу заставить его работать.Я получаю сообщение об ошибке:

неопределенный метод "существует?"для #Draper :: CollectionDecorator: 0x000055b625da7a10>

Похоже, ошибка возникает из-за частичного рендеринга в моем представлении индекса.Частичное называется "@ account.exists?"где происходит ошибка.Если я уберу вызов частичного, мой декоратор работает нормально.У меня есть чувство, что мне нужно использовать команду делегата для этого, но я не уверен.Я довольно новичок в Rails и только начинаю изучать некоторые полезные жемчужины.Я надеюсь, что кто-то может взглянуть на мой код и указать мне правильное направление, чтобы обойти эту ошибку.Спасибо!:)


Модель: Account.rb

# An account which will store many transactions and belongs to one user
class Account < ApplicationRecord
  belongs_to :user
  has_many :transactions, dependent: :destroy

  validates :name, presence: true, length: { maximum: 50, minimum: 2 }
  validates :starting_balance, presence: true
  validates :last_four, length: { maximum: 4 }

  # validates_associated :transactions

  after_create :create_initial_transaction

  def create_initial_transaction
    update(current_balance: 0.00)
    # rubocop:disable Metrics/LineLength
    Transaction.create(trx_type: 'credit', trx_date: Time.current, account_id: id, description: 'Starting Balance', amount: starting_balance)
    # rubocop:enable Metrics/LineLength
  end
end

Контроллер: accounts_controller.rb

class AccountsController < ApplicationController
  before_action :authenticate_user!
  before_action :set_account, only: %i[show edit update destroy activate deactivate]

  # GET /accounts
  # GET /accounts.json
  def index
    @accounts = current_user.accounts.where(active: true).order('created_at ASC')
    @accounts = @accounts.decorate
  end

  def inactive
    @accounts = current_user.accounts.where(active: false).order('created_at ASC')
  end

  private

  # Use callbacks to share common setup or constraints between actions.
  def set_account
    @account = current_user.accounts.find(params[:id])
  end

  # Never trust parameters from the scary internet, only allow the white list through.
  def account_params
    params.require(:account).permit(:name, :starting_balance, :current_balance, :last_four, :active, :user_id)
  end
end

Декоратор: account_decorator.rb

class AccountDecorator < Draper::Decorator
  decorates_finders
  decorates_association :user
  delegate_all

  def last_four_formatted
    last_four.present? ? ' ( ... ' + last_four.to_s + ')' : nil
  end
end

Просмотр: accounts / index.html.erb

<%= render partial: "shared/pagetitle", locals: { page_model: "account", description: "Accounts" } %>

<!-- Render the message if no accounts exist -->
<%= render partial: "accounts/noaccounts", locals: { description: "It looks like you don't have any accounts added. To add an account, click the add account button at the top of the page :)" } %>

<div class="row" id="accounts">
  <%= render partial: "accounts/account", collection: @accounts %>
</div>

Просмотр частично: account / _noaccounts.html.erb

<% if !@accounts.exists? then %>
  <div class="row">
    <div class="col s12 m8 offset-m2">
      <div class="card ob-card-gradient ob-account-card">
        <div class="card-content ob-text-primary">
          <span class="card-title">Welcome to mysite!</span>
          <div class="row">
            <br />
            <div class="col s1"><i class="material-icons valign-wrapper">account_balance</i></div>
            <div class="col s11">
            <%= description %></div>
          </div>
        </div>
      </div>
    </div>
  </div>
<% end %>

1 Ответ

0 голосов
/ 12 октября 2018

Я не знаком с драгоценным камнем draper, но вы пытаетесь вызвать exists? для того, что выглядит как набор объектов, а не для объекта ActiveRecord.И поскольку exists? является определенным в ActiveRecord , вы не сможете этого сделать.

Вы, вероятно, можете смело изменить эту строку:

<% if !@accounts.exists? then %>

to:

<% if @accounts.empty? then %>

Метод .empty? определен в нескольких местах , но суть его в том, что если объект, подобный коллекции, пуст, то он вернет true.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...