From 11183862db2498865e36489669b0b7cccc736758 Mon Sep 17 00:00:00 2001 From: TheLime03 Date: Fri, 16 Feb 2024 20:35:18 +0100 Subject: [PATCH] Refactor code to use string parameter instead of text parameter in BeautifulSoup find method --- esprit/utils.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/esprit/utils.py b/esprit/utils.py index 045e035..60f475c 100644 --- a/esprit/utils.py +++ b/esprit/utils.py @@ -20,9 +20,9 @@ def get_student_name(self): response = self.session.get(self.url) soup = BeautifulSoup(response.text, 'html.parser') - # Check if the

tag with the class "lead" and the text "Vous pouvez consulter dans cet espace :" exists + # Check if the

tag with the class "lead" and the string "Vous pouvez consulter dans cet espace :" exists p_tag = soup.find('p', class_='lead', - text='Vous pouvez consulter dans cet espace :') + string='Vous pouvez consulter dans cet espace :') if p_tag is None: print("The page does not contain the expected text.") return None @@ -43,15 +43,14 @@ def get_student_class(self): response = self.session.get(self.url) soup = BeautifulSoup(response.text, 'html.parser') - # Check if the

tag with the class "lead" and the text "Vous pouvez consulter dans cet espace :" exists + # Check if the

tag with the class "lead" and the string "Vous pouvez consulter dans cet espace :" exists p_tag = soup.find('p', class_='lead', - text='Vous pouvez consulter dans cet espace :') + string='Vous pouvez consulter dans cet espace :') if p_tag is None: print("The page does not contain the expected text.") return None - span = soup.find( - 'span', {'id': 'Label3', 'style': 'color:#CC0000;font-weight:bold;'}) + span = soup.find('span', {'id': 'Label3'}) if span is not None: return span.text.strip() else: