MobilePayTool indsætter 0 payments når et payment bliver ignoreret

[joandrsn]

Siden mobilepayments er blevet indført, var det nødvendigt at lave dette hack for at mails ikke blev afsendt hvis et mobilepayment blev ignoreret:

stregsystemet/stregsystem/models.py

Line 322 in fbaf833

if self.member.email != "" and self.amount != 0:

Jeg tænker bare at fjerne hele else-blokken i dette stykke kode.

stregsystemet/stregsystem/models.py

Lines 385 to 389 in fbaf833

	else:
	# otherwise it's an IGNORED payment
	payment = Payment(
	member=processed_payment.member,
	amount=0)

Payments med 0 i amount giver måske bare mere forvirring end det gavner.

Dette burde have været en del af mit PR review af #204, men det glemte jeg.

[falkecarlsen]

Helt enig. Indtil videre er der kun ~10 ignorerede mobilepayments i produktion som er påvirket af dette, så det vil være overkommeligt at rulle tilbage.

[falkecarlsen]

Resolved by #253

[falkecarlsen]

Not a problem to keep the amount != 0 check although redundant.