diff options
| author | Levi Bard <taktaktaktaktaktaktaktaktaktak@gmail.com> | 2022-05-16 09:29:01 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-05-16 09:29:01 +0200 |
| commit | 94e98864e39c010635e839fea984f2b4893bef1a (patch) | |
| tree | 7a24465db019e89b4fe079a58918cf00c9acbf58 /app/validators | |
| parent | dc350be6f5a9f59385e6fbc7a06b5b9c0d57aec8 (diff) | |
Allow import/export of instance-level domain blocks/allows (#1754)
* Allow import/export of instance-level domain blocks/allows. Fixes #15095 * Pacify circleci * Address simple code review feedback * Add headers to exported CSV * Extract common import/export functionality to AdminExportControllerConcern * Add additional fields to instance-blocked domain export * Address review feedback * Split instance domain block/allow import/export into separate pages/controllers * Address code review feedback * Pacify DeepSource * Work around Paperclip::HasAttachmentFile for Rails 6 * Fix deprecated API warning in export tests * Remove after_commit workaround
Diffstat (limited to 'app/validators')
| -rw-r--r-- | app/validators/admin_import_validator.rb | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/app/validators/admin_import_validator.rb b/app/validators/admin_import_validator.rb new file mode 100644 index 000000000..338ceb3a7 --- /dev/null +++ b/app/validators/admin_import_validator.rb @@ -0,0 +1,19 @@ +# frozen_string_literal: true + +class AdminImportValidator < ActiveModel::Validator + FIRST_HEADER = '#domain' + + def validate(import) + return if import.type.blank? || import.data.blank? + + # We parse because newlines could be part of individual rows. This + # runs on create so we should be reading the local file here before + # it is uploaded to object storage or moved anywhere... + csv_data = CSV.parse(import.data.queued_for_write[:original].read) + + row_count = csv_data.size + row_count -= 1 if csv_data.first&.first == FIRST_HEADER + + import.errors.add(:data, I18n.t('imports.errors.over_rows_processing_limit', count: Admin::DomainBlocksController::ROWS_PROCESSING_LIMIT)) if row_count > Admin::DomainBlocksController::ROWS_PROCESSING_LIMIT + end +end |