Changeset 230

Timestamp:

25.11.2005 00:52:02 (3 years ago)

Author:

bobe

Message:

Ajout refonte du système d’inscription pour passage par code only sur confirmation

Files:

• trunk (modified) (1 prop)
• trunk/.htaccess (deleted)
• trunk/includes (modified) (1 prop)
• trunk/includes/class.form.php (modified) (20 diffs)
• trunk/includes/engine_send.php (modified) (6 diffs)
• trunk/includes/functions.validate.php (modified) (1 diff)
• trunk/language/lang_francais.php (modified) (1 diff)
• trunk/newsletter.php (modified) (2 diffs)

trunk/includes/class.form.php

@@ -40 +40 @@
-    var $hasAccount    = false;
-    var $isRegistered  = false;
-    var $update_stats  = false;
-    var $message       = '';
-    
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-                {
-
+
+
+
+
-                }
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-            }
-            else
-            {
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
-        }
-    }
@@ -115 +271 @@
-    function subscribe()
-    {
-, $mailer
+
-        
-        $db->transaction(START_TRC);
@@ -124 +280 @@
-                'abo_email'         => $this->account['email'],
-                'abo_pseudo'        => $this->account['pseudo'],
-$this->account['code']
+generate_key()
-                'abo_register_date' => $this->account['date'],
-                'abo_status'        => $this->account['status']
@@ -152 +308 @@
-            }
-            
-confirmed, register_date
-abo_id]}, {$this->listdata[liste_id]}, $this->format, $confirmed, " . time() . "
+register_key, register_date, confirmed
+'abo_id']}, {$this->listdata['liste_id']}, $this->format, '{$this->account['code']}', " . time() . ", $confirmed
-            if( !$db->query($sql) )
-            {
@@ -166 +322 @@
-        {
-            $email_tpl = ( $this->listdata['use_cron'] ) ? 'welcome_cron2' : 'welcome_form2';
-            $link_action = 'confirmation';
-        }
-        else
-        {
-            $email_tpl = ( $this->listdata['use_cron'] ) ? 'welcome_cron1' : 'welcome_form1';
-            $link_action = 'desinscription';
-            
-            $this->alert_admin(true);
-        }
-        
-
-
-
-
-
-
-
-
+this->
+this->
+this->
+this->
+this->
+this->
+
+this->
-            'LISTE'    => unhtmlspecialchars($this->listdata['liste_name']),
-            'SITENAME' => $nl_config['sitename'],
@@ -193 +346 @@
-        if( $this->listdata['use_cron'] )
-        {
-
+this->
-                'EMAIL_NEWSLETTER' => $this->liste_email
-            ));
@@ -199 +352 @@
-        else
-        {
-
-$link_action
+this->
+
-            ));
-        }
@@ -206 +359 @@
-        if( $nl_config['enable_profil_cp'] )
-        {
-
+this->
-                'LINK_PROFIL_CP' => make_script_url('profil_cp.php')
-            ));
-        }
-        
-
+this->
-        {
-            $this->message = $lang['Message']['Failed_sending'];
@@ -221 +374 @@
-            if( $this->listdata['confirm_subscribe'] == CONFIRM_NONE )
-            {
- = true
+()
-                $message = $lang['Message']['Subscribe_2'];
-            }
@@ -237 +390 @@
-            else if( $this->listdata['confirm_subscribe'] != CONFIRM_ALWAYS )
-            {
- = true
+()
-                $message = $lang['Message']['Subscribe_2'];
-            }
@@ -249 +402 @@
-    }
-    
-
-
-, $mailer
-
-$this->code == $this->account['code']
+code, $
+
+
+
+strcmp($code, $this->account['code']) == 0
-        {
-            $time = ( empty($time) ) ? time() : $time;
@@ -261 +414 @@
-            {
-                $low_priority = ( strncmp(DATABASE, 'mysql', 5) == 0 ) ? 'LOW_PRIORITY' : '';
+                $this->account['code'] = generate_key(20);
-                
-                $db->transaction(START_TRC);
@@ -274 +428 @@
-                
-                $sql = "UPDATE $low_priority " . ABO_LISTE_TABLE . "
-
+
+
-                    WHERE liste_id = " . $this->listdata['liste_id'] . "
-                        AND abo_id = " . $this->account['abo_id'];
@@ -285 +440 @@
-                $db->transaction(END_TRC);
-                
- = true
+()
-                $this->alert_admin(true);
-                
@@ -305 +460 @@
-    }
-    
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-                {
-e vérifier la table de jointure
+\'effacer l\'entrée de la table des abonnés
-                    return false;
-                }
-                
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
-            }
-            else
-            {
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-            
-            $email_tpl = ( $this->listdata['use_cron'] ) ? 'unsubscribe_cron' : 'unsubscribe_form';
-            
-
+this->
-                'LISTE'    => unhtmlspecialchars($this->listdata['liste_name']),
-                'SITENAME' => $nl_config['sitename'],
@@ -383 +529 @@
-            if( $this->listdata['use_cron'] )
-            {
-
+this->
-                    'EMAIL_NEWSLETTER' => $this->liste_email,
-                    'CODE'             => $this->account['code']
@@ -390 +536 @@
-            else
-            {
-
-'desinscription'
+this->
+
-                ));
-            }
-            
-($mailer->send()
+$this->mailer->send(
-            {
-                $this->message = $lang['Message']['Failed_sending'];
@@ -447 +593 @@
-    }
-    
-$action
-
-$prefix = $this->listdata['form_url'] . ( ( strstr($this->listdata['form_url'], '?') ) ? '&' : '?' );
-
-return $prefix . 'action=' . $action . '&email=' . rawurlencode($this->account['email']) . '&code=' . $this->account['code'] . '&liste=' . $this->listdata['liste_id
+
+
+return $this->listdata['form_url']
+   . (( strstr($this->listdata['form_url'], '?') ) ? '&' : '?')
+   . $this->account['code
-    }
-    
-    function alert_admin($new_subscribe)
-    {
-nl_config, $db, $mailer
+db, $nl_config, $lang
-        
-        if( $new_subscribe == true )
@@ -483 +629 @@
-            if( $row = $db->fetch_array($result) )
-            {
-
-
-
-
-
-
+
+
+
+
+
-                    'EMAIL'   => $this->account['email'],
-                    'LISTE'   => unhtmlspecialchars($this->listdata['liste_name']),
@@ -497 +642 @@
-                do
-                {
-
-
+this->
+this->
-                    
-
+this->
-                        'USER' => $row['admin_login']
-                    ));
-                    
-
+this->
-                }
-                while( $row = $db->fetch_array($result) );
-            }
+        }
+    }
+    
+    function update_stats()
+    {
+        @include WA_ROOTDIR . '/includes/functions.stats.php';
+        
+        if( function_exists('update_stats') )
+        {
+            update_stats($this->listdata);
-        }
-    }

