fixed law of demeter problem for courier notification

fairmondo · Jan 22, 2015 · 6ad03c8 · 6ad03c8
1 parent 7063ef1
commit 6ad03c8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/models/business_transaction.rb b/app/models/business_transaction.rb
@@ -47,7 +47,7 @@ class BusinessTransaction < ActiveRecord::Base
            :active?, :transport_time,
            to: :article, prefix: true
   delegate :email, :nickname, to: :buyer, prefix: true
-  delegate :nickname, to: :seller, prefix: true
+  delegate :nickname, :phone, :mobile, to: :seller, prefix: true
   delegate :title, :first_name, :last_name, :address_line_1, :address_line_2, :company_name,
            :zip, :city, :country, to: :transport_address, prefix: true
   delegate :title, :first_name, :last_name, :address_line_1, :address_line_2, :company_name,

diff --git a/app/views/cart_mailer/courier_notification.text.erb b/app/views/cart_mailer/courier_notification.text.erb
@@ -17,8 +17,8 @@ Abholadresse:
 <%= @business_transaction.seller.standard_address_address_line_1 %>
 <%= @business_transaction.seller.standard_address_address_line_2 if @business_transaction.seller.standard_address_address_line_2.present? %>
 <%= @business_transaction.seller.standard_address_zip + ' ' + @business_transaction.seller.standard_address_city %>
-<%= @business_transaction.seller.phone if @business_transaction.seller.phone.present? %>
-<%= @business_transaction.seller.mobile if @business_transaction.seller.mobile.present? %>
+<%= @business_transaction.seller_phone if @business_transaction.seller_phone.present? %>
+<%= @business_transaction.seller_mobile if @business_transaction.seller_mobile.present? %>
 
 --------------------------------------------------