Conception d'un module

Super_chemist · 18/12/2011 17:49:05

Par contre, j'ai beau chercher, je ne parviens pas à reproduire le problème que tu as avec l'espace qui décale ton image. Tu es sûr que c'est l'espace qui es responsable ?

snypi · 18/12/2011 19:06:57

si tu à la dernière version j'ai virer l'espace
pensant que cela ne dérangeais pas !!
maintenant si tu à la version avec espace !
elle s'insère insidieusement
fait un diaporama avec slide et un flot left sans padding et du texte qui fil jusqu'a en dessous de ton image
tu verras lors du slide ton texte juste en dessous de l'image apparaitre puis disparaitre !!

ps j'ai poster un message sur le forum de tinymce pour notre souci
bon on se moque pas de mon anglais c'est pire que mon francais
http://www.tinymce.com/forum/viewtopic.php?id=27667

Dernière modification par snypi (18/12/2011 19:09:48)

Super_chemist · 18/12/2011 19:27:03

Bug trouvé si on a des fichiers avec des caractères spéciaux.
C'est un oubli de ma part dans zdiaporama, et on le retrouve aussi dans fdiapo !
le nom du document doit être passé à la fonction php rawurlencode() pour le résoudre

zdiaporama v1.2-beta12 http://dl.free.fr/h61AD1yod

Super_chemist · 18/12/2011 19:52:15

Pour le bug de l'espace, je crois que j'ai trouvé un moyen de le résoudre.

On a <div> <?php?></div> (donc tel que fonctionne zdiaporama) et il faut rajouter un font-size:0; à la liste des styles de la div.

De plus, j'ai vérifié, le validateur w3c dit que c'est valide. Un espace, c'est pas considéré comme du texte car tu peux t'en servir pour aérer le code (comme une tabulation ou un retour à la ligne)

EDIT : D'ailleurs, j'ai le même sous zdiaporama ! Allez hop la beta13 http://dl.free.fr/kegk2aTng

Dernière modification par Super_chemist (18/12/2011 20:07:12)

snypi · 19/12/2011 11:39:08

Super_chemist a écrit :

Bug trouvé si on a des fichiers avec des caractères spéciaux.
C'est un oubli de ma part dans zdiaporama, et on le retrouve aussi dans fdiapo !
le nom du document doit être passé à la fonction php rawurlencode() pour le résoudre
zdiaporama v1.2-beta12 http://dl.free.fr/h61AD1yod

salutation
peut tu me dire ou ce situe ce bug j'ai comparer nos fichiers et j'ai pas trouver ta modif !!
merki

snypi · 20/12/2011 10:46:52

Super_chemist a écrit :

Si tu regarde dans mon zdiaporama.php, tu verra que dans la méthode function dojob() je déclare dés la 2e ligne une variable static $zdiapo.
Le principe d'une variable statique, c'est qu'elle est accessible par toutes les instances de l'objet en question d'une même page.
En clair, tu peux faire un test
		if (!isset($fdiapo)) 
		{
			//code à exécuter la première fois
		}else{
			$fdiapo = 1;
			//code à exécuter les fois suivantes
		}
Attention, pour ne pas qu'on se marche dessus, il faut que ta variable static n'ait pas le même nom que celles des autres modules (en particulier zdiaporama), donc met bien fdiapo !
Ensuite, tu peux t'inspirer de ce que j'ai fait, à savoir que ma $zdiapo est un array, ce qui m'a permis de gérer intelligement, en incrémentant, le cas de l'identificateur unique...

Sinon, rien à voir, mais dans la 092 il n'y a pas une histoire d'intégration automatique de jquery ? Faudrait voir avec jpg comment intégrer ça pour que ce soit propre...
Je télécharge ton module, pour voir un peu ce que tu as fait, et pour voir dans quelle mesure il y a conflit. Mais en gros, si toutes les variables sont bien préfixées, il devrait pas y avoir de soucis...

j'ai testé mais cela ne fonctionne pas ?
le nom de ma variable

 static $fad_diapo= array();

