Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correction du bug de la valeur de retour de getIdealFontSize() null --> 0 🐛 #19

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

JV-conseil
Copy link

@JV-conseil JV-conseil commented Mar 23, 2021

La valeur de retour de la fonction getIdealFontSize() doit ĂȘtre de type numĂ©rique, Ă  dĂ©faut 0 mais pas null !

function getIdealFontSize (font, text, maxWidth, minSize, defaultSize) {
let currentSize = defaultSize
let textWidth = font.widthOfTextAtSize(text, defaultSize)
while (textWidth > maxWidth && currentSize > minSize) {
textWidth = font.widthOfTextAtSize(text, --currentSize)
}
return textWidth > maxWidth ? null : currentSize
}

Sinon la crĂ©ation du PDF est buggĂ©e et l'attestation n'est pas gĂ©nĂ©rĂ©e đŸš«

« Uncaught (in promise) TypeError: options.size must be of type number or n, but was actually of type null ».

Issue 👉 #20

La valeur de retour de la fonction getIdealFontSize() doit ĂȘtre de type numĂ©rique, Ă  dĂ©faut 0 mais pas `null` ! Sinon la crĂ©ation du PDF est buggĂ©e: « Uncaught (in promise) TypeError: `options.size` must be of type `number` or `n`, but was actually of type `null` ».
@JV-conseil JV-conseil changed the title Correction du bug de la valeur de retour de getIdealFontSize null --> 0 🐛 Correction du bug de la valeur de retour de getIdealFontSize() null --> 0 🐛 Mar 24, 2021
@JV-conseil
Copy link
Author

#!/usr/bin/env python3.8
# -*- coding: UTF-8 -*-

"""Simple test en langage Python pour connaĂźtre le nombre de villes
dont l'intitulé compte un nombre de caractÚres supérieur à 19.

Base officielle des codes postaux fournie par data.gouv.fr (28 mars 2017 10:15) :
https://www.data.gouv.fr/fr/datasets/base-officielle-des-codes-postaux/#resource-554590ab-ae62-40ac-8353-ee75162c05ee
"""

import logging
import sys

__author__ = "JV conseil – Internet Consulting"
__copyright__ = "Copyright (c) 2021, JV conseil – Internet Consulting, All rights reserved."
__credits__ = ["JV-conseil"]
__license__ = "BSD 3-Clause License"
__version__ = "1.0.0"
__maintainer__ = "JV-conseil"
__email__ = "contact@jv-conseil.net"
__status__ = "Development"

try:
    import pandas as pd
except ImportError:
    sys.exit("""You need pandas!
                install it from http://pypi.python.org/pypi/pandas
                or run pip install pandas""")


# Logs
logger = logging.getLogger(__name__)
logger.setLevel(logging.DEBUG)


data_csv_url = 'https://www.data.gouv.fr/fr/datasets/r/554590ab-ae62-40ac-8353-ee75162c05ee'

try:

    df = pd.read_csv(data_csv_url, sep=';', na_filter=False).to_dict(orient='records')

    ville_11 = {d['Code_postal']: (d['Nom_commune'], d) for d in df if len(d['Nom_commune']) > 19}
    ville_11 = list(sorted(ville_11.values(), key=lambda x: x[0]))

    print("### Si votre lieu de résidence et/ou de naissance compte parmi les {} villes listées ci-dessous l'[Attestation de déplacement dérogatoire numérique](https://media.interieur.gouv.fr/attestation-deplacement-derogatoire-covid-19/) ne fonctionnera pas :".format(len(ville_11)))

    for i, v in enumerate(ville_11, 1):
        print('{}. {} ({})'.format(i, v[0], v[1]['Code_postal']))

except Exception as e:
    logger.exception(e)

@cClaude
Copy link

cClaude commented Apr 9, 2021

@JV-conseil Could you rebase your branch to remove conflicts ?

La valeur de retour de la fonction getIdealFontSize() doit ĂȘtre de type numĂ©rique, Ă  dĂ©faut 0 mais pas `null` ! Sinon la crĂ©ation du PDF est buggĂ©e: « Uncaught (in promise) TypeError: `options.size` must be of type `number` or `n`, but was actually of type `null` ».

npm update --dev --verbose 🔝

Update README.md

Update README.md

Update pdf-util.js

Update package.json
Signed-off-by: JV conseil <contact@jv-conseil.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants