remove unneeded class include; use openstruct for large struct in logon profile

Isaac Lewis
1 parent 7510ff56
Showing 2 changed files with 30 additions and 64 deletions Show diff stats
lib/syspro/api_operations/query.rb
lib/syspro/get_logon_profile.rb
@@ -4,8 +4,6 @@ module Syspro
   module ApiOperations
     module Query
       module ClassMethods
-        include Request
-
         def browse(params)
           request(:get, '/Query/Browse', params)
         end
@@ -15,70 +15,38 @@ module Syspro
     def self.parse_response(resp) # rubocop:disable Metrics/MethodLength
       doc = resp.data
-      UserProfile.new(
-        doc.xpath('//CompanyName').text,
-        doc.xpath('//OperatorCode').text,
-        doc.xpath('//OperatorGroup').text,
-        doc.xpath('//OperatorEmailAddress').text,
-        doc.xpath('//OperatorLocation').text,
-        doc.xpath('//OperatorLanguageCode').text,
-        doc.xpath('//SystemLanguage').text,
-        doc.xpath('//AccountingDate').text,
-        doc.xpath('//CompanyDate').text,
-        doc.xpath('//DefaultArBranch').text,
-        doc.xpath('//DefaultApBranch').text,
-        doc.xpath('//DefaultBank').text,
-        doc.xpath('//DefaultWarehouse').text,
-        doc.xpath('//DefaultCustomer').text,
-        doc.xpath('//SystemSiteId').text,
-        doc.xpath('//SystemNationalityCode').text,
-        doc.xpath('//LocalCurrencyCode').text,
-        doc.xpath('//CurrencyDescription').text,
-        doc.xpath('//DefaultRequisitionUser').text,
-        doc.xpath('//XMLToHTMLTransform').text,
-        doc.xpath('//CssStyle').text,
-        doc.xpath('//CssSuffix').text,
-        doc.xpath('//DecimalFormat').text,
-        doc.xpath('//DateFormat').text,
-        doc.xpath('//FunctionalRole').text,
-        doc.xpath('//DatabaseType').text,
-        doc.xpath('//SysproVersion').text,
-        doc.xpath('//EnetVersion').text,
-        doc.xpath('//SysproServerBitWidth').text
+      OpenStruct.new(
+        company_name: doc.xpath('//CompanyName').text,
+        operator_code: doc.xpath('//OperatorCode').text,
+        operator_code: doc.xpath('//OperatorGroup').text,
+        operator_email_address: doc.xpath('//OperatorEmailAddress').text,
+        operator_location: doc.xpath('//OperatorLocation').text,
+        operator_language_code: doc.xpath('//OperatorLanguageCode').text,
+        system_language: doc.xpath('//SystemLanguage').text,
+        accounting_date: doc.xpath('//AccountingDate').text,
+        company_date: doc.xpath('//CompanyDate').text,
+        default_ar_branch: doc.xpath('//DefaultArBranch').text,
+        default_ap_branch: doc.xpath('//DefaultApBranch').text,
+        default_bank: doc.xpath('//DefaultBank').text,
+        default_warehouse: doc.xpath('//DefaultWarehouse').text,
+        default_customer: doc.xpath('//DefaultCustomer').text,
+        system_site_id: doc.xpath('//SystemSiteId').text,
+        system_nationality_code: doc.xpath('//SystemNationalityCode').text,
+        local_currency_code: doc.xpath('//LocalCurrencyCode').text,
+        currency_description: doc.xpath('//CurrencyDescription').text,
+        default_requisition_user: doc.xpath('//DefaultRequisitionUser').text,
+        xml_to_html_transform: doc.xpath('//XMLToHTMLTransform').text,
+        css_style: doc.xpath('//CssStyle').text,
+        css_suffix: doc.xpath('//CssSuffix').text,
+        decimal_format: doc.xpath('//DecimalFormat').text,
+        date_format: doc.xpath('//DateFormat').text,
+        functional_role: doc.xpath('//FunctionalRole').text,
+        database_type: doc.xpath('//DatabaseType').text,
+        syspro_version: doc.xpath('//SysproVersion').text,
+        enet_version: doc.xpath('//EnetVersion').text,
+        syspro_server_bit_width: doc.xpath('//SysproServerBitWidth').text
       )
     end
     private_class_method :parse_response
-
-    UserProfile = Struct.new(
-      :company_name,
-      :operator_code,
-      :operator_group,
-      :operator_email_address,
-      :operator_location,
-      :operator_language_code,
-      :system_language,
-      :accounting_date,
-      :company_date,
-      :default_ar_branch,
-      :default_ap_branch,
-      :default_bank,
-      :default_warehouse,
-      :default_customer,
-      :system_site_id,
-      :system_nationality_code,
-      :local_currency_code,
-      :currency_description,
-      :default_requisition_user,
-      :xml_to_html_transform,
-      :css_style,
-      :css_suffix,
-      :decimal_format,
-      :date_format,
-      :functional_role,
-      :database_type,
-      :syspro_version,
-      :enet_version,
-      :syspro_server_bit_width
-    )
   end
 end