le code tenté

            if (!isset($fad_diapo)) 
        {
            //code à exécuter la première fois
            self::$cms->content['js'].='
<script src="js/jquery.1.7.1.min.js" type="text/javascript"></script>
<script src="js/jquery.innerfade.js" type="text/javascript"></script>';
        }else{
            $fad_diapo = 1;
            //code à exécuter les fois suivantes

           self::$cms->content['js'].=' 
<script type="text/javascript">
$(document).ready( function () {
$(".'.$rub.'_fdiapo").innerfade({';
            if (isset($param['t']))
            {
				if($param['t']>0)
                {
                    self::$cms->content['js'].='timeout:'.$param['t'].','; //etc ...

dans mon header je ne retrouve que la partit "else"
une idée les super codeur ?

Dernière modification par snypi (20/12/2011 10:47:40)

Super_chemist · 20/12/2011 11:23:11

Pour le bug des caractères spéciaux, ça se passe dans zite/module/fdiapo.php, ligne 207 dans le foreach{}.
Remplace juste $liste[] = $docname; par $liste[] = rawurlencode($docname); et ça devrai être bon.
Vérifie mieux, c'est comme ça dans zdiaporama maintenant !

Pour la variable statique, c'est normal que tu n'ai que le else qui s'exécute. Tu vérifies que ta variable n'est pas définie (!isset) or tu l'a définie juste au dessus en disant que c'est un array...
Moi c'est juste le principe que je t'ai présenté ! Après il faut adapter...

Dans zdiaporama, je fais (!isset($zdiapo[$rub])) c'est à dire que je teste la case [$rub] qui va être ou non définie selon que c'est la 1re fois ou non que j'appelle cette rubrique.

Il vaudrait mieux que tu ai deux variables de ce type (attention : comprend d'abord, et ADAPTE ensuite, je ne me suis pas plongé dans ton code!)

static $fdiapo = 0;
static $fdiapo_array = array();

//Ensuite tu peux exécuter un code UNE seule fois avec
if ($fdiapo == 0)
{
    $fdiapo = 1;
    //Code unique//
}else{
    $fdiapo++; //facultatif bien entendu
    //Code exécuté les fois suivantes//
}

//Et tu gardes la possibilité d'avoir un code exécuté une fois PAR RUBRIQUE
if(!isset($fdiapo_array[$rub]))
{
    $fdiapo_array[$rub] = 1;
}else{
    $fdiapo_array[$rub]++;
}

Une dernière remarque, vu que ton module se nomme fdiapo, tu devrais utiliser une variable statique $fdiapo et non $fad_diapo... Reste cohérent, si un jour quelqu'un sort un module fad_diapo, ça risque de faire conflit !

EDIT: je viens de vérifier dans la doc php, apparemment une variable statique dans une fonction membre n'est accessible que dans cette fonction, au sein de sa classe.
En clair, une static $count=0; dans zdiaporama ne devrait pas interférer avec une static $count=0; dans fdiapo.
Juste pour info...

Dernière modification par Super_chemist (20/12/2011 12:27:25)

Super_chemist · 20/12/2011 12:35:01

Au fait, tu me demandais l'intérêt de pouvoir appeler deux fois la même rubrique. Deux réponses:

Primo, tu ne sais pas de quel genre de conneries sont capables tes utilisateurs. Ne jamais les sous-estimer ! :-) "Mais si, je suis sûr que ça ferait super beau si on fait un damier avec 9 fois les mêmes images..."

Deuzio, imagine que tu ais un petit diapo en 100x100 sous ton menu à droite, joli et visible sur toutes tes pages pour décorer. Maintenant, tu veux créer une page avec un grand diaporama pour voir les images en grand. Evidemment, c'est le même diaporama, sinon c'est pas drôle!

Il vaux mieux prévenir que guérir!

Dernière modification par Super_chemist (20/12/2011 12:36:56)

snypi · 20/12/2011 14:40:47

un grand merci ton soutien et ta pédagogie j'adore cela !!
avec Jpg ça fait une ziteplus + sur ce forum
je me plonge dedans dès que j'ai un peu de dispo
ps pour les toqués qui veulent plusieurs même diaporama ils ont qu'a faire plusieurs rubrique
1 avec des miniatures et l'autres avec les grandes images NA !!

Dernière modification par snypi (20/12/2011 14:42:31)

snypi · 20/12/2011 17:12:57

merci cela fonctionne bien du coup je vais proposé deux versions une avec jquery d'intégrer l'autre sans le jquery pour ceux qui on déja configurer jquery dans leur template ou dans le menu configurer !!

Super_chemist · 20/12/2011 17:20:14

Et pourquoi pas utiliser les options pour configurer le module? En passant par le fichier.opt intégré à ziteplus?

snypi · 20/12/2011 17:55:57

y'a les deux
et lorsque tu utilise plusieurs version de jquery (j'ai déja vue des soucie avec cela) il faut toujours charger le jquery du plus vieux au plus récent
donc on peut souhaitez configurer soit même le jquery via le fichier template ou configurer de ziteplus !!

Super_chemist · 20/12/2011 18:59:37

Bug trouvé, si on ne renseigne rien dans le dialog (donc tout par défaut). Le script plante et ne s'exécute pas...
Tu devrais revérifier ce qui se passe, pour chaque item, si la case est vide, si elle contient zéro... pour éviter d'insérer n'importe quoi dans le script ou le html.

Pour la vérif du dialog (fichier dialog.js si je me souviens bien), tu n'a pas de vérification du champ "transition". Il faudra adapter !

Si les images n'ont pas un format standard, elles sont étirées... Dans zdiaporama pour éviter ça j'étais passé par un max-height/max-width pour la img, et height/width pour la div.

Les images sont cliquables, mais ne renvoient sur rien... C'est vraiment nécessaire de mettre un lien dans ce cas? Si oui, tu peux peut-être au moins mettre un lien vers view.php/$docname...

Tu es bien téméraire de lancer ton module comme ça, sans un minimum de beta/debug avant ! Heureusement qu'il n'y a personne sur ce forum !

EDIT : et ça mélange un peu tout si on met deux fois la même rubrique avec des valeurs de temps différentes ! J'y tiens !

Dernière modification par Super_chemist (20/12/2011 19:15:31)

jpg · 21/12/2011 00:51:52

C'est vraiment utile d'utiliser deux versions de jquery avec ziteplus 0.9.2 ?
Car depuis qu'on peut préciser dans la config la version souhaité de jquery cela devrait suffire.
Pour l'instant les composants intégrés à ziteplus qui utilisent jquery ont juste besoin d'une version > à la 1.3.2
Ils ont l'air de bien marcher avec les versions que j'ai testé: 1.6 et 1.7
Cela devrait être plus ou moins la même chose avec vos composants

a+
Jean-Paul

snypi · 21/12/2011 01:32:55

dans ziteplus jquery via le menu configurer est que c'est pas évident à comprendre !!
super chemist et moi même on s'est posé la question ! comment cela fonctionne ?
bon c'est pas compliquer mais j'ai mis du temps à me penchez dessus alors je me dit un novice !!!
c'est pour cela que je voulais implémenté le jquery via le module
deplus j'ai déja eu des bug de compatibilité de script (sur le site du centre équestre j'ai mis tous mes script dans template.php avant le <?php echo $this->head(); ?>
du coup l'appel de jquery plantait !!)
il faut faire attention à l'ordre d'apparition des scripts car cela peut être une source de bug !! (ok il faut rien mettre dans le template!! mais bon !!)

super_chemist a écrit :

Si les images n'ont pas un format standard, elles sont étirées

effectivement je souhaite que les images soit à la bonne taille et pas à l'arrache car cela n'est pas très esthétique d'avoir des images de taille différente dans un diaporama
donc à l'utilisateur d'apprendre à optimiser ses images (c'est mon point de vue cela ce discute)

je vais me penchez sur le fait que l'on peut intégrer deux fois le mêmes diaporama et gérer les champs vides à mon retour de vacances
merci pour vos retours

Super_chemist · 21/12/2011 11:01:19

Je n'avais pas réalisé que jQuery était installé de base dans ziteplus...

@snypi : Si on installe ton fdiapo sans jQuery, et qu'on laisse la case jquery_link vide dans la config de ziteplus, le script fonctionne quand même... avec le jquery1.3 intégré à ziteplus.
Teste sur un ziteplus tout frais, mais je crois que ce n'est pas nécessaire d'intégrer jquery1.7 dans le module, ni de l'appeler par php. Juste un appel du script innerfade, ça suffit !

@snypi too: concernant la taille des images, je maintient mon point de vue ! Je fais partie de cette catégorie de personnes qui préfèrent avoir des bandes noires autour de l'image d'un film, plutôt que d'avoir l'image étirée...
Quant à compter sur tes user pour retoucher les images, comment dire... Quand ils verront que ton site flingue leurs diapos, quand ils en auront marre et qu'ils voudront refaire faire leur site... ben ils iront pas chez toi !
Si ils te demandent d'installer leur site web, c'est parce qu'ils ne savent/veulent pas installer un cms. Ils ne sont pas informaticiens, alors leur demander d'installer gimp pour retoucher la taille de leur image... J'ai essayé, une seule fois ! lol

Avoir un espace, même de taille variable selon l'image affichée, sera à mon sens toujours plus esthétique qu'un portrait étiré en paysage... Combien de tes utilisateurs connaissent les dimensions de leurs images? Toi même, tu connais la taille des images qui sortent de ton appareil photo?
Et avec un fond de couleur, une bordure, un padding, ça passe ! Mais après, c'est vrai, il faut faire un joli css, une fois pour toutes parce que si le module est bien fait, il te sort systématiquement un class=fdiapo intégrable dans le css de ton site !
Et ça nempêche pas l'utilisateur, si il le souhaite, de faire un effort pour retoucher ses images...

Bon, bonnes vacances !

Super_chemist · 21/12/2011 11:40:28

@jpg : Est-ce prévu pour un module d'avoir une page d'aide ? Un lien [?] sur la page gestion des modules, quelquechose comme ça...

Adrien

snypi · 21/12/2011 13:07:38

oui je pense que cela est une bonne idée d'avoir un bouton sur la gauche lors de l'intégration du module (comme le bouton configurer ou l'on rajoute une page pour décrire le module et sont utilisation)

super_chemist a écrit :

Toi même, tu connais la taille des images qui sortent de ton appareil photo?

justement il est préférable de les confronté ce problème car j'ai constaté que beaucoup de personne n'optimisait pas leur photo avant la publication sur le web !!!
et avec paint.net cela est assez facile à former !
mais j'ai réussi à faire en sorte que les images est un max width et max height (pour mes zites à moi il garderons une image redimensionné)
par contre le fait de mettre deux même diaporama mazette !!!
comment je peut faire pour que tinymce me rajoute la bonne classe à ma div
code de fdiapo.php

$(".'.$rub.$fdiapo_array[$rub].'_fdiapo").innerfade({';

alors la tu me pause une grosse cole

Super_chemist · 21/12/2011 14:33:22

Tinymce ne peux malheureusement pas savoir combien de diapo il y a...
Dans zdiaporama je n'ai pas ce problème car je sélectionne l'img (qui lui a son class défini par le php).

Dans module/fdiapo.php, quand tu insères ton js, tu dois garder $(".'.$rub.'_fdiapo").innerfade({' et tu dois te débrouiller pour ajouter un 'children: ".'.$rub.$fdiapo_array[$rub].'_fdiapo"' dans la liste des paramètres, donc vers la ligne 300.
Ensuite, dans le $out ligne 306, tu ajoutes une class="'.$rub.$fdiapo_array[$rub].'_fdiapo"' dans la balise <a> (ou dans la img, je sais pas, essaye).

Pas testé, mais si on regarde le code de innerfade, le children doit servir à ça !

Au fait, j'ai trouvé un bug dans zdiapo, et dans fdiapo par voie de fait : fdiapo.js, ligne 86 : il faut un double signe égal (nl.fl.value == 'none').

Super_chemist · 21/12/2011 15:13:49

@jpg : Est-ce prévu pour un module d'avoir une page d'aide ? Un lien [?] sur la page gestion des modules, quelquechose comme ça...

Autre question, peux-on dans la fonction On() faire en sorte que les droits soient automatiquement mis comme il faut ?

Adrien

EDIT: v1.2 beta14, qui corrige le bug au dessus. J'en ai profité pour modifier les class créées automatiquement, faire quelque chose d'un peu plus propre, et aussi préfixer les variables js un peu plus proprement.
http://dl.free.fr/j6o5091b2

Dernière modification par Super_chemist (21/12/2011 15:17:26)

snypi · 16/01/2012 22:06:05

tien j'ai un autre bug découvert sous fdiapo qui doit avoir aussi sous zdiaporama
je vient d'insérer une bannière avec fdiapo dans mon top
dans celui ci j'ai aussi le php de xmenu
quand j'enregistre mon fdiapo tinymce m'éfface mon code php de xmenu

ce n'est pas un bug de nos modules mais de tinymce à mon avis
qu'on ce le dise

Dernière modification par snypi (16/01/2012 22:06:42)

Super_chemist · 16/01/2012 22:34:01

En effet, tinymce fait un peu n'importe quoi avec les balises <?php?> car ce ne sont pas des balises xhtml classiques.
C'est d'ailleurs assez risqué d'éditer une page avec du code php avec tinymce...

Un conseil, quand tu insères un truc en php, arrange toi pour que ce code soit seul dans son panneau. Comme ça, tu peux modifier ce qu'il y a avant et ce qu'il y a après sans prendre le risque que tinymce ne se mélange les pinceaux.

Stefpei · 17/01/2012 13:03:06

mon "rève" serai d'avoir un paramètre supplémentaire dans le réglage des pages :
--> pouvoir interdire d'éditer certaines (celles qui ont du PHP) --> à part en mode "source", mais pas en "wiziwig"...
Il n'y-aurait plus à craindre un Webmaster inexpérimenté.

Dernière modification par Stefpei (17/01/2012 13:04:15)

Super_chemist · 17/01/2012 14:44:12

Idée intéressante !
Sinon, on peux déjà jouer avec les droits d'éditer et les logins "admin" et "webmaster", mais c'est un peu long quand il faut modifier toutes les pages une par une...

snypi · 17/01/2012 16:50:32

perso je ne trouve pas cela très top
le php sert aussi à intégrer les modules de ziteplus
cela veut dire que si je met un diaporama la page n'est plus éditable (bof)
sinon le top serait que les modules implémente directement le code html et plus du php !
mais la c'est une toute autre histoire (maintenant cela serait aussi plus sécure)
++

Forum ZitePLUS

#151 18/12/2011 17:49:05

Re : Conception d'un module

#152 18/12/2011 19:06:57

Re : Conception d'un module

#153 18/12/2011 19:27:03

Re : Conception d'un module

#154 18/12/2011 19:52:15

Re : Conception d'un module

#155 19/12/2011 11:39:08

Re : Conception d'un module

#156 20/12/2011 10:46:52

Re : Conception d'un module

#157 20/12/2011 11:23:11

Re : Conception d'un module

#158 20/12/2011 12:35:01

Re : Conception d'un module

#159 20/12/2011 14:40:47

Re : Conception d'un module

#160 20/12/2011 17:12:57

Re : Conception d'un module

#161 20/12/2011 17:20:14

Re : Conception d'un module

#162 20/12/2011 17:55:57

Re : Conception d'un module

#163 20/12/2011 18:59:37

Re : Conception d'un module

#164 21/12/2011 00:51:52

Re : Conception d'un module

#165 21/12/2011 01:32:55

Re : Conception d'un module

#166 21/12/2011 11:01:19

Re : Conception d'un module

#167 21/12/2011 11:40:28

Re : Conception d'un module

#168 21/12/2011 13:07:38

Re : Conception d'un module

#169 21/12/2011 14:33:22

Re : Conception d'un module

#170 21/12/2011 15:13:49

Re : Conception d'un module

#171 16/01/2012 22:06:05

Re : Conception d'un module

#172 16/01/2012 22:34:01

Re : Conception d'un module

#173 17/01/2012 13:03:06

Re : Conception d'un module

#174 17/01/2012 14:44:12

Re : Conception d'un module

#175 17/01/2012 16:50:32

Re : Conception d'un module

Pied de page des forums