Le laboratoire de Darathor :: Voir le sujet - [Réglé] Citation multiple

crewstyle

Invité

Posté le : 21 Fév 2006 16:18

Bonjour,
J'étais sur le point de me lancer sur le création de ce MOD lorsqu'une idée me vient à l'esprit ... celle de regarder si ce MOD n'existait pas auparavant !
Et, par surprise, je vois que je tombe directement sur vous. J'apprends avec surprise que vous l'avez grandement développer et qu'il fonctionne très bien

BRAVO lol

Je vois par contre que vous avez un soucis sur l'affichage de la 2ème image :

Citation :

sur Internet Explorer, le bouton d'ajout ne semble pas se remplacer par le bouton de retrait lorsque l'on clique dessus. Cependant il ne s'agit que d'un problème d'affichage, les messages étant tout de même correctement selectionnés et cités.

Eh bien, je voulais juste vous prévenir que j'ai déjà rencontrer ce problème sous IE que j'ai résolu en enlevant tout simplement le tag title dans la balise img de vos BBCodes !!

En clair, jaurais remplacer ceci :

Code :

$quote_img = '<img id="quote_add_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_add'] . '" alt="' . $lang['MuQu_Add_to_quote_list'] . '" title="' . $lang['MuQu_Add_to_quote_list'] . '" onclick="cookiePostAdd(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_add_display . ';" />';

$quote_remove_img = '<img id="quote_remove_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_remove'] . '" alt="' . $lang['MuQu_Remove_from_quote_list'] . '" title="' . $lang['MuQu_Remove_from_quote_list'] . '" onclick="cookiePostRemove(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_remove_display . ';" />';

Par cela :

Code :

$quote_img = '<img id="quote_add_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_add'] . '" alt="' . $lang['MuQu_Add_to_quote_list'] . '" onclick="cookiePostAdd(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_add_display . ';" />';

$quote_remove_img = '<img id="quote_remove_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_remove'] . '" alt="' . $lang['MuQu_Remove_from_quote_list'] . '" onclick="cookiePostRemove(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_remove_display . ';" />';

Apparement, IE ne gère pas le changement d'image et le title en même temps ...
Ca me fait plaisir de voir des moddeurs aussi acharnés

Vos MOD's sont tout simplement fabuleuux et ont tous leur utilité !!

Continuez comme ça

|

Invité

Posté le : 21 Fév 2006 16:40

Bon bah, très très joli MOD.
Par contre, l'erreur ne vient pas du title.

Une chose, les citations semblent s'afficher par ordre d'ID du post. Serait-il possible de les afficher par ordre de clique ??? (1er message cité, 1er message affiché)

|

crewstyle

Invité

Posté le : 21 Fév 2006 18:29

Petit rapport lol :

Je ne sais pas si c'est de temps ou temps ou apres une action précise, mais lorsque je clique sur "Répondre", voici le message retourné :

Code :

Erreur Générale
Could not obtain quotes

DEBUG MODE

SQL Error : 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ':2:{s:11:\"autologinid\";s:0:\"\";s:6:\"userid\";s:1:\"2\";}, 12096, 12545) ' at line 5

SELECT p.post_id, p.poster_id, u.username AS post_username, t.post_text, t.bbcode_uid FROM phpbb_posts p, phpbb_posts_text t, phpbb_users u WHERE p.post_id = t.post_id AND p.poster_id = u.user_id AND p.post_id IN ( phpbb2mysql_data=a:2:{s:11:\"autologinid\";s:0:\"\";s:6:\"userid\";s:1:\"2\";}, 12096, 12545) ORDER BY p.post_id DESC

Line : 868
File : posting.php

(peut-etre a-t-il un rapport sur le fait que je suis admin et que je poste dans un sujet VERROUILLÉ ...)

Quant au problème sous IE, je l'ai résolu de cette manière ... dans le fichier viewtopic.php, chercher :

Code :

$quote_add_img = '<img id="quote_add_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_add'] . '" alt="' . $lang['MuQu_Add_to_quote_list'] . '" title="' . $lang['MuQu_Add_to_quote_list'] . '" onclick="cookiePostAdd(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_add_display . ';" />';

$quote_remove_img = '<img id="quote_remove_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_remove'] . '" alt="' . $lang['MuQu_Remove_from_quote_list'] . '" title="' . $lang['MuQu_Remove_from_quote_list'] . '" onclick="cookiePostRemove(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_remove_display . ';" />';

