Ticket #497 (closed new feature: fixed)

Opened 6 months ago

Last modified 4 months ago

jFormsBase : prepareDaoFromControls

Reported by: bastnic Assigned to: bastnic
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:forms Version: 1.0.2
Severity: normal Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 1 Blocking:

Description

La méthode saveToDao de jFormsbase est très pratique, mais on peut vouloir profiter du préremplissage qu'il effectue en fonction d'un formulaire sans pour autant directement vouloir l'insérer en base de données.

Ainsi une nouvelle méthode dans jFormsBase qui retournerait un dao préremplit qui nous permettrait par la suite de pouvoir le modifier et faire nous-même l'insertion. Ca peut-être utile si on a 40 champs qui sont remplis via un formulaire et seulement quelques uns calculés. Ca évite de se taper à la main les 40 champs...

De plus saveToDao pourrait faire appel à cette méthode prepareDaoFromControls et se contenter d'insérer et retourner l'id.

Attachments

patch-497.diff (2.4 kB) - added by bastnic on 04/04/08 03:41:07.
pas taper

Change History

03/12/08 11:24:28 changed by bastnic

  • status changed from new to assigned.
  • owner set to bastnic.

04/04/08 03:40:46 changed by bastnic

  • review set to review?.

Ce n'est vraiment pas un domaine que je maîtrise donc je ne suis pas sûr des conséquences que cela peut avoir. Mais dans l'idée moi j'aime bien la fonctionnalité offerte par ce patch.

On pourrait éviter le compact en créant plusieurs méthodes, mais là c'était juste pour faire simple et montrer l'idée.

Je l'utilise comme ça : 

extract($form->prepareDaoFromControls($this->dao_comment, null, $this->dbProfil), EXTR_PREFIX_ALL, "form");
$form_daorec->id_article = $this->param('id_article');
$form_dao->insert($form_daorec);

Dans mon controller, à la place de l'habituel saveToDao. Je peux ainsi définir le contenu d'autres champs que ceux remplis par formulaire très simplement.

Je suis à peu près sur que c'est une horrible façon de faire donc je demande surtout l'avis de gens bien meilleurs que moi :p. Merci d'avance.

04/04/08 03:41:07 changed by bastnic

  • attachment patch-497.diff added.

pas taper

04/04/08 10:21:08 changed by laurentj

  • review changed from review? to review+.
  • milestone set to Jelix 1.1.

La patch est correct, mais il y a une chose qui me gène, c'est le retour sous forme d'array... Mais je ne vois pas d'autres solutions plus efficace pour l'instant...

04/04/08 13:28:26 changed by bastnic

  • status changed from assigned to closed.
  • resolution set to fixed.

fixed in the trunk.

we can clean it in the future :p.

04/27/08 00:31:13 changed by laurentj

  • docneeded set to 1.
Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed