MOdule Climat énergie

[laurent00]

voici un module que j'ai bricolé
son but est de calculer simplement les étiquettes énergie et climat d'une maison d'habitation
mis à jour dans sa version J!3

http://lj01.cmoi.cc/telechargement/mod_DPEmaison_j35.zip

ou testable ici
merci de vos avis

[laurent00]

Re : MOdule Climat énergie

trois modif que je voudrais faire mais ne sais pas comment

- lorsqu'on arrive sur la page si on a pas cliqué sur CALCULER l'onglet "Résultat" n'affiche rien, je voudrais qu'il affiche un message "saisir les données avant" et que ce message disparaisse si on a calculé ...

- lorsque l'on fait CALCULER je souhaiterais que l'onglet résultat soit actif dès que le calcul est fini ...

- et comment faire pour que l'utilisateur choisisse un modèle d'impression différent ... actuellement il faut le faire en backend ...

merci

[laurent00]

Re : MOdule Climat énergie

pas d'idées ... !?

[jisse03]

Re : MOdule Climat énergie

Bonjour,

Vu la qualité de programmation du module, je reste très réservé.

Ton lien vers la page de résultats n'embarque pas les données correctement, donc il est logique que la confirmation soit demandée.

Pour que l'onglet résultat soit actif, il te faut utiliser des requêtes Ajax/JSON.

Et pour les modèles d'impression, il faut activer un select en frontal, et router vers la bonne méthode, encore de l'Ajax en vue.

[laurent00]

Re : MOdule Climat énergie

évidemment je suis un bricoleur pas un développeur ...
qu'est-ce qui fait que ma programmation est légère, peut être pourrais-je commencer par améliorer cela ...

LJ

[jisse03]

Re : MOdule Climat énergie

Commences par tes printpdf. Pourquoi les avoir doublés par langue ? En utilisant directement les ini de langue, il n'en faudrait qu'une seule version et pas une par langue.

$surface = trim($_POST['surface'])

par exemple, est risqué, tu utilises directement une valeur POST, sans aucun filtrage, ce qui pourrait être une faille de sécurité. Utilises JParameter( uniquement 2.5) ou JForm ou JInput (2.5 et 3.x) qui permettent de trquer le typage de données attendues et nettoient toute tentative d'injection SQL ou LFI.

L'utilisation d'une classe type JParameter ou JInput te permet aussi d'éviter les global et de maintenir un code plus propre.

Dans tes PDF, l'image est également codée en dur, vers ton site, au lieu d'être un paramètre.

[laurent00]

Re : MOdule Climat énergie

merci j’intègre ces éléments

pour les injections sql je pensais que cela n'aurait pas d'effet car rien ne s’enregistre en base de données ...
LJ

[jisse03]

Re : MOdule Climat énergie

Par contre une faille LFI, ça n'a pas besoin de SQL...

[laurent00]

Re : MOdule Climat énergie

comment appeler mes Class Joomla Jtext Jinput dans mon fichier pdf ?
LJ

[jisse03]

Re : MOdule Climat énergie

Ta classe pdf tournant depuis un environnement Joomla!, et puisque tu as un fichier langue fr-FR.mod_DPEmaison.ini, par exemple en prenant element/0_printpdf.php

// ligne 46
$str =" Votre étiquette ENERGIE CLIMAT ";
// à remplacer par
$str = Jtext::_( 'MOD_DPE_MAISON_YOUR_ENERGY_TITLE' ); 

par exemple et en respectant les préconisations de préfixer les clés de chaînes par le nom de l'extension.
Idem pour toutes les chaînes à traduire.

Tant qu'à faire, profite sen aussi pour mettre d'aplomb les clés de langue déjà définies.

[laurent00]

Re : MOdule Climat énergie

j'ai fait cela mais je tombe sur une erreur
Fatal error: Class 'Jtext' not found

donc ma class pdf ne tourne pas sous joomlac'est une libriairie qui est le fichier fpdf ...
LJ

[jisse03]

Re : MOdule Climat énergie

Importes le framework Joomla! avant ta classe.

[laurent00]

Re : MOdule Climat énergie

çà je sais pas faire !

[jisse03]

Re : MOdule Climat énergie

define('_JEXEC', 1);
define( 'JPATH_BASE',  __DIR__ . '../../' ) ;
require_once JPATH_BASE . '/includes/defines.php' ;
require_once JPATH_BASE . '/includes/framework.php' ;
$app = JFactory::getApplication('site');
$app->initialise(); 

[laurent00]

Re : MOdule Climat énergie

oui j'ai aussi tester mais le JPATH BASE me renvoi dans les cordes une url jusque dans mon module du coup il ne trouve pas le framework ... j'ai aussi testé cela

define( '_JEXEC', 1 );
define('JPATH_BASE', dirname(__FILE__) );
define( 'DS', DIRECTORY_SEPARATOR );
/* Required Files */
require_once ( JPATH_BASE .DS.'includes'.DS.'defines.php' );
require_once ( JPATH_BASE .DS.'includes'.DS.'framework.php' ); 

comment lui faire remonter trois répertoires ...
LJ