From 0ef04b2506c84433179248e3a1636306e1612b2e Mon Sep 17 00:00:00 2001
From: Martin Lensment <mlensment@gmail.com>
Date: Thu, 10 Sep 2015 16:21:40 +0300
Subject: [PATCH] Refactor #2623

---
 app/controllers/epp/domains_controller.rb |  1 -
 app/models/epp/domain.rb                  | 22 +++++++++++++---------
 db/schema-read-only.rb                    |  1 +
 db/structure.sql                          |  3 +++
 4 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/app/controllers/epp/domains_controller.rb b/app/controllers/epp/domains_controller.rb
index d3111ddfa..1e778b839 100644
--- a/app/controllers/epp/domains_controller.rb
+++ b/app/controllers/epp/domains_controller.rb
@@ -63,7 +63,6 @@ class Epp::DomainsController < EppController
 
   def delete
     authorize! :delete, @domain, @password
-
     # all includes for bullet
     @domain = Epp::Domain.where(id: @domain.id).includes(nameservers: :versions).first
 
diff --git a/app/models/epp/domain.rb b/app/models/epp/domain.rb
index a19a0d772..c3215d774 100644
--- a/app/models/epp/domain.rb
+++ b/app/models/epp/domain.rb
@@ -487,17 +487,21 @@ class Epp::Domain < Domain
       manage_automatic_statuses
       true # aka 1001 pending_delete
     else
-      throw :epp_error, {
-        code: '2304',
-        msg: I18n.t(:object_status_prohibits_operation)
-      } unless pending_deletable?
-
-      self.delete_at = Time.zone.now + Setting.redemption_grace_period.days
-      set_pending_delete
-      set_server_hold if server_holdable?
-      save(validate: false)
+      set_pending_delete!
     end
   end
+
+  def set_pending_delete!
+    throw :epp_error, {
+      code: '2304',
+      msg: I18n.t(:object_status_prohibits_operation)
+    } unless pending_deletable?
+
+    self.delete_at = Time.zone.now + Setting.redemption_grace_period.days
+    set_pending_delete
+    set_server_hold if server_holdable?
+    save(validate: false)
+  end
   # rubocop: enable Metrics/PerceivedComplexity
   # rubocop: enable Metrics/CyclomaticComplexity
 
diff --git a/db/schema-read-only.rb b/db/schema-read-only.rb
index 7bcf1f877..f7de8c32d 100644
--- a/db/schema-read-only.rb
+++ b/db/schema-read-only.rb
@@ -201,6 +201,7 @@ ActiveRecord::Schema.define(version: 20150910113839) do
     t.integer  "legacy_id"
     t.string   "statuses",           array: true
     t.hstore   "status_notes"
+    t.integer  "legacy_history_id"
     t.integer  "copy_from_id"
   end
 
diff --git a/db/structure.sql b/db/structure.sql
index b2c6c0bca..c517c6576 100644
--- a/db/structure.sql
+++ b/db/structure.sql
@@ -597,6 +597,7 @@ CREATE TABLE contacts (
     legacy_id integer,
     statuses character varying[],
     status_notes hstore,
+    legacy_history_id integer,
     copy_from_id integer
 );
 
@@ -4929,5 +4930,7 @@ INSERT INTO schema_migrations (version) VALUES ('20150810114746');
 
 INSERT INTO schema_migrations (version) VALUES ('20150825125118');
 
+INSERT INTO schema_migrations (version) VALUES ('20150827151906');
+
 INSERT INTO schema_migrations (version) VALUES ('20150910113839');