developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 11 years ago

Closed 10 years ago

#1263 closed enhancement (fixed)

jDao: Ajout d'une méthode getTable dans le Dao

Reported by: isepman Owned by: isepman
Priority: normal Milestone: Jelix 1.3.2
Component: jelix:dao Version: trunk
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Bonjour, Il existe une méthode getProperties de l'objet jDaoFactoryBase mais il n'existe pas de getter pour les tables. Or cela pourrait être pratique. J'en ai besoin pour les evenements automatiques Dao:

J'ai besoin d'enregistrer des modification faites dans certaines tables, mais pas toutes. Ces tables peuvent êtres modifiées par plusieurs Daos avec les evenements insertBefore, updateBefore...etc. J'ai d'autre part un fichier de config qui contient les tables que je veux écouter avec des paramètres de configuration. Il est fastidieux de recopier ces paramètres par Dao,

(exemple :

module-dao1 = "paramètres"

module-dao2 = "paramètres"

)

je préfererai le faire par table...

table = "paramètres"


Je propose donc 2 méthodes, a rajouter dans jDaoFactoryBase

(Ca, c'est de la contribution!)


    public function getTables() {
    	return $this->_tables;
    }
    public function getPrimaryTable() {
    	return $this->_primaryTable;
    }

Sinon, pour l'instant, ma façon de récupérer les tables est assez moche:

    $fact = jDao::get($eventDao,$profile);
    $id = $fact->getPrimaryKeyNames();
    $props = $fact->getProperties();
    $table = $props[$id[0]]['table'];

Merci pour votre réponse,

Est ce que ça vaut le coup que j'envoie des diff et tests unitaires?

Change History (4)

comment:1 Changed 11 years ago by laurentj

  • Milestone changed from Jelix 1.1.8 to Jelix 1.3
  • Severity changed from normal to minor

comment:2 Changed 10 years ago by laurentj

  • Milestone Jelix 1.3 deleted

comment:3 Changed 10 years ago by laurentj

  • Owner set to isepman
  • Status changed from new to reviewing

comment:4 Changed 10 years ago by laurentj

  • Milestone set to Jelix 1.3.2
  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.