lib/cieloz/requisicao_transacao.rb in cieloz-0.0.8 vs lib/cieloz/requisicao_transacao.rb in cieloz-0.0.9

@@ -21,22 +21,22 @@
   end
 
   validate :parcela_minima?,
     if: "not @dados_pedido.nil? and not @forma_pagamento.nil?"
 
-  validates :autorizar, presence: true, inclusion: {
+  validates :autorizar, inclusion: {
     in: [
       SOMENTE_AUTENTICAR, AUTORIZAR_SE_AUTENTICADA,
       AUTORIZAR_NAO_AUTENTICADA, AUTORIZACAO_DIRETA, RECORRENTE
     ]
   }
   # validates string values because false.blank? is true, failing presence validation
-  validates :capturar,  presence: true, inclusion: { in: ["true", "false"] }
+  validates :capturar, inclusion: { in: ["true", "false"] }
 
   with_options if: "@autorizar != AUTORIZACAO_DIRETA" do |txn|
     txn.validates :url_retorno, presence: true
-    txn.validates :url_retorno, length: { in: 1..1024 }
+    txn.validates :url_retorno, length: { maximum: 1024 }
   end
 
   validates :campo_livre, length: { maximum: 128 }
 
   def nested_validations
@@ -50,13 +50,12 @@
       end
     }
   end
 
   def parcela_minima?
-    if @dados_pedido.valid? and @forma_pagamento.valid?
-      if @dados_pedido.valor / @forma_pagamento.parcelas < 500
-        errors.add :forma_pagamento, :minimum_installment_not_satisfied
-      end
+    valor, parcelas = @dados_pedido.valor.to_i, @forma_pagamento.parcelas.to_i
+    if parcelas > 0 and valor / parcelas < 500
+      @dados_pedido.errors.add :valor, :minimum_installment_not_satisfied
     end
   end
 
   def somente_autenticar
     @autorizar = SOMENTE_AUTENTICAR