From 6e7a104fb0b56904609011fff0dec177b3612dd6 Mon Sep 17 00:00:00 2001 From: Place Jean-Marie Date: Fri, 17 Sep 2021 12:18:55 +0200 Subject: [PATCH] pylint corrections (suite) --- app/scodoc/sco_placement.py | 122 +++++++++++++--------- app/templates/scodoc/forms/placement.html | 2 +- 2 files changed, 71 insertions(+), 53 deletions(-) diff --git a/app/scodoc/sco_placement.py b/app/scodoc/sco_placement.py index 55544e05..dd5e97eb 100644 --- a/app/scodoc/sco_placement.py +++ b/app/scodoc/sco_placement.py @@ -70,11 +70,31 @@ import sco_version COORD = "Coordonnées" SEQ = "Continue" +TOUS = "Tous" + + +def _get_group_info(evaluation_id): + # groupes + groups = sco_groups.do_evaluation_listegroupes( + evaluation_id, include_default=True + ) + has_groups = False + groups_tree = {} + for group in groups: + partition = group["partition_name"] or TOUS + group_id = group["group_id"] + group_name = group["group_name"] or TOUS + if partition not in groups_tree: + groups_tree[partition] = {} + groups_tree[partition][group_name] = group_id + if partition != TOUS: + has_groups = True + nb_groups = len(groups_tree) + return groups_tree, has_groups, nb_groups class PlacementForm(FlaskForm): """Formulaire pour placement des étudiants en Salle""" - TOUS = "Tous" evaluation_id = HiddenField("evaluation_id") file_format = RadioField( "Format de fichier", @@ -104,27 +124,20 @@ class PlacementForm(FlaskForm): ) submit = SubmitField("OK") + def __init__(self, formdata=None, data=None): + super().__init__(formdata=formdata, data=data) + self.groups_tree = {} + self.has_groups = None + self.group_tree_length = None + self.nb_groups = None + self.set_evaluation_infos(data["evaluation_id"]) + def set_evaluation_infos(self, evaluation_id): """Initialise les données du formulaire avec les données de l'évaluation.""" eval_data = sco_evaluations.do_evaluation_list({"evaluation_id": evaluation_id}) if not eval_data: raise ScoValueError("invalid evaluation_id") - # groupes - groups = sco_groups.do_evaluation_listegroupes( - evaluation_id, include_default=True - ) - self.groups_tree = {} - self.has_groups = False - for group in groups: - partition = group["partition_name"] or self.TOUS - group_id = group["group_id"] - group_name = group["group_name"] or self.TOUS - if partition not in self.groups_tree: - self.groups_tree[partition] = {} - self.groups_tree[partition][group_name] = group_id - if partition != self.TOUS: - self.has_groups = True - self.groups_tree_length = len(self.groups_tree) + self.groups_tree, self.has_groups, self.nb_groups = _get_group_info(evaluation_id) if self.has_groups: choices = [] for partition in self.groups_tree: @@ -141,6 +154,7 @@ class _DistributeurContinu: self.position = 1 def suivant(self): + """Retounre la désignation de la place suivante""" retour = self.position self.position += 1 return retour @@ -155,6 +169,7 @@ class _Distributeur2D: self.index = 1 def suivant(self): + """Retounre la désignation de la place suivante""" retour = (self.index, self.rang) self.rang += 1 if self.rang > self.nb_rangs: @@ -167,9 +182,8 @@ def placement_eval_selectetuds(evaluation_id): """Creation de l'écran de placement""" form = PlacementForm( request.form, - data={"evaluation_id": int(evaluation_id), "groups": PlacementForm.TOUS}, + data={"evaluation_id": int(evaluation_id), "groups": TOUS}, ) - form.set_evaluation_infos(evaluation_id) if form.validate_on_submit(): runner = PlacementRunner(form) if not runner.check_placement(): @@ -195,6 +209,7 @@ def placement_eval_selectetuds(evaluation_id): class PlacementRunner: """Execution de l'action définie par le formulaire""" + def __init__(self, form): """Calcul et génération du fichier sur la base des données du formulaire""" self.evaluation_id = form["evaluation_id"].data @@ -239,6 +254,9 @@ class PlacementRunner: "Controle : %s (coef. %g)" % (self.evaltitre, self.eval_data["coefficient"]), ] + self.styles = None + self.plan = None + self.listetud = None def check_placement(self): """Vérifie que l'utilisateur courant a le droit d'édition sur les notes""" @@ -248,6 +266,7 @@ class PlacementRunner: ) def exec_placement(self): + """Excéute l'action liée au formulaire""" self._repartition() if self.file_format == "xls": return self._production_xls() @@ -263,14 +282,11 @@ class PlacementRunner: self.plan = self._affectation_places() def _build_listetud(self): - if None in [g["group_name"] for g in self.groups]: # tous les etudiants - getallstudents = True - else: - getallstudents = False + get_all_students = None in [g["group_name"] for g in self.groups] # tous les etudiants etudids = sco_groups.do_evaluation_listeetuds_groups( self.evaluation_id, self.groups, - getallstudents=getallstudents, + getallstudents=get_all_students, include_dems=True, ) listetud = [] # liste de couples (nom,prenom) @@ -359,24 +375,24 @@ class PlacementRunner: ) return tab.make_page(format="pdf", with_html_headers=False) - def _one_header(self, ws): + def _one_header(self, worksheet): cells = [ - ws.make_cell("Nom", self.styles["2bi"]), - ws.make_cell("Prénom", self.styles["2bi"]), + worksheet.make_cell("Nom", self.styles["2bi"]), + worksheet.make_cell("Prénom", self.styles["2bi"]), ] if self.etiquetage == COORD: - cells.append(ws.make_cell("Colonne", self.styles["2bi"])) - cells.append(ws.make_cell("Ligne", self.styles["2bi"])) + cells.append(worksheet.make_cell("Colonne", self.styles["2bi"])) + cells.append(worksheet.make_cell("Ligne", self.styles["2bi"])) else: - cells.append(ws.make_cell("Place", self.styles["2bi"])) + cells.append(worksheet.make_cell("Place", self.styles["2bi"])) return cells - def _headers(self, ws, nb_listes): + def _headers(self, worksheet, nb_listes): cells = [] for _ in range(nb_listes): - cells += self._one_header(ws) - cells.append(ws.make_cell("")) - ws.append_row(cells) + cells += self._one_header(worksheet) + cells.append(worksheet.make_cell("")) + worksheet.append_row(cells) def _make_styles(self, ws0, ws1): # polices @@ -515,7 +531,7 @@ class PlacementRunner: ws0.append_row(cells_c) ws0.set_row_dimension_height(row, space / 25) - def _feuille1(self, ws, maxlines): + def _feuille1(self, worksheet, maxlines): # etudiants - feuille1 # structuration: # 1 page = maxlistes listes @@ -529,13 +545,13 @@ class PlacementRunner: widths = [] for _ in range(maxlistes): widths += gabarit - ws.set_column_dimension_width(value=widths) + worksheet.set_column_dimension_width(value=widths) nb_etu_restant = len(self.listetud) - self._titres(ws) + self._titres(worksheet) nb_listes = min( maxlistes, nb_etu_restant // maxlines + 1 ) # nombre de colonnes dans la page - self._headers(ws, nb_listes) + self._headers(worksheet, nb_listes) # construction liste alphabétique # Affichage lines = [[] for _ in range(maxlines)] @@ -544,31 +560,33 @@ class PlacementRunner: for etud in sorted(self.plan, key=lambda e: e[0][0]): # tri alphabétique # check for skip of list or page if col > 0: # add a empty cell between lists - lines[lineno].append(ws.make_cell()) - lines[lineno].append(ws.make_cell(etud[0][0], self.styles["2l"])) - lines[lineno].append(ws.make_cell(etud[0][1], self.styles["2m1"])) + lines[lineno].append(worksheet.make_cell()) + lines[lineno].append(worksheet.make_cell(etud[0][0], self.styles["2l"])) + lines[lineno].append(worksheet.make_cell(etud[0][1], self.styles["2m1"])) if self.etiquetage == COORD: - lines[lineno].append(ws.make_cell(etud[1][1], self.styles["2m2"])) - lines[lineno].append(ws.make_cell(etud[1][0], self.styles["2r"])) + lines[lineno].append( + worksheet.make_cell(etud[1][1], self.styles["2m2"]) + ) + lines[lineno].append(worksheet.make_cell(etud[1][0], self.styles["2r"])) else: - lines[lineno].append(ws.make_cell(etud[1], self.styles["2r"])) + lines[lineno].append(worksheet.make_cell(etud[1], self.styles["2r"])) lineno = lineno + 1 if lineno >= maxlines: # fin de liste col = col + 1 lineno = 0 if col >= maxlistes: # fin de page for line_cells in lines: - ws.append_row(line_cells) + worksheet.append_row(line_cells) lines = [[] for _ in range(maxlines)] col = 0 - ws.append_blank_row() + worksheet.append_blank_row() nb_etu_restant -= maxlistes * maxlines nb_listes = min( maxlistes, nb_etu_restant // maxlines + 1 ) # nombre de colonnes dans la page - self._headers(ws, nb_listes) + self._headers(worksheet, nb_listes) for line_cells in lines: - ws.append_row(line_cells) + worksheet.append_row(line_cells) def _excel_feuille_placement(self): """Genere feuille excel pour placement des etudiants. @@ -586,8 +604,8 @@ class PlacementRunner: workbook = ScoExcelBook() - SheetName0 = "Emargement" - ws0 = workbook.create_sheet(SheetName0) + sheet_name_0 = "Emargement" + ws0 = workbook.create_sheet(sheet_name_0) # ajuste largeurs colonnes (unite inconnue, empirique) width = 4500 * column_width_ratio if nb_rangs > 5: @@ -599,8 +617,8 @@ class PlacementRunner: "ABCDEFGHIJKLMNOPQRSTUVWXYZ"[col + 1 : col + 2], width ) - SheetName1 = "Positions" - ws1 = workbook.create_sheet(SheetName1) + sheet_name_1 = "Positions" + ws1 = workbook.create_sheet(sheet_name_1) self._make_styles(ws0, ws1) self._feuille0(ws0, space) diff --git a/app/templates/scodoc/forms/placement.html b/app/templates/scodoc/forms/placement.html index 7f169cd3..f2c11b15 100644 --- a/app/templates/scodoc/forms/placement.html +++ b/app/templates/scodoc/forms/placement.html @@ -32,7 +32,7 @@ {% for partition in form.groups_tree %} {% if partition == 'Tous' %} - Groupes + Groupes {% endif %} {{ partition }}