0

验证：check_product_stock

def check_product_stock

  @thisproduct = product.id

   @productbeingchecked = Product.find_by_id(@thisproduct)

   @stocknumber = @productbeingchecked.stock_number

  if producto.en_stock == 0
   raise "El Producto no tiene stock suficiente para completar la venta"

   #errors.add :venta, "Producto con pedos de stock"
   return false
  end
true
end

end

我需要能够验证模型（销售）的创建，如果它是关联（产品），在名为 stock_number 的 product.column 中没有达到零。

我想我需要重写整个事情，但现在使用 validate :check_product_stock 然后从头开始构建一个方法来检查产品是否没有达到零并且它应该发出一个快速通知并留在同一个地方（销售/新)

class Venta < ActiveRecord::Base

  hobo_model # Don't put anything above this

  belongs_to :cliente, :accessible => true
  belongs_to :producto, :accessible => true
  belongs_to :marca, :accessible => true
  belongs_to :vendedor
  belongs_to :instalador
  has_many :devolucions



  fields do
    numero_de_serie     :string
    precio_de_venta      :integer
    precio_de_instalacion :integer, :default => "0"
    forma_de_pago enum_string(:contado, :tarjeta)
    status enum_string(:activa, :cancelada)
    timestamps
  end

  validates_presence_of :cliente, :precio_de_venta, :vendedor, :precio_de_instalacion

  validate_on_create :check_product_stock

  after_save :descontar_precio_de_instalacion_si_el_instalador_es_a_destajo

#def stock_error

  #flash[:notice] = "Producto con pedos de stock"

#  redirect_to :controller => :venta, :action => :stock_error

     #errors.add_to_base("Producto con pedos de stock") 

 # end

def check_product_stock

if producto.en_stock == 0
 raise "El Producto no tiene stock suficiente para completar la venta"

 #errors.add :venta, "Producto con pedos de stock"
   return false
  end
true
end

#def check_product_stock
#  if producto.en_stock == 0
 #  errors.add :producto, "El Producto no tiene stock suficiente para completar la venta"
 #  return false
#  end
#  true # guards against returning nil which is interpreted as false.
#end




def descontar_precio_de_instalacion_si_el_instalador_es_a_destajo





  @este_instalador_id = instalador.id

  @instalador = Instalador.find_by_id(@este_instalador_id)


  if @instalador.a_destajo?

    @last_venta = Venta.find(:last)

    @vid = @last_venta.id

    @precio_de_instalacion_original = precio_de_instalacion

    @mitad_de_instalacion = @precio_de_instalacion_original/2

    #Venta.update(@vid, :precio_de_instalacion => @mitad_de_instalacion)

    ActiveRecord::Base.connection.execute "UPDATE ventas SET precio_de_instalacion = #{@mitad_de_instalacion} WHERE id = #{@vid};"



  end


end








#after_save :reduce_product_stock_number

# def reduce_product_stock_number
 #  Producto.decrement_counter(:en_stock, producto.id)
#  end




  # --- Permissions --- #

  def create_permitted?
    true
  end

  def update_permitted?
    false
  end

  def destroy_permitted?
    false
  end

  def view_permitted?(field)
    true
  end

end

这是我的观察者，它减少了 producto 中的 en_stock 列：

class VentaObserver < ActiveRecord::Observer

def after_save(venta)

      @venta_as_array = venta




      if venta.producto_id?

      @pid = @venta_as_array[:producto_id]

      Producto.decrement_counter(:en_stock, @pid)

      end

      if venta.cart_id

        @cid = @venta_as_array[:cart_id] 

        @cart = Cart.find_by_id(@cid)  

        for item in @cart.cart_items do
         #  @pid = @cart.cart_items.producto.id



             Producto.decrement_counter(:en_stock, item.producto.id)

        end

        #si el instalador es a destajo se debe descontar la mitad del rpecio de instalacion






        end

end

end
4

2 回答 2

4

看来你这里出了很多问题。

首先，你做的工作太多了。这就是你真正需要的。

before_save :check_product_stock

def check_product_stock
  if product.stocknumber == 0
   flash[:notice] = "Producto con pedos de stock"
  end
end

其次，闪存哈希在模型中不可用。您可以使用 ActiveRecord 错误对象通过替换为控制器和视图提供错误

flash[:notice] =和errors.add_to_base

我使用了errors.add_to_base，因为该错误不完全是该模型的一部分，但仍会阻止保存。

第三，您似乎在某个时候减少了 product.stocknumber。可能作为 before_validation，因此在检查 product.stocknumber 在 save 调用之前是否为 0 期间，product.stocknumber 很可能小于 0。

因此，让我们更改 if 条件以反映这一点。

unless product.stocknumber > 0

最后，您正在使用 before_save 回调，因此仅添加错误不会取消事务。您需要在保存/创建/更新/valdiaiton 回调之前/之后返回 false 以取消事务。

把这一切放在一起给你

before_save :check_product_stock

def check_product_stock
  unless product.stocknumber > 0
   errors.add_to_base "Producto con pedos de stock"
   return false
  end
  true # guards against returning nil which is interpreted as false.
end

至于显示这些错误，您可以在视图中的对象上使用不错的 error_messages_for 帮助器。或者将错误复制到控制器中的闪存哈希中。

在视图中：

<%= error_mesages_for :order %>

或者，在 else 块中的控制器中if @order.save：

 flash[:errors] = @order.errors.full_messages.join "<br />"

在闪存哈希中传递错误时，我更喜欢错误而不是通知，因为如果一个操作碰巧同时产生通知或错误，它可以通过 CSS 轻松区分两者。此外，始终显示带有红色文本的类的 flash[:errors] 更加干燥，将逻辑写入您的视图以确定 flash[:notice] 的内容是错误还是通知。

于 2009-11-18T04:46:08.500 回答
2

闪光灯在该型号中不可用。你需要做这样的事情：

errors.add :stocknumber, "Producto con pedos de stock"

然后在控制器和视图中使用 flash。

于 2009-11-18T02:20:11.037 回答