Context Navigation

← Previous Ticket
Next Ticket →

Ticket #228 (closed defect: fixed)

Opened 9 years ago

Last modified 7 years ago

Sauvegarde impossible des MDD avec nom composée

Reported by:	benworld2	Owned by:	benjamin
Priority:	normal	Milestone:	alternc-0.9-1
Component:	Bureau: bases de données	Version:	mailman-1.5
Severity:	block	Keywords:
Cc:

Description

Bon, j'ai remarqué un petit bug

Si les bases user sont du style nomuser_nombase <-ok tout fonctionne en revanche si le nom est composé du type nomuser_nombase_ajout ca ne marche plus, lorqu'on crée le backup, le panneau d'admin nous retourne :

This database already exists!

Voila, peut etre pas grand chose, mais ca m'ennuie quand mm pas mal en fait.

Change History

comment:1 Changed 9 years ago by benworld2

Désolé, le titre c'est pas MDD mais BDD ...

comment:2 Changed 9 years ago by benworld2

je pense que l'erreur viens de la :

dans m_mysql.php

$dbname=$mem->userlogin?.($dbn?"_":"").$dbn; $db->query("SELECT * FROM db WHERE uid='$cuid' AND db='$dbname';"); if (!$db->num_rows()) {

$err->raise("mysql",4); return false;

solution :

soit empecher l'utilisateur d'utiliser le _ dans le nom de sa base soit modifier ces lignes de code pour pouvoir sauvegarder.

comment:3 Changed 9 years ago by benworld2

Bon, voila, j'ai trouvé l'erreur et je l'ai corrigé :

Deja, erreur du regex lors de la creation des bases :

function add_db

changer :

if (!ereg("^{[0-9a-z]*",$dbn)) {}

if (!ereg("^{[0-9a-z]*$",$dbn)) {}

Et pour permettre la sauvegarde des bases deja crée a cause de ce bug (donc de type nomutilisateur_bas_indice (et donc avec deux ou plus de _ )

changer :

dans la function get_dblist()

changer : list($dbu,$dbn)=explode("_",$db->f("db")); en $pos=strpos($db->f("db"),"_"); list($dbu,$dbn)=array(substr($db->f("db"),0,$pos - 1),substr($db->f("db"),$pos + 1));

dans la function get_mysql_details($dbn)

changer : list($dbu,$dbn)=explode("_",$db->f("db")); en $pos=strpos($db->f("db"),"_"); list($dbu,$dbn)=array(substr($db->f("db"),0,$pos - 1),substr($db->f("db"),$pos + 1));

dans la function del_db($dbn)

changer : if (!ereg("^{[0-9a-z]*$",$dbn)) {
en
if (!ereg("}[0-9a-z_]*$",$dbn)) {

et dans la fonction function put_mysql_backup($

changer : if (!ereg("^{[0-9a-z]*$",$dbn)) {
en
if (!ereg("}[0-9a-z_]*$",$dbn)) {

les deux dernier permettent de suprimer les bases avec nom composé et les sauvegarder.

Avec ces deux modif, pls possible de crée des bases avec nom composée MAIS possibilité de sav les BDD existantes avec nom composée

Voila, enjoy

comment:4 Changed 9 years ago by benjamin

Owner changed from anonymous to benjamin
Status changed from new to closed
Resolution set to fixed

Résolu ce jour dans le CVS, J'ai seulement corrigé lors de la création des bases de données. Appliquez les autres patchs dans vos serveurs existants, et demandez aux quelques rares bases créées du type "login_db_name" de modifier leur nom de base ...

comment:5 Changed 7 years ago by anonymous

Note: See TracTickets for help on using tickets.

Download in other formats: