app.py

@@ -55,7 +55,7 @@ async def handleException(request, exception):
 		traceback.print_exc()
 
 	if statusCode == 403:
-		await clear_session(r)
+		clear_session(r)
 	return r
 
 

email_util.py

@@ -80,17 +80,16 @@ async def send_unconfirm_message(room_order, orders):
 	issues_html += "</ul>"
 
 	for member in orders:
-		if(member.status != 'canceled'):
+		plain_body = EMAILS_TEXT["ROOM_UNCONFIRM_TEXT"]['plain'].format(member.name, room_order.room_name, issues_plain)
-			plain_body = EMAILS_TEXT["ROOM_UNCONFIRM_TEXT"]['plain'].format(member.name, room_order.room_name, issues_plain)
+		html_body = render_email_template(EMAILS_TEXT["ROOM_UNCONFIRM_TITLE"], EMAILS_TEXT["ROOM_UNCONFIRM_TEXT"]['html'].format(member.name, room_order.room_name, issues_html))
-			html_body = render_email_template(EMAILS_TEXT["ROOM_UNCONFIRM_TITLE"], EMAILS_TEXT["ROOM_UNCONFIRM_TEXT"]['html'].format(member.name, room_order.room_name, issues_html))
+		plain_text = MIMEText(plain_body, "plain")
-			plain_text = MIMEText(plain_body, "plain")
+		html_text = MIMEText(html_body, "html")
-			html_text = MIMEText(html_body, "html")
+		message = MIMEMultipart("alternative")
-			message = MIMEMultipart("alternative")
+		message.attach(plain_text)
-			message.attach(plain_text)
+		message.attach(html_text)
-			message.attach(html_text)
+		message['Subject'] = f'[{EMAIL_SENDER_NAME}] Your room cannot be confirmed'
-			message['Subject'] = f'[{EMAIL_SENDER_NAME}] Your room cannot be confirmed'
+		message['To'] = f"{member.name} <{member.email}>"
-			message['To'] = f"{member.name} <{member.email}>"
+		memberMessages.append(message)
-			memberMessages.append(message)
 
 	if len(memberMessages) == 0: return
 

ext.py

@@ -20,10 +20,6 @@ class Order:
 	
 		self.time = time()
 		self.data = data
-		if(len(self.data['positions']) == 0):
-			for fee in data['fees']:
-				if(fee['fee_type'] == "cancellation"):
-					self.data['status'] = 'c'
 		self.status = {'n': 'pending', 'p': 'paid', 'e': 'expired', 'c': 'canceled'}[self.data['status']]
 		self.secret = data['secret']
 
@@ -222,9 +218,8 @@ class Order:
 			#if ans['question'] == 40:
 			#	del self.answers[i]['options']
 			#	del self.answers[i]['option_identifiers']
-
+		
-		ans = [] if self.status == "canceled" else self.answers 
+		res = await pretixClient.patch(f'orderpositions/{self.position_id}/', json={'answers': self.answers}, expectedStatusCodes=None)
-		res = await pretixClient.patch(f'orderpositions/{self.position_id}/', json={'answers': ans}, expectedStatusCodes=None)
 		
 		if res.status_code != 200:
 			e = res.json()

messages.py

