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.

Sign up for GitHub

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

fix(import): remove unused config variables #3344

Merged
merged 3 commits into from
Jan 24, 2025
Merged

fix(import): remove unused config variables #3344

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9f10c8b
fix(config): drop some unused config variable in the default example …
jacquesfize Jan 23, 2025
8cb8abe
wip
jacquesfize Jan 23, 2025
128a8ad
fix(doc) add removed doc, remove from config_schema removed config
Pierre-Narcisi Jan 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions backend/geonature/core/imports/checks/errors.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,11 +61,11 @@ class ImportCodeError:
MISSING_GEOM : str
The geometry is missing
GEOMETRY_OUTSIDE : str
The geometry is outside the polygon in the GeoNature configuration (`INSTANCE_BOUNDING_BOX`)
The geometry is outside the polygon defined by ID_AREA_RESTRICTION in the configuration
NO_GEOM : str
No geometry given (wherever WKT or latitude/longitude)
GEOMETRY_OUT_OF_BOX : str
The geometry is outside the perimeter of the instance geography # FIXME: clarify (confusion with GEOMETRY_OUTSIDE)
The geometry is outside of a bounding box
ERRONEOUS_PARENT_ENTITY : str
The parent entity is not valid
NO_PARENT_ENTITY : str
Expand Down
17 changes: 1 addition & 16 deletions backend/geonature/core/imports/config_schema.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,13 +69,6 @@
# If VALUE MAPPING is not allowed, you must specify the DEFAULT_VALUE_MAPPING_ID
DEFAULT_VALUE_MAPPING_ID = 3

INSTANCE_BOUNDING_BOX = [-5.0, 41, 10, 51.15]

ALLOW_FIELD_MAPPING = True
DEFAULT_FIELD_MAPPING_ID = 1
# Parameter to define if the checkbox allowing to change display mode is displayed or not.
DISPLAY_CHECK_BOX_MAPPED_FIELD = True

# Parameter to define the rank shown in the doughnut chart in the import report
# must be in ['regne', 'phylum', 'classe', 'ordre', 'famille', 'sous_famille', 'tribu', 'group1_inpn', 'group2_inpn']
DEFAULT_RANK = "regne"
Expand All @@ -96,20 +89,12 @@ class ImportConfigSchema(Schema):
) # FIXME: unused
FILL_MISSING_NOMENCLATURE_WITH_DEFAULT_VALUE = fields.Boolean(load_default=True)
DISPLAY_MAPPED_VALUES = fields.Boolean(load_default=True) # FIXME: unused
INSTANCE_BOUNDING_BOX = fields.List(
fields.Float, load_default=INSTANCE_BOUNDING_BOX
) # FIXME: unused
ENABLE_BOUNDING_BOX_CHECK = fields.Boolean(load_default=True) # FIXME : unused
# When setting PER_DATASET_UUID_CHECK=True (used for import in synthese):
# - Replace the unicity constraint on unique_id_sinp with an unicity constraint on (unique_id_sinp,id_dataset).
# - Disable per-row dataset import by setting display=False in gn_imports.bib_fields
# for the id_dataset field belonging to synthese destination.
PER_DATASET_UUID_CHECK = fields.Boolean(load_default=False)
ALLOW_FIELD_MAPPING = fields.Boolean(load_default=ALLOW_FIELD_MAPPING) # FIXME: unused
DEFAULT_FIELD_MAPPING_ID = fields.Integer(
load_default=DEFAULT_FIELD_MAPPING_ID
) # FIXME: unused
DISPLAY_CHECK_BOX_MAPPED_FIELD = fields.Boolean(load_default=True)

CHECK_PRIVATE_JDD_BLURING = fields.Boolean(load_default=True)
CHECK_REF_BIBLIO_LITTERATURE = fields.Boolean(load_default=True)
CHECK_EXIST_PROOF = fields.Boolean(load_default=True)
Expand Down
21 changes: 0 additions & 21 deletions config/default_config.toml.example
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -625,21 +625,6 @@ MEDIA_CLEAN_CRONTAB = "0 1 * * *"
# Encodage des fichiers importés autorisées
ENCODAGE = ["UTF-8"]

