openMairie.org | Démonstration | Documentation | Forum

Prop0050 - Suppression idz

prop-fmwk
prop-fmwk-4-9

#1

Porteur principal : Florent Michon

Porteur secondaire : Sofien Timezouaght

Résumé

Suppression de la gestion du titre par la variable idz

Motivation

Cette variable idz est utilisée pour afficher le titre de la page dans le module form, il n’est donc pas très cohérent d’arriver sur la page avec le titre en paramètre alors que nous sommes sur l’objet en question. L’idée est donc de faire gérer complètement à l’objet ce libellé.

Pré-requis

  • Aucun

Proposition et implémentation

  • On supprime les éléments dans le gen du gen/sql/pgsql/*.inc.php qui n’apportent rien à la génération (les deux conditions qui concatènent des éléments à $ent ne possèdent aucun élément spécifique à la table générée)
  • On supprime dans om_table l’ajout du contenu de la seconde colonne au lien d’accès à la vue du form sur l’élément
  • On remplace les deux éléments supprimés par la génération d’une méthode dans gen/obj/*.class.php get_default_libelle() qui elle contient le champ représentant le libellé de l’enregistrement
  • On appelle la méthode get_default_libelle() dans les méthodes getFormTitle et getSubFormTitle de om_dbform
  • On supprime toutes les traces de idz

Branche : http://adullact.net/scm/viewvc.php/openmairie/openmairie_exemple/branches/4.9.0-develop-prop0050-remove-idz/?pathrev=4196
Commit fonctionnel :

Risques

  • les risques sont limités puisque la méthode dbform::getFormTitle() permettait déjà de bypasser la variable idz.
  • le strtoupper php sur idz a été supprimé, il est possible de le repositionner en css avec : #title .libelle { text-transform: uppercase; }

#2

Le développement est conforme aux spécifications, OK pour le merge vers la branche d’intégration.
Test réalisé sur la branche de développement 4.7.0 de l’application openADS.

→ Rebase de la branche de développement 4.9.0-develop-prop0050-remove-idz depuis la branche d’intégration 4.9.0-develop
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4222

→ Ajout du changelog
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4223

→ Merge de la branche de développement 4.9.0-develop-prop0050-remove-idz vers la branche d’intégration 4.9.0-develop
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4224

→ Suppression de la branche de développement 4.9.0-develop-prop0050-remove-idz
https://adullact.net/scm/viewvc.php/openmairie?view=revision&revision=4225