@@ -3,16 +3,15 @@ ROOM_ERROR_TYPES = {
 	'unpaid': "Somebody in your room has not paid for their reservation, yet.",
 	'type_mismatch': "A member in your room has a ticket for a different type of room capacity. This happens when users swap their room types with others, without abandoning the room.",
 	'daily': "Some member in your room has a Daily ticket. These tickets do not include a hotel reservation.",
-	'capacity_mismatch': "The number of people in your room mismatches your type of ticket.",
+	'capacity_mismatch': "The number of people in your room mismatches your type of ticket."
-	'canceled': "Someone in your room canceled his booking and it was removed from your room."
 }
 
 EMAILS_TEXT = {
 	"ROOM_UNCONFIRM_TITLE": "Your room got unconfirmed",
 	"ROOM_UNCONFIRM_TEXT": {
-		'html': "Hello <b>{0}</b><br>We had to <b>unconfirm or change</b> your room <i>'{1}'</i> due to the following issues:<br></p>{2}<br><p>Please contact your room's owner or contact our support for further informations at <a href=\"https://furizon.net/contact/\"> https://furizon.net/contact/</a>.<br>Thank you.<br><br><a class=\"link\" style=\"background-color: #1095c1; color: #fff;\" href=\"https://reg.furizon.net/manage/welcome\">Manage booking</a>",
+		'html': "Hello <b>{0}</b><br>We had to <b>unconfirm</b> your room <i>'{1}'</i> due to the following issues:<br></p>{2}<br><p>Please contact your room's owner or contact our support for further informations at <a href=\"https://furizon.net/contact/\"> https://furizon.net/contact/</a>.<br>Thank you.<br><br><a class=\"link\" style=\"background-color: #1095c1; color: #fff;\" href=\"https://reg.furizon.net/manage/welcome\">Manage booking</a>",
 
-		'plain': "Hello {0}\nWe had to unconfirm or change your room '{1}' due to the following issues:\n{2}\nPlease contact your room's owner or contact our support for further informations at https://furizon.net/contact/.\nThank you\n\nTo manage your booking: https://reg.furizon.net/manage/welcome"
+		'plain': "Hello {0}\nWe had to unconfirm your room '{1}' due to the following issues:\n{2}\nPlease contact your room's owner or contact our support for further informations at https://furizon.net/contact/.\nThank you\n\nTo manage your booking: https://reg.furizon.net/manage/welcome"
 	},
 	
 

utils.py

@@ -164,17 +164,6 @@ async def unconfirm_room_by_order(order, room_members=None, throw=True, request=
 		await p.edit_answer('room_confirmed', "False")
 		await p.send_answers()
 
-async def remove_members_from_room(order, removeMembers):
-	didSomething = False
-	for member in removeMembers:
-		if (member in order.room_members):
-			order.room_members.remove(member)
-			didSomething = True
-	if(didSomething):
-		await order.edit_answer("room_members", ','.join(order.room_members))
-		await order.send_answers()
-	return didSomething
-
 async def validate_rooms(request, rooms, om):
 	logger.info('Validating rooms...')
 	if not om: om = request.app.ctx.om
@@ -182,7 +171,6 @@ async def validate_rooms(request, rooms, om):
 	# rooms_to_unconfirm is the room that MUST be unconfirmed, room_with_errors is a less strict set containing all rooms with kind-ish errors
 	rooms_to_unconfirm = []
 	room_with_errors = []
-	remove_members = []
 
 	# Validate rooms
 	for order in rooms:
@@ -207,58 +195,43 @@ async def validate_rooms(request, rooms, om):
 	# Get confirmed rooms that fail validation
 	failed_confirmed_rooms = list(filter(lambda fr: (fr[0].room_confirmed == True), rooms_to_unconfirm))
 
-	didSomething = False
-
 	if len(failed_confirmed_rooms) == 0:
 		logger.info('[ROOM VALIDATION] No rooms to unconfirm.')
-	else:
+		return
-		didSomething = True
-		logger.info(f"[ROOM VALIDATION] Trying to unconfirm {len(failed_confirmed_rooms)} rooms...")
 
-		# Try unconfirming them
+	logger.info(f"[ROOM VALIDATION] Trying to unconfirm {len(failed_confirmed_rooms)} rooms...")
-		for rtu in failed_confirmed_rooms:
+
-			order = rtu[0]
+	# Try unconfirming them
-			member_orders = rtu[2]
+	for rtu in failed_confirmed_rooms:
-			logger.warning(f"[ROOM VALIDATION] [UNCONFIRMING] Unconfirming room {order.code}...")
+		order = rtu[0]
-			
+		member_orders = rtu[2]
-			# Unconfirm and email users about the room
+		logger.warning(f"[ROOM VALIDATION] [UNCONFIRMING] Unconfirming room {order.code}...")
+		
+		# Unconfirm and email users about the room
+		if UNCONFIRM_ROOMS_ENABLE:
+			await unconfirm_room_by_order(order, member_orders, False, None, om)
+
+	logger.info(f"[ROOM VALIDATION] Sending unconfirm notice to room members...")
+	sent_count = 0
+	# Send unconfirm notice via email
+	for rtu in failed_confirmed_rooms:
+		order = rtu[0]
+		member_orders = rtu[2]
+		try:
 			if UNCONFIRM_ROOMS_ENABLE:
-				await unconfirm_room_by_order(order, member_orders, False, None, om)
+				await send_unconfirm_message(order, member_orders)
-
+			sent_count += len(member_orders)
-	for r in rooms_to_unconfirm:
+		except Exception as ex:
-		order = r[0]
+			if EXTRA_PRINTS: logger.exception(str(ex))
-		removeMembers = r[3]
+	logger.info(f"[ROOM VALIDATION] Sent {sent_count} emails")
-		if len(removeMembers) > 0:
-			logger.warning(f"[ROOM VALIDATION] [REMOVING] Removing members '{','.join(removeMembers)}' from room {order.code}")
-
-			if UNCONFIRM_ROOMS_ENABLE:
-				didSomething |= await remove_members_from_room(order, removeMembers)
-			if(r not in failed_confirmed_rooms): failed_confirmed_rooms.append(r)
-
-
-	if(didSomething):
-		logger.info(f"[ROOM VALIDATION] Sending unconfirm notice to room members...")
-		sent_count = 0
-		# Send unconfirm notice via email
-		for rtu in failed_confirmed_rooms:
-			order = rtu[0]
-			member_orders = rtu[2]
-			try:
-				if UNCONFIRM_ROOMS_ENABLE:
-					await send_unconfirm_message(order, member_orders)
-				sent_count += len(member_orders)
-			except Exception as ex:
-				if EXTRA_PRINTS: logger.exception(str(ex))
-		logger.info(f"[ROOM VALIDATION] Sent {sent_count} emails")
 		
 
 async def check_room(request, order, om=None):
 	room_errors = []
 	room_members = []
-	remove_members = []
 	use_cached = request == None
 	if not om: om = request.app.ctx.om
-	if not order or not order.room_id or order.room_id != order.code: return (order, False, room_members, remove_members)
+	if not order or not order.room_id or order.room_id != order.code: return (order, False, room_members)
 	
 	# This is not needed anymore you buy tickets already 
 	#if quotas.get_left(len(order.room_members)) == 0:
@@ -276,12 +249,8 @@ async def check_room(request, order, om=None):
 		if res.room_id != order.code:
 			room_errors.append((res.code, 'room_id_mismatch'))
 			allOk = False
-
+		
-		if res.status == 'canceled':
+		if res.status != 'paid':
-			room_errors.append((res.code, 'canceled'))
-			remove_members.append(res.code)
-			allOk = False
-		elif res.status != 'paid':
 			room_errors.append((res.code, 'unpaid'))
 		
 		if res.bed_in_room != bed_in_room:
@@ -301,4 +270,4 @@ async def check_room(request, order, om=None):
 		if order.room_confirmed:
 			allOk = False
 	order.set_room_errors(room_errors)
-	return (order, allOk, room_members, remove_members)
+	return (order, allOk, room_members)