Story#114471339 - move parse_legal_document_from_frame to one braking point

This commit is contained in:
Vladimir Krylov 2016-02-25 17:01:41 +02:00
parent 7a659da6a3
commit 30fe6bd1e6

View file

@ -150,7 +150,7 @@ class Epp::Contact < Contact
legal_frame = frame.css('legalDocument').first legal_frame = frame.css('legalDocument').first
at[:legal_documents_attributes] = self.class.legal_document_attrs(legal_frame) at[:legal_documents_attributes] = self.class.legal_document_attrs(legal_frame)
if doc = attach_legal_document(parse_legal_document_from_frame(frame)) if doc = attach_legal_document(Epp::Domain.parse_legal_document_from_frame(frame))
frame.css("legalDocument").first.content = doc.path if doc && doc.persisted? frame.css("legalDocument").first.content = doc.path if doc && doc.persisted?
self.legal_document_id = doc.id self.legal_document_id = doc.id
end end
@ -227,7 +227,7 @@ class Epp::Contact < Contact
end end
def add_legal_file_to_new frame def add_legal_file_to_new frame
legal_document_data = Epp::Contact.parse_legal_document_from_frame(frame) legal_document_data = Epp::Domain.parse_legal_document_from_frame(frame)
return unless legal_document_data return unless legal_document_data
doc = LegalDocument.create( doc = LegalDocument.create(
@ -242,16 +242,4 @@ class Epp::Contact < Contact
self.legal_document_id = doc.id self.legal_document_id = doc.id
end end
def parse_legal_document_from_frame frame
ld = frame.css('legalDocument').first
return nil unless ld
return nil if ld.text.starts_with?(ENV['legal_documents_dir'])
{
body: ld.text,
type: ld['type']
}
end
end end