Re: [Dolibarr-dev] [Task #7349] Patch + Correction mise en page adresse contact sur PDF

[Eldy]

Raphaël Bertrand (Résultic) a écrit :
> Bonjour,
>
> Désolé d'être forcé de vous faire patcher le patch,
> mais il manque un retour à la ligne dans einstein
> qui aurait du être rajouté.
> Ceci est corrigé dans la présente PJ
Intégré.
> D'autre part une question se pose concernant l'affichage de forme 
> Prénom NOM au lieu de NOM Prénom, car cet affichage étant géré 
> globalement par la fonction getFullName de htdocs/contact.class.php la 
> modification serait d'un impact non négligeable, puisque agissant 
> partout où l'on fait appel à cette fonction (soit assez fréquemment).
Ceci est facile à gérer sans créer d'impact: Il suffit d'ajouter un 3eme 
paramètre dans la fonction qui serait "nameorder" et qui vaudrait 0 par 
défaut si non renseigné.
Si 0, la fonction fait la meme chose que maintenant, si 1 alors on 
affiche nom prenom dans ordre inverse.
Ainsi il y a compatibilité sans impact sur le code (car mode par defaut 
= mode actuel) et la où tu veux un ordre inverse il te suffit d'ajouter 
le 3eme parametre.

> Concernant le débat sur le copyright, c'est moi qui ais choisi de 
> l'indiquer ainsi pour mentionner le cadre dans lequel j'effectuais le 
> travail, car cela me paraissait normal.
> Cependant, si cela pose problème il est tout à fait possible de la 
> modifier selon ce dont vous aurez convenu.
Continue de faire comme tu fais. Nous n'avons jamais eu le cas de 
"stagiaire" travaillant sur Dolibarr, aussi nous devons d'abord savoir 
comment déclarer une ligne copyright dans un tel cas (user seul ou user 
+ nom societe). J'avoue ne pas avoir la réponse mais la question est 
intéressante...
Le débat est toujours ouvert. Au pire on corrigera après coup qd on sera 
fixé.
> Je proposerais prochainement un patch permettant de ne pas afficher 
> dans les PDF le "sélectionnez un pays" parasite qui vient quand 
> aucun pays n'a été affecté pour le destinataire.
La je ne pense pas que ce soit une bonne idée. En effet, le pays est un 
élément "crucial" qui caractérise un tiers car a des impacts sur les 
regles d'affectation par defaut du taut tva par exemple. Il faut donc au 
maximum "forcer" l'utilisateur à renseigner cette information. Et meme 
si on rend lechamp oblitatoire a la creation, il y aura tjs des tiers 
deja saisi donc un rappel incitant à compléter la fiche me semble 
important a chaque fois que possible.
> *Raphaël Bertrand*
> Résultic - Management & Informatique
> Siège : 11 rue Tronchet - 69006 LYON
> Bureaux : 11 pl Maréchal Lyautey - 69006 LYON
> Fixe. 08 74 77 00 70
> Fax. 08 25 24 85 02
> E-Mail : address@hidden
>
> Raphaël Bertrand (Résultic) a écrit :
> > Bonjour,
> >
> > Ci joint un patch cummulatif incluant l'ajout du support de l'ajout 
> > d'un watermark personnalisé sur les brouillons,
> > et la correction du bug de mise en page sur les pdf générés par 
> > einstein, crabe et azur dans le cas de l'utilisation de l'adresse 
> > d'un contact.
> > Ce défaut de mise en page était lié à une non homogénéité dans la 
> > manière d'imprimer les coordonnées dans le cas de l'utilisation d'un 
> > contact ou de sa non utilisation.
> > Il a été corrigé en appliquant au cas d'un contact choisi la méthode 
> > proposée dans le cas d'une non utilisation d'une adresse spécifique, 
> > celui-ci étant fonctionnel sans défaut.
> > (utilisation d'une chaine avec des retours à la ligne pour gérer 
> > l'adresse au lieu de plusieurs chaines distinctes)
> >
> >
> > *Raphaël Bertrand*
> > Résultic - Management & Informatique
> > Siège : 11 rue Tronchet - 69006 LYON
> > Bureaux : 11 pl Maréchal Lyautey - 69006 LYON
> > Fixe. 08 74 77 00 70
> > Fax. 08 25 24 85 02
> > E-Mail : address@hidden
> >
> > Raphaël Bertrand (Résultic) a écrit :
> > > Voici le patch prêt à être publié s'il vous convient.
> > > Il ajoute la fonctionnalité de watermark sur les
> > > proposition, factures, commandes et fiches d'intervention
> > > Après activation par définition d'un texte de watermark dans la 
> > > configuration des modules correspondants.
> > >
> > > L'ajout du watermark se fait de facon autonome au niveau du modèle 
> > > de document, dans la fonction _pagehead.
> > >
> > >
> > > *Raphaël Bertrand*
> > > Résultic - Management & Informatique
> > > Siège : 11 rue Tronchet - 69006 LYON
> > > Bureaux : 11 pl Maréchal Lyautey - 69006 LYON
> > > Fixe. 08 74 77 00 70
> > > Fax. 08 25 24 85 02
> > > E-Mail : address@hidden
> > >
> > > Raphaël Bertrand (Résultic) a écrit :
> > > > Après simplification du code (qui ne fait plus appel à d'autres 
> > > > fonctions que celles déjà présentes),
> > > > et ajout du support du watermark sur les factures en plus des 
> > > > propales,
> > > > voici ci joint le patch que cela donne (généré par CVS+winmerge)
> > > >
> > > > *Raphaël Bertrand*
> > > > Résultic - Management & Informatique
> > > > Siège : 11 rue Tronchet - 69006 LYON
> > > > Bureaux : 11 pl Maréchal Lyautey - 69006 LYON
> > > > Fixe. 08 74 77 00 70
> > > > Fax. 08 25 24 85 02
> > > > E-Mail : address@hidden
> > > >
> > > > Raphaël Bertrand (Résultic) a écrit :
> > > > > Venant juste d'avoir la réponse,
> > > > > et ayant déjà commencé à le faire,
> > > > > je vous transmets ce que j'ai fait pour le moment.
> > > > >
> > > > > Au final, je fais juste appel à une fonction supplémentaire 
> > > > > _rotate dans le module pdf, mais celle-ci peut être mise inline le 
> > > > > cas échéant, n'étant utilisée qu'une seule fois (ou presque).
> > > > >
> > > > > *Raphaël Bertrand*
> > > > > Résultic - Management & Informatique
> > > > > Siège : 11 rue Tronchet - 69006 LYON
> > > > > Bureaux : 11 pl Maréchal Lyautey - 69006 LYON
> > > > > Fixe. 08 74 77 00 70
> > > > > Fax. 08 25 24 85 02
> > > > > E-Mail : address@hidden
> > > > >
> > > > > Eldy a écrit :
> > > > > > Raphaël Bertrand (Résultic) a écrit :
> > > > > > > Bonjour,
> > > > > > >
> > > > > > > J'ai commencé à travailler sur la task #7349,
> > > > > > > en regardant comment je pouvais ajouter le filigrane "Brouillon" 
> > > > > > > sur les propales dans un premier temps.
> > > > > > >
> > > > > > > J'ai trouvé sur le site FPDF un exemple de watermark:
> > > > > > > http://www.fpdf.org/?go=script&id=9
> > > > > > > qui utilise les fonctions de la surcouche PDF_rotate:
> > > > > > > http://www.fpdf.org/?go=script&id=2
> > > > > > >
> > > > > > > Cependant Dolibarr utilise la librairie FPDI, à travers la 
> > > > > > > classe ModelePDFPropales 
> > > > > > > (htdocs\includes\modules\propale\modules_propale.php),
> > > > > > > et le module pdf correspondant au type de template
> > > > > > > (htdocs\includes\modules\propale\pdf_propale_azur.modules.php 
> > > > > > > par exemple)
> > > > > > >
> > > > > > > Je pense que c'est ici qu'il faut que j'ajoute le watermark, 
> > > > > > > dans la fonction _pagehead de chacun de ces squelettes, et 
> > > > > > > concernant les fonctions récupérées, soit les ajouter aux 
> > > > > > > modules pdf sans les mutualiser, soit les mutualiser dans un 
> > > > > > > fichier autre (une lib)
> > > > > > >
> > > > > > > Merci de me dire ce que vous en pensez.
> > > > > > Je suggère de faire beaucoup plus simple. Il n'est pas nécessaire 
> > > > > > de surcharger par une librairie juste pour ce besoin (Dolibarr a 
> > > > > > déjà pas mal de dépendances). On peut très bien intégrer un 
> > > > > > watermark en fond de page en laissant les lettres droites et en 
> > > > > > les indentant en hauteur et largeur comme cela
> > > > > >
> > > > > > B
> > > > > >   R
> > > > > >      O
> > > > > >          U
> > > > > >             I
> > > > > >               L
> > > > > >                  L
> > > > > >                     O
> > > > > >                         N
> > > > > >
> > > > > > Et pour cela tu peux te contenter des fonctions de base de FPDF. 
> > > > > > Le tout est de mettre la marque en couleur et derrière tout autre 
> > > > > > texte.
> > > > >
> > > > > ------------------------------------------------------------------------ 
> > > > >
> > > > >
> > > > > _______________________________________________
> > > > > 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
> > >
> > >
> > > ------------------------------------------------------------------------ 
> > >
> > >
> > > _______________________________________________
> > > 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
> ------------------------------------------------------------------------
>
> _______________________________________________
> Dolibarr-dev mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/dolibarr-dev


--
Laurent Destailleur.
---------------------------------------------------------------
EMail: address@hidden
Web: http://www.destailleur.fr
IM: IRC=Eldy, Jabber=Eldy

AWStats (Author) : http://awstats.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
AWBot (Author) : http://awbot.sourceforge.net
Dolibarr (Contributor) : http://www.dolibarr.org