openMairie.org | Démonstration | Documentation | Forum

[intégrée] Prop0041 - Suppression des fichiers sql/pgsql/*.form.inc.php

Statut : intégrée dans la 4.8.0

Porteur principal : Florent Michon

Porteur secondaire : Sofien Timezouaght

Résumé

Les scripts sql/<OM_DB_PHPTYPE>/*.form.inc.php ne sont plus générés par le générateur. Les variables que ces scripts définissaient sont désormais générées dans des méthodes de la classe métier. Si les scripts existent, alors ce sont en priorité les variables de ces scripts qui sont récupérées.

Motivation

  • Améliorer la lisibilité du code en gérant tous les éléments de notre objet dans la classe de cette objet.
  • Contrer la problématique de consommation mémoire lors de traitements nécessitant l’instanciation de chaque objet d’une table. Cette instanciation répétée chargeait en mémoire les scripts .inc.php à chaque instanciation et plusieurs fois par instanciation. En PHP il n’y a pas de moyen vider de la mémoire des fichiers inclus.

Pré-requis

RAS

Proposition et implémentation

Les scripts sql/<OM_DB_PHPTYPE>/*.form.inc.php ne sont plus générés par le générateur.

Les variables que ces scripts définissaient sont désormais générées dans des méthodes de la classe métier :

  • dbform::get_var_sql_forminc__tableSelect() # $tableSelect
  • dbform::get_var_sql_forminc__champs() # $champs
  • dbform::get_var_sql_forminc__selection() # $selection
  • dbform::get_var_sql_forminc__sql_*() # $sql_*

Si les scripts existent, alors ce sont en priorité les variables de ces scripts qui sont récupérées. Il est proposé de conserver le fonctionnement actuel pour continuer à proposer la possibilité d’avoir des spécificités en fonction du SGBD utilisé.

Branche :

Modification fonctionnelle :

Risques

Le développement est conforme aux spécifications, OK pour le merge vers la branche d’intégration.

→ Rebase depuis la branche d’intégration 4.8.0-develop
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4128

→ Merge dans la branche d’intégration 4.8.0-develop
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4129

→ Suppression de la branche de développement prop0041-remove_forminc_files
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4130