Remplacer par :

Code :

$quote_add_img = '<img id="quote_add_' . $postrow[$i]['post_id'] . '" name="quote_add_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_add'] . '" alt="' . $lang['MuQu_Add_to_quote_list'] . '" onclick="cookiePostAdd(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\'); document.getElementById(\'quote_remove_' . $postrow[$i]['post_id'] . '\').style.display=\'inline\'; this.style.display=\'none\';" style="cursor: hand; display: ' . $quote_add_display . ';" />';

$quote_remove_img = '<img id="quote_remove_' . $postrow[$i]['post_id'] . '" name="quote_remove_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_remove'] . '" alt="' . $lang['MuQu_Remove_from_quote_list'] . '" onclick="cookiePostRemove(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\'); document.getElementById(\'quote_add_' . $postrow[$i]['post_id'] . '\').style.display=\'inline\'; this.style.display=\'none\';" style="cursor: hand; display: ' . $quote_remove_display . ';" />';

Voilà tout ^^
@++ pour des nouvelles aventures

|

Darathor

Site Admin
Membre

Inscrit le : 02 Nov 2003
Messages : 1101
Localisation : Strasbourg

Posté le : 21 Fév 2006 23:42

Bonjour ^^

crewstyle a écrit :

J'étais sur le point de me lancer sur le création de ce MOD lorsqu'une idée me vient à l'esprit ... celle de regarder si ce MOD n'existait pas auparavant !

Héhé, typiquement le truc que je fais quasiment jamais :-D

(cela dit, comme les rares mods que j'ai pas fait moi-même, je les ai quand même retouchés de partout pour qu'ils collent avec mes MODs et mes préférences, ça change pas grand chose en terme de temps pour moi... Puis ça me permet de proposer des version alternatives aux MODs existants ^^)

Anonymous a écrit :

Une chose, les citations semblent s'afficher par ordre d'ID du post. Serait-il possible de les afficher par ordre de clique ??? (1er message cité, 1er message affiché)

C'est envisageable ça je pense... Faudrait que je vérifie mais les id sont a priori stockés dans le cookie dans l'ordre d'ajout donc ça ne doit pas necessiter trop de modifs ^^ Faudra que je voie ça

(par contre, perso je préfère en ordre chronologuqe (ID), donc ce sera en option

Je mets toujours plein d'options dans mes derniers MODs :-D

Avec mon MOD qui ajoute des champs dans les profils, c'est super vite fait

)

crewstyle a écrit :

Je ne sais pas si c'est de temps ou temps ou apres une action précise, mais lorsque je clique sur "Répondre", voici le message retourné :
Code :
Erreur Générale
Could not obtain quotes

DEBUG MODE

SQL Error : 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ':2:{s:11:\"autologinid\";s:0:\"\";s:6:\"userid\";s:1:\"2\";}, 12096, 12545) ' at line 5

SELECT p.post_id, p.poster_id, u.username AS post_username, t.post_text, t.bbcode_uid FROM phpbb_posts p, phpbb_posts_text t, phpbb_users u WHERE p.post_id = t.post_id AND p.poster_id = u.user_id AND p.post_id IN ( phpbb2mysql_data=a:2:{s:11:\"autologinid\";s:0:\"\";s:6:\"userid\";s:1:\"2\";}, 12096, 12545) ORDER BY p.post_id DESC

Line : 868
File : posting.php
(peut-etre a-t-il un rapport sur le fait que je suis admin et que je poste dans un sujet VERROUILLÉ ...)

Alors ça j'ai jamais eu :aw:

faut que je regarde ça de plus près, je suis pas en état à cette heure-là

crewstyle a écrit :

Quant au problème sous IE, je l'ai résolu de cette manière ... dans le fichier viewtopic.php, chercher :
Code :
$quote_add_img = '<img id="quote_add_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_add'] . '" alt="' . $lang['MuQu_Add_to_quote_list'] . '" title="' . $lang['MuQu_Add_to_quote_list'] . '" onclick="cookiePostAdd(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_add_display . ';" />';

$quote_remove_img = '<img id="quote_remove_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_remove'] . '" alt="' . $lang['MuQu_Remove_from_quote_list'] . '" title="' . $lang['MuQu_Remove_from_quote_list'] . '" onclick="cookiePostRemove(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\');" style="cursor: pointer; display: ' . $quote_remove_display . ';" />';

