Release 1.2.4

3 views
Skip to first unread message

jojaba

unread,
Mar 25, 2009, 5:59:07 AM3/25/09
to Plume CMS - developers
Je me suis dit qu'il serait plus intéressant de parler de la prochaine
mise à jour dans un sujet à part (ce sera plus facile à retrouver plus
tard...)
Pour mémoire je note ici ce qui pourrait être fait (je mets ça au
conditionnel parce qu'en listant les différentes choses à voir, je me
suis rendu compte que cette liste n'était pas si courte que ça !) :
* Problème de la barre oblique de la balise <hr /> qui est
systématiquement retirée par l'application lorsqu'on ajoute un
attribut "style" ou "class" lors de la validation de l'article/de la
brève.
* Ajout de liens pointant vers l'aide pour les commentaires (le texte
d'aide doit être ajouté) et correction du lien vers l'aide pour la
date d'expiration.
* Problème du rss ? (pas sûr que ça vienne de Plume CMS)
* Mise à jour de la traduction grecque proposée par Sonickidon de
BabelZilla surle site de traduction que j'ai mis en place
* Ajout du plugin "Alertcom" de kato (modification du fichier
config.php obligatoire). Je l'utilise sur mon site perso, il
fonctionne impeccablement et c'est à mon avis une fonctionnalité qui
rendra énormément service. Le fait de ne pas être traduit en français
(la notification est en anglais) n'est pas une raison pour ne pas
l'inclure à la prochaine release. Je l'envoie vers le svn en
attendant..
* Bug chez OVH signalé par LapS ici :
http://groups.google.com/group/plume-cms-developers/t/610187e1d3afd668?hl=fr
(je ne crois pas que tu as corrigé ça Loïc)
* Eventuellement, revoir le formulaire de multi-upload...
Si vous voyez d'autres choses, n'hésitez pas à les ajouter ici (même
si cela ne voudra pas forcément dire qu'elles seront réglées pour
cette nouvelle version)

Jojaba

unread,
Mar 27, 2009, 3:15:38 AM3/27/09
to plume-cms-...@googlegroups.com
jojaba a écrit :

* Ajout de liens pointant vers l'aide pour les commentaires (le texte
d'aide doit être ajouté) et correction du lien vers l'aide pour la
date d'expiration.
  
Règlé et envoyé sur SVN ;-)
* Ajout du plugin "Alertcom" de kato (modification du fichier
config.php obligatoire). Je l'utilise sur mon site perso, il
fonctionne impeccablement et c'est à mon avis une fonctionnalité qui
rendra énormément service. Le fait de ne pas être traduit en français
(la notification est en anglais) n'est pas une raison pour ne pas
l'inclure à la prochaine release. Je l'envoie vers le svn en
attendant..

  
Fait ! J'ai encore tenté de faire reconnaître le dossier "locale" d'alertcom dans le fichier "register.php" mais je n'y arrive vraiment pas !  J'avais réussi à un moment en faisant un include de prepend.php au début du fichier mais cela aboutissait à des erreurs sur la page Web... Il faudrait trouver les bons fichiers à inclure pour que que ça fonctionne (probablement un jeu d'enfant pour toi Loïc :-P ). Tout est prêt autrement, la fonction __() est en place pour les chaînes devant être traduites, la traduction en français, les fichiers de locale des différentes langes prêts à être traduits, le fichier config.php-dist modifié (à ce propos, est-ce que le fichier config.php sur l'hébergement sera mis à jour en cas de mise jour de plume intégrant alertcom ? Je suppose que non, il faudra donc procéder à un copier/coller manuel des lignes à ajouter.)

* Eventuellement, revoir le formulaire de multi-upload...
  
