Merge pull request #104 from internetee/113524121-legal_doc_min_size

113524121 legal doc min size
This commit is contained in:
Timo Võhmar 2016-03-16 17:13:23 +02:00
commit 9e32f71d2f
6 changed files with 20 additions and 2 deletions

View file

@ -27,3 +27,16 @@ $(document).on 'page:change', ->
form.find('.js-ident-tip').show()
$('.js-contact-form').trigger('restoreDefault')
$('[data-legal-document]').each (i, fileInput)->
minSize = 8 * 1024 # 8kB
maxSize = 8 * 1024 * 1024; # 8 MB
$(fileInput).closest('form').submit (e) ->
if (files = fileInput.files).length
fileSize = files[0].size
if fileSize < minSize
alert 'Document size should be more than 8kB'
return false
else if fileSize > maxSize
alert 'Document size should be less than 8MB'
return false

View file

@ -5,6 +5,7 @@ class Epp::Contact < Contact
self.inheritance_column = :sti_disabled
before_validation :manage_permissions
def manage_permissions
return unless update_prohibited? || delete_prohibited?
add_epp_error('2304', nil, nil, I18n.t(:object_status_prohibits_operation))

View file

@ -6,6 +6,7 @@ class Epp::Domain < Domain
attr_accessor :is_renewal, :is_transfer
before_validation :manage_permissions
def manage_permissions
return if is_admin # this bad hack for 109086524, refactor later
return true if is_transfer || is_renewal

View file

@ -9,6 +9,9 @@ class LegalDocument < ActiveRecord::Base
belongs_to :documentable, polymorphic: true
validates :body, length: { minimum: (1.37 * 8.kilobytes).ceil }, if: ->(file){ file.path.blank? && !Rails.env.staging?}
before_create :add_creator
before_save :save_to_filesystem

View file

@ -10,4 +10,4 @@
= f.label :legal_document, t(:legal_document)
%p.help-block= t(:legal_document_max_size)
.col-md-7
= f.file_field :legal_document, :value => ''
= f.file_field :legal_document, :value => '', data: {legal_document: true}

View file

@ -18,7 +18,7 @@
= label_tag 'domain[legal_document]', t(:legal_document), class: c
%p.help-block= t(:legal_document_max_size)
.col-md-7
= file_field_tag 'domain[legal_document]', required: fr
= file_field_tag 'domain[legal_document]', required: fr, data: {legal_document: true}
.col-md-4
%p.domain-general-help= t(:domain_general_help).html_safe
%p.domain-admin-contact-help= t(:domain_admin_contact_help).html_safe