Remplacer par :
Code :
$quote_add_img = '<img id="quote_add_' . $postrow[$i]['post_id'] . '" name="quote_add_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_add'] . '" alt="' . $lang['MuQu_Add_to_quote_list'] . '" onclick="cookiePostAdd(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\'); document.getElementById(\'quote_remove_' . $postrow[$i]['post_id'] . '\').style.display=\'inline\'; this.style.display=\'none\';" style="cursor: hand; display: ' . $quote_add_display . ';" />';

$quote_remove_img = '<img id="quote_remove_' . $postrow[$i]['post_id'] . '" name="quote_remove_' . $postrow[$i]['post_id'] . '" src="' . $images['MuQu_icon_quote_remove'] . '" alt="' . $lang['MuQu_Remove_from_quote_list'] . '" onclick="cookiePostRemove(' . $topic_id . ', ' . $postrow[$i]['post_id'] . ', \'' . $board_config['cookie_name'] . '\'); document.getElementById(\'quote_add_' . $postrow[$i]['post_id'] . '\').style.display=\'inline\'; this.style.display=\'none\';" style="cursor: hand; display: ' . $quote_remove_display . ';" />';

'vais voir ça ^^ En gros t'as rajouté un attibut name en plus de l'id ? (le truc qui m'embête c'est qu'un name sur autre chose que des formulaire, c'est pas valide XHTML 1.1 :-/

Il est chiant IE à necessiter des trucs pareils :blase:

)

| Profil | MP | E-mail | Site web

crewstyle

Invité

Posté le : 22 Fév 2006 00:05

Salut Darathor

Autant pour moi, l'attribut name est resté ... il était là pour mes 1ers tests !! Vous n'en avez pas du tout besoin vu que je fais un getElementById et non un getElementsByName !!
Donc, ça sera valide XHTML 1.1 lol

Quant à l'erreur, c'est vrai qu'elle est assez bizarre ... d'autant plus bizarre que je viens de tester ton MOD sur un autre forum verrouillé et ... rien !! Tout fonctionne normalement ... Là je vous avoue que ... je sèche lol

Sinon, j'ai vu ce MOD sur les forum IPB (qui sont plutot bien fait, je trouve) et j'ai décidé de le reproduire (tout comme mon MOD MiniProfile permettant d'afficher une popup pour chaque membre dans le viewtopic à l'aide d'un bouton placé à coté du bouton profile) ... mais vu que vous y êtes ...

Oh, une question par rapport à ça : pour la citation multiple, est-ce que le passage par cookie est l'unique solution ???

Sinon, merci pour tout

J'aime bien ce que vous faites lol

|

Darathor

Site Admin
Membre

Inscrit le : 02 Nov 2003
Messages : 1101
Localisation : Strasbourg

Posté le : 22 Fév 2006 00:28

Oh, j'ai mis un ByName ? :eek:

Outche, dans ce cas pas étonnant que ça foire... Cela dit, ça m'étonne un peu mais bon, je corrigerai ^^ (cela dit, ça doit se trouver dans le fichier js, pas dans le tpl a priori).

Concernant le "mini profile", j'ai vu ça sur les forums vBulletin aussi je crois... Ça ouvre un cadre par dessus la page (par opposition à un vraie pop-up qui ouvre une nouvelle fenêtre), c'est ça ? C'est vrai que ça peut être une intéressante alternative aux colonnes de gauche à ralonge (sous l'avatar) ^^ Mais c'est pas dans mes projets (j'ai déjà beaucoup trop de mods à faire), donc vous pouvez le faire sans risque de le retrouver chez moi dans l'immédiat

| Profil | MP | E-mail | Site web

Invité

Posté le : 22 Fév 2006 02:20