Je rappelle que je l'ai installé sur le site de test : http://plumecmstest.free.fr/
Je l'avais laissé comme il était, il semble fonctionner à peu près correctement (je rappelle qu'il est possible d'indiquer dans l'outil de configuration de plume  le nombre de champs d'upload à ajouter). Il arrive à uploader les fichiers valides pas trop volumineux (j'ai l'impression que la limitation de 1.9 Mo est un peu optimiste, je suppose que free a limité l'upload d'une autre manière pour éviter la surcharge du serveur, peut-être que l'upload en séquence n'est pas supporté...) et à notifier cela (derrière le champ correspondant). Il arrive à détecter, notifier et à ne pas uploader les fichiers non valides (exe par exemple). Il arrive à détecter les noms de fichiers non valides (avec accents par exemple) et à le notifier. Cependant, il semblerait que le script ne fonctionne pas lors de la détection de fichier trop volumineux. Je pense qu'il y a une erreur d'écriture de ma part dans la prise en compte des erreurs survenues lors de l'upload indiquées par $_FILES['fichier_uploadé']['error']...
Je vais tenter de revoir ça ce week end mais toute aide est la bienvenue !

Pour information, copier/coller du code modifié :

## Traitement des fichiers ########################
/* Upload of the file */
//Defining some variables
$number_multi_up = $_PX_config['multi_upload'];//Number of submission fields to display
$message = array();//Message to notify if success or error when uploading
if ($is_writable && !empty($_FILES['up_file']) && (false === $m->error())) {
   
    for ($i=0; $i<=$number_multi_up-1; $i++)//Loop for handling each submission field
    {//Simplification of arrays
         $file = $_FILES['up_file'];
     $tmp_file = $file['tmp_name'][$i];
         $file_name = $file['name'][$i];
    if (!$file_name) {//If field empty
        continue;//Next iteration
    }

    if (version_compare(phpversion(),'4.2.0','>=')) {
        $upd_error = $file['error'][$i];
    } else {
        $upd_error = 0;
    }

    if ($upd_error != 0) {
        switch ($upd_error) {
        case UPLOAD_ERR_INI_SIZE:
            $message[$i] = '<span class="errmsg">'.__('Error: The size of the file excess the maximum size.').'</span>';
            break;
        case UPLOAD_ERR_FORM_SIZE:
            $message[$i] = '<span class="errmsg">'.__('Error: The size of the file excess the maximum size.').'</span>';
            break;
        case UPLOAD_ERR_PARTIAL:
            $message[$i] = '<span class="errmsg">'.__('Error: File not fully transfered.').'</span>';
            break;
        case UPLOAD_ERR_NO_FILE:
            $message[$i] = '<span class="errmsg">'.__('Error: No file.').'</span>';
            break;
        }
        continue;//next iteration
    } elseif(!file_exists($tmp_file)) {
        $message[$i] = '<span class="errmsg">'.__('Error: No file.').'</span>';
    continue;//next iteration
    } elseif(filesize($tmp_file) > config::f('max_upload_size')) {
        $message[$i] = '<span class="errmsg">'.__('Error: The size of the file excess the maximum size.').'</span>';
    continue;//next iteration
    } else {
        $file_name = preg_replace( '#(\s+)#', '_', $file_name);
        if(!isFileSafe($file_name)) {
            $message[$i] = '<span class="errmsg">'.__('Error: Not an authorized type of file. The file name must contain only simple letters, digits, "-", "_" and a recognized extension. No spaces allowed.').'</span>';
    continue;//next iteration
        } else {
            if (@copy($tmp_file,$up_dir.'/'.$current_dir.$file_name)) {
                @chmod($up_dir.'/'.$current_dir.$file_name, 0666);
                /* create thumb */
                $thumb = $up_dir.'/thumb/'.md5($current_dir.$file_name).'.jpg';
                @unlink($thumb); //ensure that the old thumbnail is removed if available
                @$msg=cropImg($up_dir.'/'.$current_dir.$file_name, $thumb, 150, 100);
                @chmod($thumb, 0666);

                $message[$i] = '<span class="message">'.__('File successfully uploaded.').'</span>';
                $up_success = true;
                //I don't know why this line below must be deleted for this code to work...
                /*header('Location: xmedia.php?dir='.rawurlencode($current_dir)
                       .'&mode='.$mode.'&env='.$env);*/
            } else {
                $message[$i] = '<span class="errmsg">'.__('An error occured during the transfer of the file.').'</span>';
            }
        }
    }
}
}
################################################

###Modification du formulaire#########################
<?php if($is_writable) : ?>

