Skip to content

Commit 7df616b

Browse files
committed
simplify salesChannel property accessor
1 parent 98aaff9 commit 7df616b

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

WooCommerce/Classes/ViewRelated/Orders/Order Details/Order Summary Section/SummaryTableViewCell.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ final class SummaryTableViewCell: UITableViewCell {
3131
func configure(_ viewModel: SummaryTableViewCellViewModel) {
3232
titleLabel.text = viewModel.billedPersonName
3333
subtitleLabel.text = viewModel.subtitle
34-
salesChannelLabel.text = viewModel.formattedSalesChannel
34+
salesChannelLabel.text = viewModel.salesChannel
3535
salesChannelLabel.isHidden = (salesChannelLabel.text == nil)
3636
display(presentation: viewModel.presentation)
3737
}

WooCommerce/Classes/ViewRelated/Orders/Order Details/Order Summary Section/SummaryTableViewCellViewModel.swift

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,21 +9,19 @@ struct SummaryTableViewCellViewModel {
99

1010
private let billingAddress: Address?
1111
private let dateCreated: Date
12-
private let salesChannel: String?
12+
private(set) var salesChannel: String?
1313

1414
let presentation: OrderStatusPresentation
1515

1616
private let calendar: Calendar
17-
private let order: Order
1817

1918
init(order: Order,
2019
status: OrderStatus?,
2120
calendar: Calendar = .current) {
22-
self.order = order
2321

2422
billingAddress = order.billingAddress
2523
dateCreated = order.dateCreated
26-
salesChannel = order.createdVia
24+
salesChannel = order.salesChannel?.description
2725

2826
presentation = OrderStatusPresentation(
2927
style: status?.status ?? order.status,
@@ -50,12 +48,6 @@ struct SummaryTableViewCellViewModel {
5048
formatter.timeZone = .siteTimezone
5149
return formatter.string(from: dateCreated)
5250
}
53-
54-
/// Textual representation of the sales channel
55-
///
56-
var formattedSalesChannel: String? {
57-
order.salesChannel?.description
58-
}
5951
}
6052

6153
private extension SummaryTableViewCellViewModel {

0 commit comments

Comments
 (0)