Skip to content

Commit 011ac8a

Browse files
committed
Fix seller reports for leaderless groups
1 parent b12568f commit 011ac8a

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

uber/site_sections/dealer_reports.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -176,11 +176,13 @@ def seller_comptroller_info(self, out, session):
176176
rows = []
177177
for group in dealer_groups:
178178
if group.status in c.DEALER_ACCEPTED_STATUSES and group.is_dealer:
179+
name = (group.leader.legal_name or group.leader.full_name) if group.leader else ''
180+
phone = group.phone or (group.leader.cellphone if group.leader else '')
179181
rows.append([
180182
group.name,
181183
group.email,
182-
group.leader.legal_name or group.leader.full_name,
183-
group.phone if group.phone else group.leader.cellphone,
184+
name,
185+
phone,
184186
group.address1,
185187
group.address2,
186188
group.city,
@@ -248,13 +250,15 @@ def seller_tax_info(self, out, session):
248250
approved_groups = session.query(Group).filter(Group.status.in_(c.DEALER_ACCEPTED_STATUSES)).all()
249251
rows = []
250252
for group in approved_groups:
253+
name = group.leader.full_name if group.leader else ''
254+
phone = group.phone or (group.leader.cellphone if group.leader else '')
251255
if group.is_dealer:
252256
rows.append([
253257
group.name,
254-
group.leader.full_name,
258+
name,
255259
group.email,
256260
group.physical_address,
257-
group.phone if group.phone else group.leader.cellphone,
261+
phone,
258262
group.special_needs,
259263
group.admin_notes,
260264
group.wares,

0 commit comments

Comments
 (0)