Re: [Dolibarr-dev] Canvas

[Régis Houssin]

afin de coller au MVC voila comment on pourrait voit la chose:

M : Model (accès en base)
V : View (template)
C : Controller (contrôle, traitement des données)

on est d'accord que la classe "societe.class.php" est le "Model" du module société et ne doit pas contenir de traitement d'affichage,
dans le cas des canvas, "thirdparty.default.class.php" est le "Model" du canvas qui hérite de société
et "edit.tpl.php" est un des affichages (View) du canvas

il nous manque donc un "Controller",
nous pourrions donc définir un canvas de la sorte :

répertoire societe/canvas/default/
"thirdparty.default.class.php" : le "Model" du canvas (hérite de la classe Societe)
"card.default.class.php" : le "Controller" de la page "soc.php" spécifique a ce canvas (hérite de la classe CardCommon ci-dessous), il sert d'interface entre "soc.php" et le "Model"
"card.edit.tpl.php, card.view.tpl.php, card.create.tpl.php" : les différentes "View" (code html) de la page "soc.php"

répertoire societe/canvas/
"card.common.class.php" : le "Controller" de la page "soc.php" commune aux canvas

soc.php ne serait là que pour appeler les différents "Controller" et afficher le template

ce genre de fonctionnement pourrait s'appliquer à tout type de page (canvas ou non)

Vous en pensez quoi ?


Le 02/09/10 23:37, Laurent Destailleur a écrit :

La je viens de faire des modifs qui ont l'air de marcher pour un mode mixte.

Par defaut le champ canvas est null en table.
Quand on appelle une url avec canvas=xxx alors cela utilise le masque du canvas xxx et cela stocke en base avec canvas=xxx. Ainsi en modif on utilise aussi le canvas=xxx.

Ainsi je pense qu'on a un mode mixte qui permet d'avancer sur l'amélioration du code de la fonction canvas. Les 2 modes sont possibles.
Je te laisse tester et voir si j'ai pas trop cassé.


Le 02/09/2010 23:32, Régis Houssin a écrit :

Oui je sais, mais je comptais bien séparer tout ça, je n'ai pas terminé

-----------------------------------------
Régis Houssin
Tél. +33633020797
http://www.dolibarr.fr
http://www.dolibox.fr

Le 2 sept. 2010 à 22:05, "Laurent Destailleur (Eldy)" <address@hidden> a écrit :

Il y a aussi plein de vilaine choses dans le code des canvas.

Ainsi il n'y a plus de séparation MVC et on trouve du code de présentation dans les classe metiers comme la fonction
ajax_selectThirdPartyType(), assign_values, assign_values, etc... dans la classe métier societe.

Il faudrait sortir tout cela de la classe societe et mettre cela dans le template ou la page qui appelle le template.

-- 
Eldy (Laurent Destailleur).
---------------------------------------------------------------
EMail: address@hidden
Web: http://www.destailleur.fr

Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: address@hidden
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: address@hidden
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net


_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev

_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev

-- 
Laurent.
-----------------------------------------
EMail: address@hidden
Web: http://www.destailleur.fr
Messenger MSN:    address@hidden
Messenger Jabber: eldy
Tel: 0662724322

_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev

Cordialement,
-- 
Régis Houssin
---------------------------------------------------------
Cap-Networks
30, Quai de Verdun
71700 Tournus
FRANCE
VoIP: +33 1 83 62 40 03
GSM: +33 6 33 02 07 97
Web: http://www.cap-networks.com/
Email: address@hidden

Dolibarr developer: address@hidden
Web Portal: http://www.dolibarr.fr/
SaaS offers: http://www.dolibox.fr/
Shop: http://www.dolistore.com/
Development platform: http://www.dolibarr.pro/
---------------------------------------------------------

regis_houssin.vcf
Description: Vcard