phpBB-fr.com

[lifeburnout]

• Titre: shoutbox AJAX
• Description: shoutbox AJAX dynamique sur l'index de votre forum.
• Version: [beta5]1.0.0
• Auteur: lifeburnout
• téléchargement: ici
• démonstration: http://lifeburnout.legtux.org/forum/index.php
• Captures d'écran (non à jour): membre connecté:
  
  
  invité:
  
  
  panneau d'admin:
  
  

[Steph]

Salut, voici les toutes premières questions :

• Est ce que la Chatbox prend en compte le pseudo de l'utilisateur automatiquement si celui ci est inscris et loguer sur le forum ?
• Est ce que la Chatbox est personnalisable ?

[lifeburnout]

Salut, voici les toutes premières questions :

• Est ce que la Chatbox prend en compte le pseudo de l'utilisateur automatiquement si celui ci est inscris et loguer sur le forum ?
• Est ce que la Chatbox est personnalisable ?

bonjour Steph,

oui la chatbox est personalisable
et oui le pseudo est automatiquement loguer, sauf pour les invité ou il devront rentré un pseudo.

à bientôt.

[Dakin Quelia]

Bonjour lifeburnout,

Une chat en JS, ce n'est pas le top, je pense. C'est mieux en AJAX.

A bientôt,

[lifeburnout]

Et pourtant elle marche bien et elle est rapide.

En plus ce n'est qu'un mod en développement donc sa peut changer.

-- EDIT --
ha non, c'est de l'ajax parce que j'utilise:
XMLHttpRequest avec une requête et la réponse qui suit, si je l'apprend que maintenant, c'est que je vient juste de me renseigner avec mon ami google. Je corrigerais le fichier install.xml pour dire que c'est de l'ajax plus tard parce que la je vais partir.

[Steph]

et oui le pseudo est automatiquement loguer, sauf pour les invité ou il devront rentré un pseudo.

Bien bien cela va de soi, je vais donc tester dès que possible, merci

[ErnadoO]

Dans chat.php, remplace tes

Code: Tout sélectionner

if($user->data['user_id'] == ANONYMOUS) 

par

Code: Tout sélectionner

if(!$user->data['is_registered']) 

Tu utilises plusieurs fois la superglobale $_POST, pourquoi ne pas utiliser request_var()?

Code: Tout sélectionner

if($message_post_text == '') 

à remplacer par

Code: Tout sélectionner

if(empty($message_post_text)) 

même chose pour if($user_chat == '')

Ton switch ne sert à rien, pour deux raisons: $mode ne peut valoir que 'action' et c'est le seul cas de figure que tu traites.

Pour ce passage:

Code: Tout sélectionner

if($user->data['user_id'] == ANONYMOUS)
{
    $template->assign_vars(array(
        'MESSAGE'       => $message,
        'U_ACTION'      => "{$phpbb_root_path}index.$phpEx?mode=action"
    ));
}
else
{
    $template->assign_vars(array(
        'MESSAGE'       => $message,
        'U_ACTION'      => append_sid("{$phpbb_root_path}index.$phpEx", 'mode=action')
    ));
} 

perso, anonyme ou pas, j'utiliserais toujours le même code, le 2em, par convention.

Pour $user_chat, tu lui assignes des valeurs différentes en fonction que l'user soit logué ou pas, de cette façon:

Code: Tout sélectionner

if($user->data['user_id'] != ANONYMOUS)
{
    $user_chat = $user->data['username'];
}

......

            if($user->data['user_id'] == ANONYMOUS)
            {
                $user_chat = $_POST['user'];
            } 

C'est crade et pas optimisé; la façon propre étant:

Code: Tout sélectionner

$user_chat = request_var('user', $user->data['username']); 