<form enctype="multipart/form-data" action="xmedia.php" method="post">
<fieldset><legend><?php   echo __('Add a file to the current folder'); ?></legend>
<p><label><?php  echo sprintf( __('Choose a file (maximum size %s)'), prettySize($_PX_config['max_upload_size'])); ?> <?php echo $m->HelpLink('xmedia', 'h-add-file'); ?></label></p>
<?php //Loop to display the number of fields set in the configuration file
for ($i=0; $i<=$number_multi_up-1; $i++) {
?>
<p style="margin: 0 5px;"><input name="up_file[]" type="file" />&nbsp;
<?php //Displaying the message if exists
if (!empty($message[$i])){
               echo $message[$i];
        }
?>
<input type="hidden" name="MAX_FILE_SIZE" value="<?php echo $_PX_config['max_upload_size']; ?>" />
<input type="hidden" name="mode" value="<?php echo $mode; ?>" />
<input type="hidden" name="env" value="<?php echo $env; ?>" />
<input type="hidden" name="dir" value="<?php echo $current_dir; ?>" /></p>
<?php
//End of loop
}
?>
<p><input class="submit" type="submit" value="<?php echo  __('Add the file'); ?>" /></p>
</fieldset>
</form>
<?php endif; ?>
##################################################

Loic d'Anterroches

unread,
Mar 27, 2009, 3:52:28 AM3/27/09
to plume-cms-...@googlegroups.com
Salut,

désolé pour le retard dans ma réponse. Voici quelques idées remarques :

> * Problème de la barre oblique de la balise <hr /> qui est
> systématiquement retirée par l'application lorsqu'on ajoute un
> attribut "style" ou "class" lors de la validation de l'article/de la
> brève.

On se demande tous les deux ce qui se passe. Je vais regarder plus ew
détails la semaine prochaine.

> * Ajout de liens pointant vers l'aide pour les commentaires (le texte
> d'aide doit être ajouté) et correction du lien vers l'aide pour la
> date d'expiration.

Ok.

> * Problème du rss ? (pas sûr que ça vienne de Plume CMS)

Fixé dans le dépôt.

> * Mise à jour de la traduction grecque proposée par Sonickidon de
> BabelZilla surle site de traduction que j'ai mis en place

Ok.

> * Ajout du plugin "Alertcom" de kato (modification du fichier
> config.php obligatoire). Je l'utilise sur mon site perso, il
> fonctionne impeccablement et c'est à mon avis une fonctionnalité qui
> rendra énormément service. Le fait de ne pas être traduit en français
> (la notification est en anglais) n'est pas une raison pour ne pas
> l'inclure à la prochaine release. Je l'envoie vers le svn en
> attendant..

Ok.

> * Bug chez OVH signalé par LapS ici :
> http://groups.google.com/group/plume-cms-developers/t/610187e1d3afd668?hl=fr
> (je ne crois pas que tu as corrigé ça Loïc)

Je regarde la semaine prochaine.

> * Eventuellement, revoir le formulaire de multi-upload...
> Si vous voyez d'autres choses, n'hésitez pas à les ajouter ici (même
> si cela ne voudra pas forcément dire qu'elles seront réglées pour
> cette nouvelle version)

Pas ok. Je souhaite pouvoir tester cela sérieusement avant une possible
inclusion. L'upload de fichiers peut très facilement introduire des
problèmes de sécurité.

Merci pour cette bonne checklist,
loïc

>
> >

Jojaba

unread,
Mar 27, 2009, 5:42:53 AM3/27/09
to plume-cms-...@googlegroups.com
Ça fait déjà 2 messages que j'envoie au groupe, mais ils ne sont pas
publiés, ni diffusés. Je teste à nouveau là, avant de renvoyer le
message que j'avais écrit à propos de cette release.


>
>
>

Loic d'Anterroches

unread,
Mar 27, 2009, 5:47:12 AM3/27/09
to plume-cms-...@googlegroups.com

ok pour celui-là.

loïc

Jojaba

