Ricky883249's questions

我有一个模型Entity，可以有offers和invoices。用户可以创建自定义数字格式来显示发票号码。例如，内部发票号码23可以变成2025INV00000023或类似的。现在，对于offers，我们需要能够使用不同的数字格式。23可能会变成2025OFF00000023。

我通过创建一个类来实现这一点NumberFormat，用户可以通过多种方式配置数字格式。数字格式与实体相关联，因为出于税务/会计目的，实体的所有发票都需要具有统一/一致的编号。

我的Entity模型开始如下：

class Entity < ApplicationRecord
  has_many :invoices, dependent: :restrict_with_error
  has_many :offers, dependent: :restrict_with_error

还有一些不相关的has_many

  belongs_to :invoice_number_format, class_name: 'NumberFormat'
  belongs_to :offer_number_format, class_name: 'NumberFormat'

现在我的NumberFormat类需要有belongs_to关系的另一端。但我不确定加倍是否是正确的方法，以防止dependent: :restrict_with_error用户删除仍在使用的数字格式。VSCode 的自动格式化程序只会在保存时删除第二个。我当然可以保存而不进行格式化（并且在编辑此文件时要非常小心，不要忘记），经过初步测试后，它似乎有效，但我想知道我的方法是否不理想？

  has_many :entities, inverse_of: :invoice_number_format,
                      dependent: :restrict_with_error
  # This line gets deleted on save
  has_many :entities, inverse_of: :offer_number_format,
                      dependent: :restrict_with_error

我有一个控制器“destroy”方法，用于检查各种依赖对象，并会失败并显示相应的警告消息。数据库端也会阻止删除，has_many :invoices, dependent: :restrict_with_error但对于用户来说，删除失败是无声的。

Rubocop 认为“分配分支条件大小太高”，我也想重构它，因为它看起来有点重复。但是，我对返回值感到困惑。当我将检查移到私有函数中时，返回值会从函数中返回，但我还需要它来结束“destroy”方法。我有点需要双重返回。

def destroy
    unless Current.user.default_entity.owner == Current.user ||
           Current.user.default_entity.user_privileged?('manage_contacts')

      redirect_back(fallback_location: contacts_path, alert: t('errors.no_access_rights')) and return
    end

    unless @contact.invoices.count.zero?
      redirect_back(fallback_location: contacts_path,
                    alert: t('errors.cant_delete_contact_with_invoices')) and return
    end

    unless @contact.offers.count.zero?
      redirect_back(fallback_location: contacts_path,
                    alert: t('errors.cant_delete_contact_with_offers')) and return
    end

    @contact.destroy

    redirect_to contacts_path, status: :see_other
  end

在我的应用程序中，其他控制器中也有类似的方法。是否有重构此类方法的模式？

我正在尝试设置 2 个模型（实体、用户），其中每个用户可以与多个实体合作，每个实体可以有多个用户与其合作。我希望关系是唯一的，即用户不能与同一个实体合作两次，而实体也不能有同一个用户两次。

我正在使用 has_many 通过关系，并且在“通过”类中存储了访问权限等额外信息。

class Entity < ApplicationRecord
  has_many :user_entity_roles, dependent: :destroy
  accepts_nested_attributes_for :user_entity_roles
  has_many :users, through: :user_entity_roles, dependent: :restrict_with_error
end

class User < ApplicationRecord
  has_many :user_entity_roles, dependent: :destroy
  has_many :entities, through: :user_entity_roles, dependent: :restrict_with_error
end

class UserEntityRole < ApplicationRecord
  belongs_to :entity, touch: true
  belongs_to :user
end

现在，由于我构建的表单中出现错误，我创建了重复关系。当然，我可以通过不让表单出现错误来尝试防止这种情况，但我想知道是否有办法强制仅使用用户和实体的唯一组合？

我正在尝试为控制器编写测试。“创建”操作需要表单数据，如果表单数据不符合要求，则会被拒绝为“无法处理的实体”：

Failure:
ContactsControllerTest#test_should_post_create [test/controllers/contacts_controller_test.rb:36]:
Expected response to be a <3XX: redirect>, but was a <422: Unprocessable Entity>

我在下面包括了模型、控制器、测试控制器和装置。授权仍然是开箱即用的，但在这里不应该成为因素（其他测试运行良好）。

我怀疑参数中的外键是罪魁祸首，但我不知道如何检查。我还在测试参数中尝试了@saluation.id 和@debitor.id。

模型

class Contact < ApplicationRecord
    validates :last_name, presence: true
    validates :email, format: {
        with: URI::MailTo::EMAIL_REGEXP
    }
    belongs_to :salutation
    belongs_to :debitor

控制器

class ContactsController < ApplicationController

  def create
      @contact = Contact.new(contact_params)
      @debitors = Debitor.all
      @salutations = Salutation.all
    
      if @contact.save 
        redirect_to contacts_path
      else
        render :new, status: :unprocessable_entity
      end
  end

  private
  def contact_params
    params.require(:contact).permit(:last_name, :first_name, :email, :salutation_id, :debitor_id)
  end

end

测试

require "test_helper"

class ContactsControllerTest < ActionDispatch::IntegrationTest
  setup do
    @user = users(:lazaro_nixon)
    @contact = contacts(:one)
    @salutation = salutations(:one)
    @debitor = debitors(:one)
  end


  test "should post create" do
    sign_in_as @user

    post contacts_url, :params => { :contact => { last_name: "Lastname", first_name: "Firstname", email: "[email protected]", salutation: @salutation, debitor: @debitor }}
    assert_response :redirect
    follow_redirect!
    assert_response :success
  end

end

装置 - 接触器

one:
  last_name: MyString
  first_name: MyString
  email: MyString
  salutation: one
  debitor: one

固定装置 - 致意

one:
  salutation: MyString

固定装置 - 债务人

one:
  company: MyString
  street: MyString
  house_number: MyString
  zip: 1
  city: MyString
  payment_terms: one