Ainsi, le champ n'étant visible que pour les invités, ta variable prendra soit le contenu du champ, soit l'username du membre (puisque le champ n'aura pas été envoyé par le formulaire, c'est la valeur par défaut qui sera utilisé, à savoir l'username).
du coup, cette condition

Code: Tout sélectionner

                if($user_chat == '')
                {
                    if($user->data['user_id'] == ANONYMOUS)
                    {
                        $template->assign_vars(array(
                            'S_NO_USER_POST' => true,
                            'NO_USER'        => $user->lang['NO_USER']
                        ));
                    }
                } 

est à revoir, puisque $user_chat ne sera jamais vide (il faut donc faire une comparaison avec l'username du membre anonyme, si true, alors c'est que l'invité n'a spécifié aucun username t c'est vu attribuer celui de l'utilisateur anonyme).

Ce bout de code:

Code: Tout sélectionner

if($user->data['user_id'] == ANONYMOUS)
{
    $template->assign_vars(array(
        'USER_INVIT'    => $_SESSION['user']
    ));
}
else
{
    $template->assign_vars(array(
        'USER_INVIT'    => $user->data['username']
    ));
} 

C'est pas bon, le champ user étant invisible pour les membres connecté, quel intérêt de lui envoyer une valeur par défaut dans ce cas? Ceci est mieux:

Code: Tout sélectionner

$template->assign_vars(array(
    'USER_INVIT'    => (!$user->data['is_registered']) ? $_SESSION['user'] : '',
)); 

Ce ne sont là que les erreurs de codage grossières visibles sans vraiment chercher la petite bête, après, sur la méthodologie, il y aurait à dire aussi pour optimiser ton code

[lifeburnout]

Merci ErnadoO.

mise à jour vers la 0.0.1 > 0.0.2
zip et lien mise à jour.
changer chatbox.php fournit dans l'archive.

changement de la 0.0.1 > 0.0.2:

• code de la chatbox plus propre
• chatbox un peu plus jolie.

[ErnadoO]

- Ton switch inutile est toujours présent
- Tu utilises toujours la superglobale $_POST au lieu de request_var()


En fait tu n'as fais que remplacer les bouts de codes dont je donnais la correction, sans tenir comptes des autres remarques

[Jonathan44]

Bonjour.
D'après le test que j'ai fais sur l'index de ton forum on est redirigé en haut de la page a chaque envoi. dommage que l'on ne reste pas au niveaux de la shout...

Jonathan...

[pete parker]

bonjour , jonathan44 a bien dit le mot shout .... ça serait pas une shoutbox, ta chatbox en réalité ?

[lifeburnout]

:) bonjour , jonathan44 a bien dit le mot shout .... ça serait pas une shoutbox, ta chatbox en réalité ?

en faites je ne sais pas trop la différence en une shoutbox et une chatbox, mais pour moi c'est pareil.

Bonjour.
D'après le test que j'ai fais sur l'index de ton forum on est redirigé en haut de la page a chaque envoi. dommage que l'on ne reste pas au niveaux de la shout...

Jonathan...

je sais, j'essaye de corriger le problème, mais je suis déjà un peut pris donc ce n'est pas très facile.

- Ton switch inutile est toujours présent
- Tu utilises toujours la superglobale $_POST au lieu de request_var()


En fait tu n'as fais que remplacer les bouts de codes dont je donnais la correction, sans tenir comptes des autres remarques

je sais, je corrigerais sa demain ou ce soir, parce que j'ai fait la mise à jour vite fait. Mais je commence à comprendre, il me faut maintenant le temps de mettre en forme.

-- EDIT --
mise à jour vers la 0.0.2 > 0.0.3
zip et lien mise à jour.
changer les fichiers fournit dans l'archive

voici les correction apporté:
fichier chatbox.php moins lours
les superglobale $_POST on été remplacé par request_var()
on utilise plus de switch
no-cache mieux fait
absurdité enlevé
le fichier template chatbox.html est plus propre.

à bientôt.

[joebart]

Salut

Dis moi, on peut tester en démo ?

Bbcodes et smilies intégrés ou non ?

Merci pour le taf

[lifeburnout]

bonjour, non je n'est pas intégré de BBcode est smilies pour l'instant, mais il viendront après. Mais si tu veux voir une démo, regarde ici:
http://lifeburnout.clanfree.net/forum/index.php

à bientôt.

[lifeburnout]

mise à jour vers la 0.0.3 > 0.0.4
zip et lien mise à jour.
changer les fichiers fournit dans l'archive

ensuite ouvrez index.php et cherchez:

Code: Tout sélectionner

$user->add_lang('chatbox');

et supprimez ce bout de code.

correction d'un bug avec l'encodage.
moins d'installation à faire.