Changeset 90526 in spip-zone


Ignore:
Timestamp:
Jun 25, 2015, 4:50:57 PM (6 years ago)
Author:
cedric@…
Message:

eviter une requete inutile si pas id_panier en session, echappements SQL dans la requete pour simplifier la lecture et l'analyse du code, date PHP au lieu de NOW()

File:
1 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/paniers/trunk/action/remplir_panier.php

    r90486 r90526  
    66/**
    77 * Remplir un panier avec un objet quelconque
    8  * @param unknown_type $arg
    9  * @return unknown_type
     8 * @param string $arg
    109 */
    1110function action_remplir_panier_dist($arg=null) {
     
    2221        // Il faut cherche le panier du visiteur en cours
    2322        include_spip('inc/session');
    24         $id_panier = session_get('id_panier');
    25        
    26         //est-ce que le panier est bien en base
    27         $id_panier_base = intval(sql_getfetsel(
    28                         'id_panier',
    29                         'spip_paniers',
    30                         array(
    31                                 'id_panier = '.sql_quote($id_panier),
    32                                 'statut = '.sql_quote('encours')
    33                         )
    34         ));
     23        $id_panier_base = 0;
     24        if ($id_panier = session_get('id_panier')){
     25                //est-ce que le panier est bien en base
     26                $id_panier_base = intval(sql_getfetsel(
     27                                'id_panier',
     28                                'spip_paniers',
     29                                array(
     30                                        'id_panier = '.intval($id_panier),
     31                                        'statut = '.sql_quote('encours')
     32                                )
     33                ));
     34        }
    3535       
    3636        // S'il n'y a pas de panier, on le crée
     
    4949                        'spip_paniers_liens',
    5050                        array(
    51                                 'id_panier = '.$id_panier,
     51                                'id_panier = '.intval($id_panier),
    5252                                'objet = '.sql_quote($objet),
    53                                 'id_objet = '.$id_objet
     53                                'id_objet = '.intval($id_objet)
    5454                        )
    5555                ));
     
    5959                                'spip_paniers_liens',
    6060                                array('quantite' => $quantite_deja + $quantite),
    61                                 'id_panier = '.$id_panier.' and objet = '.sql_quote($objet).' and id_objet = '.$id_objet
     61                                'id_panier = '.intval($id_panier).' and objet = '.sql_quote($objet).' and id_objet = '.intval($id_objet)
    6262                        );
    6363                }
     
    7878                sql_updateq(
    7979                        'spip_paniers',
    80                         array('date'=>'NOW()'),
    81                         'id_panier = '.$id_panier
     80                        array('date'=>date('Y-m-d H:i:s')),
     81                        'id_panier = '.intval($id_panier)
    8282                );
    8383        }
Note: See TracChangeset for help on using the changeset viewer.