unread,
Mar 27, 2009, 5:49:55 AM3/27/09
to plume-cms-...@googlegroups.com
Ah ben ça y est, mes messages sont publiés à présent (en tout cas sur le
Web). Par contre, je ne les ai pas obtenu dans ma boîte aux lettres (en
général ça le faisait). Le message le plus important se trouve avant le
tiens dans le groupe :
http://groups.google.com/group/plume-cms-developers/t/191f860e0cdc365b?hl=fr

Luigi

unread,
Mar 27, 2009, 8:27:09 AM3/27/09
to Plume CMS - developers
SI on pouvait aussi inclure dans cette version une révision du
parser / validateur hmt, ça serait bien. En effet, le <hr /> n'est pas
le seul truc qui bloque. Dès que l'on ajoute des vidéos (genre flash
youtube et companie), souvent ca casse aussi. Et ca se répercute aussi
sur le fil rss par voie de fait.
> tiens dans le groupe :http://groups.google.com/group/plume-cms-developers/t/191f860e0cdc365...

Jojaba

unread,
Mar 27, 2009, 11:04:43 AM3/27/09
to plume-cms-...@googlegroups.com
Luigi a écrit :

> SI on pouvait aussi inclure dans cette version une révision du
> parser / validateur hmt, ça serait bien. En effet, le <hr /> n'est pas
> le seul truc qui bloque. Dès que l'on ajoute des vidéos (genre flash
> youtube et companie), souvent ca casse aussi. Et ca se répercute aussi
> sur le fil rss par voie de fait.
L'ajout de code provenat de youtube est forcément "cassé" puisque le
code n'est pas "propre" et prévu de fonctionner sur un maximum de
navigateur (embed est une balise propriétaire donc non valide donc
supprimée automatiquement)...

Jojaba

unread,
Mar 30, 2009, 2:05:25 AM3/30/09
to plume-cms-...@googlegroups.com
Luigi a écrit :

> Dès que l'on ajoute des vidéos (genre flash
> youtube et companie), souvent ca casse aussi. Et ca se répercute aussi
> sur le fil rss par voie de fait.
>
Au lieu d'une révision complète, je verrais plutôt la mise en place d'un
balisage indiquant une portion de code à ne pas analyser (donc ignorée
par le parser html). C'est vrai que quand on est en production (je parle
maintenant pour les webmaster pro alors que je suis totalement amateur
en la matière :-\ ), ou quand on y comprend pas grand-chose au HTML, il
est intéressant de pouvoir rapidement insérer un code prêt à l'emploi...
C'est pourquoi une balise dédiée à la fonctionnalité dont je viens de
parler serait vraiment la bienvenue. L'idée me vient de dotclear qui
propose une balise pour pouvoir insérer du code html dans une fenêtre de
composition wiki (il me sembe que c'est ///, ou quelque chose du même
genre, enfin peu importe...). Je ne pense pas que cela soit trop
difficile à mettre en place (je suis très mal placé pour dire ça puisque
je ne comprends pas grand chose au JavaScript... :-P ) et ça rendrait
vraiment service.
Pour ceux qui seraient intéressés par l'implémentation d'une fonction de
ce genre, les fichiers à modifier se trouvent dans le dossier
manager/tools/visualedit/js, il me semble que c'est le fichier
jsToolBar.wysiwyg.js qui est le plus important (à faire confirmer par
Loïc). Le "nettoyeur" commence à la ligne 375, mais je pense qu'il
faudrait modifier à partir de la ligne 434... d'autres fichiers devront
également être modifiés à mon avis (dans manager/inc et manager/extinc).
Donc avis aux amateurs...pros du codage :-P

jojaba

unread,
Apr 5, 2009, 3:52:16 AM4/5/09
to Plume CMS - developers
Un petit mot pour signaler que Sonickydon, le traducteur Grec
Babelzillien, vient de terminer la mise à jour des fichiers de
traduction (ajout du fragment concernant les commentaires dans le
fichier d'aide et traduction du plugin alertcom + révision des
fichiers pour le template par défaut, le template du Manager et de
l'installation). Il a fourni un gros boulot et je l'ai remercié
chaleureusement. Tout a été envoyé vers le dépot svn.
Reply all
Reply to author
Forward
0 new messages