Refactor

README.md

@@ -63,7 +63,6 @@ For development configuration, add:
 ```
 
 Configuration on plain TCP EPP is as follows:
-Add:
 ```apache
 <IfModule mod_epp.c>
   <Directory "/usr/lib/cgi-bin/epp">

app/controllers/epp/commands_controller.rb

@@ -1,6 +1,5 @@
 class Epp::CommandsController < ApplicationController
   include Epp::Common
-  include Epp::CommandsHelper
   include Epp::DomainsHelper
 
   OBJECT_TYPES = {

app/helpers/epp/commands_helper.rb

@@ -1,6 +0,0 @@
-module Epp::CommandsHelper
-  def command_params_for type
-    node_set = parsed_frame.css("epp command #{type} #{type}").children.select(&:element?)
-    node_set.inject({}) {|hash, obj| hash[obj.name.to_sym] = obj.text;hash }
-  end
-end

app/helpers/epp/domains_helper.rb

@@ -12,14 +12,16 @@ module Epp::DomainsHelper
   ### HELPER METHODS ###
 
   def domain_create_params
-    cp = command_params_for('create')
+    node_set = parsed_frame.css("epp command create create").children.select(&:element?)
+    command_params = node_set.inject({}) {|hash, obj| hash[obj.name.to_sym] = obj.text;hash }
+
     {
-      name: cp[:name],
+      name: command_params[:name],
       registrar_id: current_epp_user.registrar.try(:id),
       registered_at: Time.now,
       valid_from: Date.today,
-      valid_to: Date.today + cp[:period].to_i.years,
+      valid_to: Date.today + command_params[:period].to_i.years,
-      auth_info: cp[:authInfo]
+      auth_info: command_params[:authInfo]
     }
   end