Projet

Général

Profil

Development #48252

ods : utiliser une regex pour déterminer si un nombre est un nombre

Ajouté par Frédéric Péters il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
03 novembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

On fait

    try:
        float(value)
    except ValueError:
        return False

mais ça passe désormais avec une chaine avec des underscores, "1000_1000" → 10002000. (python3)

Il faudrait sans doute d'abord passer par une regex (ou autre) pour accepter uniquement ce qui a la forme x(.y).


Fichiers

Révisions associées

Révision 9a5459c4 (diff)
Ajouté par Frédéric Péters il y a plus de 3 ans

ods: do not consider _ number separator as valid (#48252)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Sujet changé de utiliser une regex pour déterminer si un nombre est un nombre à ods : utiliser une regex pour déterminer si un nombre est un nombre
#3

Mis à jour par Frédéric Péters il y a plus de 3 ans

#4

Mis à jour par Thomas Noël il y a plus de 3 ans

Moi qui pensait libre un patch sur LibreOffice Calc, c'est raté.

Pour la regex j'aurai proposé « ^\d*\.?d*$ » afin d'autoriser aussi « .5 » et « 5. » qui fonctionnent avec float()

#5

Mis à jour par Frédéric Péters il y a plus de 3 ans

Pour la regex j'aurai proposé « ^\d*\.?d*$ » afin d'autoriser aussi « .5 » et « 5. » qui fonctionnent avec float()

Oui mais pour l'argument, 1_000 ça marche aussi avec float(); je préfère désormais être strict dans l'interprétation.

#6

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée

Comme ça ne concerne que l'export ODS, go.

#7

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 9a5459c43994964cd43493e3da6e158cda9418b3
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Nov 3 17:00:07 2020 +0100

    ods: do not consider _ number separator as valid (#48252)
#8

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF