From 80b90f98a503eb0114fa9ad977ade9b02f935567 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabrice=20PENHO=C3=8BT?= Date: Tue, 26 Oct 2021 18:02:43 +0200 Subject: [PATCH] =?UTF-8?q?Adaptation=20du=20code=20=C3=A0=20la=20nouvelle?= =?UTF-8?q?=20version=20de=20la=20classe=20principale=20du=20module.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/Pagination.ts | 8 ++++---- src/Selector.ts | 2 +- src/SortingField.ts | 2 +- src/exampleWithCSV.ts | 2 +- src/exampleWithHTML.ts | 2 +- src/exampleWithJSON.ts | 2 +- tests/paginationSpec.ts | 20 +++++++++----------- 7 files changed, 18 insertions(+), 20 deletions(-) diff --git a/src/Pagination.ts b/src/Pagination.ts index 5940e37..721c6be 100644 --- a/src/Pagination.ts +++ b/src/Pagination.ts @@ -18,7 +18,7 @@ export class Pagination implements Paginations // L'élément du DOM devant recevoir la liste des pages doit exister constructor(converter: FreeDatas2HTML, pagesElt: DOMElement, pagesName="Pages") { - if(converter.fields === undefined) + if(converter.fields.length === 0) throw new Error(errors.paginationNeedDatas); else { @@ -114,13 +114,13 @@ export class Pagination implements Paginations } // Création du "; this.pages.values=[]; for(let j=1; j <= nbPages; j++) diff --git a/src/Selector.ts b/src/Selector.ts index e43c6b6..10ebe2a 100644 --- a/src/Selector.ts +++ b/src/Selector.ts @@ -17,7 +17,7 @@ export class Selector implements Selectors // Le champ duquel le sélecteur tire ses données doit exister ? constructor(converter: FreeDatas2HTML, datasFieldNb: number, elt: DOMElement, separator?: string) { - if(converter.fields === undefined || converter.datas.length === 0) + if(converter.fields.length === 0 || converter.datas.length === 0) throw new Error(errors.filterNeedDatas); else if(! converter.checkFieldExist(Number(datasFieldNb))) throw new Error(errors.selectorFieldNotFound); diff --git a/src/SortingField.ts b/src/SortingField.ts index 36e3199..ce3125c 100644 --- a/src/SortingField.ts +++ b/src/SortingField.ts @@ -13,7 +13,7 @@ export class SortingField implements SortingFields // Injection de la classe principale, mais uniquement si les noms des champs ont été importés et affichés correctement constructor(converter: FreeDatas2HTML, datasFieldNb: number, fieldsDOMSelector: string="th") { - if(converter.fields === undefined) + if(converter.fields.length === 0) throw new Error(errors.sortingFieldNeedDatas); else if(! converter.checkFieldExist(datasFieldNb)) throw new Error(errors.sortingFieldNotFound); diff --git a/src/exampleWithCSV.ts b/src/exampleWithCSV.ts index 3576cbe..4929d59 100644 --- a/src/exampleWithCSV.ts +++ b/src/exampleWithCSV.ts @@ -56,7 +56,7 @@ const initialise = async () => pagination.options2HTML(); // Affichage initial - converter.datasCounter={ id:"compteur" }; + converter.datasCounterElt={ id:"compteur" }; await converter.run(); // Création d'outils permettant de filtrer les données des champs de données diff --git a/src/exampleWithHTML.ts b/src/exampleWithHTML.ts index e420c5e..43a5969 100644 --- a/src/exampleWithHTML.ts +++ b/src/exampleWithHTML.ts @@ -38,7 +38,7 @@ const initialise = async () => pagination.options2HTML(); // Affichage initial - converter.datasCounter={ id:"compteur" }; + converter.datasCounterElt={ id:"compteur" }; await converter.run(); // Création d'un filtre par auteur : diff --git a/src/exampleWithJSON.ts b/src/exampleWithJSON.ts index 3b3c825..5fa824d 100644 --- a/src/exampleWithJSON.ts +++ b/src/exampleWithJSON.ts @@ -39,7 +39,7 @@ const initialise = async () => pagination.options2HTML(); // Affichage initial - converter.datasCounter={ id:"compteur" }; + converter.datasCounterElt={ id:"compteur" }; await converter.run(); // Création d'un filtre par auteur : diff --git a/tests/paginationSpec.ts b/tests/paginationSpec.ts index 8272a3e..37af67a 100644 --- a/tests/paginationSpec.ts +++ b/tests/paginationSpec.ts @@ -149,42 +149,40 @@ describe("Test de la pagination.", () => it("S'il n'y a pas de pagination définie, le sélecteur de pages ne doit pas être proposé.", () => { pagination.selectedValue=undefined; - pagination.pages2HTML(118); + pagination.pages2HTML(); const selectElement=document.getElementById("pages").innerHTML; expect(selectElement).toEqual(""); }); it("S'il n'y pa plus de données que le nombre de lignes par page, le sélecteur de pages ne doit pas être proposé.", () => { - pagination.pages2HTML(10); + pagination.selectedValue=500; + pagination.pages2HTML(); const selectElement=document.getElementById("pages").innerHTML; expect(selectElement).toEqual(""); }); - it("Si il y a plus de données que le nombre de lignes autorisées par page, un listant les pages doit être affiché.", () => { - pagination.pages2HTML(11); - let selectElement=document.getElementById("pages").innerHTML; - expect(selectElement).toEqual(fixtures.selectorFor2Pages); - pagination.pages2HTML(118); - selectElement=document.getElementById("pages").innerHTML; + pagination.pages2HTML(); + const selectElement=document.getElementById("pages").innerHTML; expect(selectElement).toEqual(fixtures.selectorForManyPages); }); it("Après chaque actualisation de la liste de pages, la valeur sélectionnée est remise à 0.", () => { - pagination.pages2HTML(11); + pagination.pages2HTML(); let selectElement=document.getElementById("freeDatas2HTMLPagesSelector") as HTMLInputElement; expect(pagination.pages.selectedValue).toEqual(1); pagination.pages.selectedValue=2; - pagination.pages2HTML(11); + pagination.pages2HTML(); selectElement=document.getElementById("freeDatas2HTMLPagesSelector") as HTMLInputElement; expect(pagination.pages.selectedValue).toEqual(1); }); it("La manipulation du sélecteur de pages doit enregistrer la valeur sélectionnée et appeler la fonction actualisant l'affichage.", () => { - pagination.pages2HTML(31); + pagination.pages2HTML(); spyOn(converter, "refreshView"); const selectElement=document.getElementById("freeDatas2HTMLPagesSelector") as HTMLInputElement; selectElement.value="4";