Aïe ... Nous nous sommes mal compris lol
Le ByName, c'est moi qui l'ai ajouter pour faire mes tests afin de faire fonctionner le changement d'image. Il m'a donc fallut ajouter l'attribut name aux images ... devenu inutile vu que la solution est de passer par les Id

Quant au MOD MiniProfile, sachez que je l'ai déjà réalisé

Voici de plus amples informations : ~lien~

De plus, j'aime bien l'idée de propositions alternatives à des MOD's existants

Donc si vous vous lancez sur un MOD que j'ai déjà fais, j'ai juste une seule chose à vous dire : faites-vous plaisir

A bientot

|

Darathor

Site Admin
Membre

Inscrit le : 02 Nov 2003
Messages : 1101
Localisation : Strasbourg

Posté le : 22 Fév 2006 16:11

Oh, je le voyais un peu différemment le "mini-profil"

Moi j'aurais fait un équivalent d'une bulle d'aide qui s'ouvre quand on passe la souris sur l'avatar par exemple... Je ferai peut-être ça un jour d'ailleurs...

Sinon, j'ai testé en utilisant votre méthode, en définissant l'attribut style.display plutôt que tout l'attribut style et en effet, ça marche sous IE ^^ (par contre, ça décale le bouton citer de quelques pixels mais bon c'est un détail

). Je l'intégrerai donc à la prochaine version :smile:

Merci ^^

| Profil | MP | E-mail | Site web

crewstyle

Invité

Posté le : 22 Fév 2006 17:44

Darathor a écrit :

Oh, je le voyais un peu différemment le "mini-profil" Moi j'aurais fait un équivalent d'une bulle d'aide qui s'ouvre quand on passe la souris sur l'avatar par exemple... Je ferai peut-être ça un jour d'ailleurs...

Sinon, j'ai testé en utilisant votre méthode, en définissant l'attribut style.display plutôt que tout l'attribut style et en effet, ça marche sous IE ^^ (par contre, ça décale le bouton citer de quelques pixels mais bon c'est un détail ). Je l'intégrerai donc à la prochaine version Merci ^^

Merci à vous Darathor

Personnellement, mon template fait en sorte que le bouton ne se déplce pas

Donc, pour moi, il est parfait ce MOD.

Par contre, une question plus sérieuse ... à votre avis, y a-t-il moyen de hacker un site en passant par le cookie créé par ce MOD ???
Enfait, je vous pose la question car, à l'époque où je fesait partie de l'équipe des moddeurs de phpBB-fr, j'avais lu que le passage des forums phpBB de 2.0.16 en 2.0.17 était dû à un hacking par cookie ...

Donc, voilà, point de vue sécurité, je ne sais pas si on peut installer ce MOD ...
A mon avis, je vous répondrais lorsque je me ferai hacker lol ^^

Sinon, c'est une belle idée la bulle d'aide s'ouvrant lorsque la souris passe sur l'avatar

@++

|

Darathor

Site Admin
Membre

Inscrit le : 02 Nov 2003
Messages : 1101
Localisation : Strasbourg

Posté le : 22 Fév 2006 17:50

Heu niveau sécu, je ne pense pas qu'il y ait de risque : seuls les cookies spécifiques à ce mod sont affectés et ils ne contiennent qu'une liste d'IDs... Donc je ne vois pas trop...

Quoiqu'à la réflexion, y aurait moyen qu'il récupère des posts venant d'autres topics (donc potentiellement de forums privés) :-/

Va falloir que je rajoute un test pour qu'il vérifie que les posts cités appartiennent bien au topic considéré et le problème sera réglé ^^

| Profil | MP | E-mail | Site web

Darathor

Site Admin
Membre

Inscrit le : 02 Nov 2003
Messages : 1101
Localisation : Strasbourg

Posté le : 28 Avr 2006 18:46

Volà, j'ai posté la nouvelle version ^^

Elle corrige a priori les deux problèmes :
- le bug avec IE.
- la faille qui aurait permis de citer des messages auxquels on n'est pas censé avoir accès (forums privés) : seuls les messages appartenant bien au topic courant sont cités.

| Profil | MP | E-mail | Site web

Dernière édition par Darathor le 06 Aoû 2006 18:37; édité 1 fois