# Bounding box des données de l'instance.
# Utilisé pour lever des warning lorsque les données sont en dehors.
# Format: [XMIN, YMIN, XMAX, YMAX]
# Par défaut: France métropolitaine incluant la Corse
INSTANCE_BOUNDING_BOX = [-5.0, 41.0, 10.0, 51.15]

# Activer la vérification de l'appartenance des données importées à la bounding
# box définie dans INSTANCE_BOUNDING_BOX
ENABLE_BOUNDING_BOX_CHECK = true

# Activer la vérification des UUID dans les données importées avec ceux présent dans la Synthèse
# Attention : Il est conseillé de désactiver cette option lorsque les données de la synthèse
# sont très volumineuses.
ENABLE_SYNTHESE_UUID_CHECK = true

# Taille maximale du fichier chargé (en Mo)
MAX_FILE_SIZE=500

Expand All @@ -661,12 +646,6 @@ MEDIA_CLEAN_CRONTAB = "0 1 * * *"
# Leve un warning et non une erreur sur les lignes concernées
FILL_MISSING_NOMENCLATURE_WITH_DEFAULT_VALUE = false

# Encodages acceptés
ENCODAGE = ["UTF-8"]

# Afficher le bouton pour afficher/masquer les champs déjà mappés
DISPLAY_CHECK_BOX_MAPPED_FIELD = true

# Active la vérification de l'existence du champs "floutage" si le JDD est privé
CHECK_PRIVATE_JDD_BLURING = true

Expand Down
6 changes: 1 addition & 5 deletions docs/admin/import-admin.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,18 +58,14 @@ Vous pouvez surcoucher ces différents paramètres en les ajoutant directement d
============================================== ============================================================================================================================================================================
ENCODAGE Liste des encodages
acceptés
INSTANCE_BOUNDING_BOX Zone rectangulaire dans laquelle les données doivent être localisées
ENABLE_BOUNDING_BOX_CHECK Activer la vérification de l'appartenance des données importées à la bounding box définie dans INSTANCE_BOUNDING_BOX
ENABLE_SYNTHESE_UUID_CHECK Activer la vérification des UUID dans les données importées avec ceux de la Synthèse
MAX_FILE_SIZE Taille maximale du fichier chargé (en Mo)
SRID SRID autorisés pour les fichiers en entrée
ALLOWED_EXTENSIONS Extensions autorisées (seul le format CSV est accepté actuellement)
ALLOW_VALUE_MAPPING Activer ou non l'étape du mapping des valeurs
DEFAULT_VALUE_MAPPING_ID Si le mapping des valeurs est désactivé, specifier l'identifiant du mapping qui doit être utilisé
FILL_MISSING_NOMENCLATURE_WITH_DEFAULT_VALUE Rempli les valeurs de nomenclature erronées par la valeur par défaut
DISPLAY_CHECK_BOX_MAPPED_FIELD Afficher le bouton pour afficher/masquer les champs déjà mappés
CHECK_PRIVATE_JDD_BLURING Active la vérification de l'existence du champs "floutage" si le JDD est privé
CHECK_REF_BIBLIO_LITTERATURE Active la vérification de la référence bibliographique fournie si la valeur du champs source = "litterature"
CHECK_REF_BIBLIO_LITTERATURE Active la vérification de la référence bibliographique fournie si la valeur du champs source = "litterature"
CHECK_EXIST_PROOF Active la vérification qu'une preuve d'existence est fournie si preuve existence = "oui"
EXPORT_REPORT_PDF_FILENAME Customiser le nom du fichier de rapport de l'import
DEFAULT_RANK Paramètre pour définir le rang utilisé pour le diagramme camembert du rapport d'import.
Expand Down
Loading