trunk/includes/engine_send.php

@@ -165 +165 @@
-    // On récupère les infos sur les abonnés destinataires
-    //
-.abo_
+l.
-        FROM " . ABONNES_TABLE . " AS a
-
-
+
-                AND al.liste_id  = $listdata[liste_id]
-                AND al.confirmed = " . SUBSCRIBE_CONFIRMED . "
@@ -294 +293 @@
-                        'abo_pseudo' => '',
-                        'abo_email'  => $address,
-abo_
+
-                        'abo_id'     => -1
-                    ));
@@ -305 +304 @@
-                        'abo_pseudo' => '',
-                        'abo_email'  => $address,
-abo_
+
-                        'abo_id'     => -1
-                    ));
@@ -387 +386 @@
-                {
-                    $tags_replace = array_merge($tags_replace, array(
-abo_
+
-                        'WA_EMAIL' => rawurlencode($row['abo_email'])
-                    ));
@@ -460 +459 @@
-        $sql = "SELECT COUNT(*) AS num_dest, al.send
-            FROM " . ABO_LISTE_TABLE . " AS al
-
-
+
-                    AND a.abo_status = " . ABO_ACTIF . "
-            WHERE al.liste_id    = $listdata[liste_id]
@@ -583 +581 @@
-        else
-        {
-
-
+
-            
-            $link = array(

trunk/includes/functions.validate.php

@@ -29 +29 @@
-
-define('FUNCTIONS_VALIDATE_INC', true);
-
-/**
- * check_email()
- * 
- * Vérification de l'email
- * 
- * @param string  $email   Email à vérifier
- * @param integer $liste   Id de la liste concernée
- * @param string  $action  Action en cours
- * 
- * @return array
- */
-function check_email($email, $liste = 0, $action = '', $disable_check_mx = false)
-{
-    global $db, $nl_config, $lang;
-    
-    if( !class_exists('Mailer') )
-    {
-        require WAMAILER_DIR . '/class.mailer.php';
-    }
-    
-    //
-    // Vérification syntaxique de l'email
-    //
-    if( Mailer::validate_email($email) == false )
-    {
-        return array('error' => true, 'message' => $lang['Message']['Invalid_email']);
-    }
-    
-    $sql = 'SELECT ban_email FROM ' . BANLIST_TABLE . ' 
-        WHERE liste_id = ' . $liste;
-    if( $result = $db->query($sql) )
-    {
-        while( $row = $db->fetch_array($result) )
-        {
-            if( preg_match('/\b' . str_replace('*', '.*?', $row['ban_email']) . '\b/i', $email) )
-            {
-                return array('error' => true, 'message' => $lang['Message']['Email_banned']);
-            }
-        }
-    }
-    
-    $abodata   = array();
-    $sql_email = $db->escape(strtolower($email));
-    
-    $sql = "SELECT a.abo_id, a.abo_pseudo, a.abo_pwd, a.abo_email, a.abo_lang, a.abo_register_key, a.abo_register_date,
-            a.abo_status, al.format, al.confirmed, al.register_date
-        FROM " . ABONNES_TABLE . " AS a
-            LEFT JOIN " . ABO_LISTE_TABLE . " AS al ON al.abo_id = a.abo_id
-                AND al.liste_id = $liste
-        WHERE LOWER(a.abo_email) = '$sql_email'";
-    if( !($result = $db->query($sql)) )
-    {
-        trigger_error('Impossible de tester les tables d\'inscriptions', ERROR);
-    }
-    
-    if( $abodata = $db->fetch_array($result) )
-    {
-        if( isset($abodata['confirmed']) )
-        {
-            if( $action == 'inscription' && $abodata['confirmed'] == SUBSCRIBE_CONFIRMED )
-            {
-                return array('error' => true, 'message' => $lang['Message']['Allready_reg']);
-            }
-            else if( $action == 'confirmation' && $abodata['confirmed'] == SUBSCRIBE_CONFIRMED )
-            {
-                return array('error' => true, 'message' => $lang['Message']['Allready_confirm']);
-            }
-        }
-        else if( $action != 'inscription' )
-        {
-            return array('error' => true, 'message' => $lang['Message']['Unknown_email']);
-        }
-    }
-    else if( $action != 'inscription' )
-    {
-        return array('error' => true, 'message' => $lang['Message']['Unknown_email']);
-    }
-    
-    if( !$disable_check_mx && $nl_config['check_email_mx'] && $abodata === false )
-    {
-        //
-        // Vérification de l'existence d'un Mail eXchanger sur le domaine de l'email, 
-        // et vérification de l'existence du compte associé (La vérification de l'existence du 
-        // compte n'est toutefois pas infaillible, les serveurs smtp refusant parfois le relaying, 
-        // c'est à dire de traiter les demandes émanant d'un entité extérieure à leur réseau, et 
-        // pour une adresse email extérieure à ce réseau)
-        //
-        $mailer = new Mailer();
-        $mailer->smtp_path = WAMAILER_DIR . '/';
-        
-        if( $mailer->validate_email_mx($email) == false )
-        {
-            return array('error' => true, 'message' => $lang['Message']['Unrecognized_email']);
-        }
-    }
-    
-    return array('error' => false, 'abodata' => $abodata);
-}
-
-/**

trunk/language/lang_francais.php

@@ -249 +249 @@
-$lang['Message']['Upload_error_7']          = "Échec de l'écriture du fichier sur le disque";
-$lang['Message']['Invalid_filename']        = "Nom de fichier non valide";
+$lang['Message']['Invalid_action']          = "Action non valide";
-$lang['Message']['Invalid_ext']             = "Cette extension de fichier a été interdite";
-$lang['Message']['weight_too_big']          = "Le poids total des fichiers joints excède le maximum autorisé, il ne vous reste que %.2f octets de libre";

trunk/newsletter.php

@@ -62 +62 @@
-}
-
+$action  = ( !empty($_REQUEST['action']) ) ? trim($_REQUEST['action']) : '';
+$email   = ( !empty($_REQUEST['email']) ) ? trim($_REQUEST['email']) : '';
-$message = '';
+$code    = '';
+$liste   = ( isset($_REQUEST['liste']) ) ? intval($_REQUEST['liste']) : 0;
-
-
-
+
-{
-{$varname} = ( !empty($_REQUEST[$varname]) ) ? $_REQUEST[$varname] : ''
+code = $match[1]
-}
-
@@ -73 +76 @@
-// Compatibilité avec les version < 2.3.x
-//
-if(
+else if( !empty($action) && !empty($email) &&
-{
-    $code = substr($code, 0, 20);
-}
-
-$action != ''
+!empty($action) || !empty($code)
-{
-
-
-
+
+
+
+
+
+
+
+
+
-    {
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
-        {
-
-
-
-
-
-
-
-