Changeset 81379 in spip-zone


Ignore:
Timestamp:
Mar 14, 2014, 4:31:01 PM (5 years ago)
Author:
cedric@…
Message:

Report de r81375 : Detection de collisions d'URLs :

  • on ne suffixe que si l'url identique concerne un autre objet (peut etre une url identique du même objet avec un id_parent non nul)
  • quand on veut piquer une ancienne URL d'un autre objet, il faut que celle plus recente soit bien differente (sinon on arrivait a piquer une URL valide au pretexte que l'objet en avait une autre identique avec un id_parent non nul)
  • on log mieux les reattribution et reactualisation
  • quand on suffixe l'URL on reappelle recursivement la fonction url_insert pour refaire tout le test de collision proprement
File:
1 edited

Legend:

Unmodified
Added
Removed
  • _core_/plugins/urls_etendues/action/editer_url.php

    r79783 r81379  
    9696
    9797        // Si l'insertion echoue, c'est une violation d'unicite.
    98         $where_urllike = 'url LIKE '.url_sql_quote_like($set['url']);
    99         $where_thisurl = $where_urllike." AND id_parent=".intval($set['id_parent']);
     98        $where_urllike = 'url LIKE '.url_sql_quote_like($set['url'])." AND NOT(type=".sql_quote($set['type'])." AND id_objet=".intval($set['id_objet']).")";
     99        $where_thisurl = $where_urllike.($has_parent?" AND id_parent=".intval($set['id_parent']):"");
    100100        if (
    101101                // si pas de parent defini, il faut que cette url soit unique, independamment de id_parent
     
    108108                        (!is_dir(_DIR_RACINE.$set['url']) AND !file_exists(_DIR_RACINE.$set['url']))
    109109                        // un vieux url
    110                         AND $vieux = sql_fetsel('*', 'spip_urls', $where_thisurl)
     110                        AND $vieux = sql_fetsel('*', 'spip_urls', $where_thisurl,'','perma DESC')
    111111                        // qui n'est pas permanente
    112112                        AND !$vieux['perma']
     
    114114                  AND $courant = sql_fetsel('*', 'spip_urls',
    115115                          'type='.sql_quote($vieux['type']).' AND id_objet='.sql_quote($vieux['id_objet'])
     116                          .' AND url<>'.sql_quote($set['url'])
    116117                          .' AND date>'.sql_quote($vieux['date']), '', 'date DESC', 1)
    117118                  ) {
     
    125126                        sql_updateq('spip_urls', $set, $where_thisurl);
    126127                        sql_updateq('spip_urls', array('date' => date('Y-m-d H:i:s')), $where_thisurl);
     128                        spip_log("reattribue url ".$vieux['url']
     129                                ." de ".$vieux['type']."#".$vieux['id_objet']." (parent ".$vieux['id_parent'].")"
     130                                ." A ".$set['type']."#".$set['id_objet']." (parent ".$set['id_parent'].")","urls"._LOG_INFO_IMPORTANTE);
    127131                }
    128132
     
    136140                        // il peut etre du a un changement de casse de l'url simplement
    137141                        // pour ce cas, on reecrit systematiquement l'url en plus d'actualiser la date
    138                         do {
    139                                 $where = "type=".sql_quote($set['type'])
    140                                                                  ." AND id_objet=".intval($set['id_objet'])
    141                                                                  ." AND id_parent=".intval($set['id_parent'])
    142                                                                  ." AND url LIKE ";
    143                                 if (
    144                                         !is_dir(_DIR_RACINE.$set['url']) AND !file_exists(_DIR_RACINE.$set['url'])
    145                                         AND sql_countsel('spip_urls', $where  .url_sql_quote_like($set['url']))) {
    146                                         sql_updateq('spip_urls', array('url'=>$set['url'], 'date' => date('Y-m-d H:i:s')), $where  .url_sql_quote_like($set['url']));
    147                                         spip_log("reordonne ".$set['type']." ".$set['id_objet']);
     142                        $where = "type=".sql_quote($set['type'])
     143                                                         ." AND id_objet=".intval($set['id_objet'])
     144                                                         ." AND id_parent=".intval($set['id_parent'])
     145                                                         ." AND url LIKE ";
     146                        if (
     147                                !is_dir(_DIR_RACINE.$set['url']) AND !file_exists(_DIR_RACINE.$set['url'])
     148                                AND sql_countsel('spip_urls', $where  .url_sql_quote_like($set['url']))) {
     149                                sql_updateq('spip_urls', array('url'=>$set['url'], 'date' => date('Y-m-d H:i:s')), $where  .url_sql_quote_like($set['url']));
     150                                spip_log("reordonne ".$set['type']." ".$set['id_objet'],"urls");
     151                                $redate = false;
     152                        }
     153                        else {
     154                                $set['url'] .= $separateur.$set['id_objet'];
     155                                if (strlen($set['url']) > 200)
     156                                        //serveur out ? retourner au mieux
     157                                        return false;
     158                                elseif (sql_countsel('spip_urls', $where . url_sql_quote_like($set['url']))) {
     159                                        sql_updateq('spip_urls', array('url'=>$set['url'], 'date' => date('Y-m-d H:i:s')), $where .url_sql_quote_like($set['url']));
    148160                                        $redate = false;
    149                                         continue;
    150161                                }
    151162                                else {
    152                                         $set['url'] .= $separateur.$set['id_objet'];
    153                                         if (strlen($set['url']) > 200)
    154                                                 //serveur out ? retourner au mieux
    155                                                 return false;
    156                                         elseif (sql_countsel('spip_urls', $where . url_sql_quote_like($set['url']))) {
    157                                                 sql_updateq('spip_urls', array('url'=>$set['url'], 'date' => date('Y-m-d H:i:s')), $where .url_sql_quote_like($set['url']));
    158                                                 $redate = false;
    159                                                 continue;
    160                                         }
     163                                        // remettre id_parent et perma comme il faut si besoin
     164                                        if (!$has_parent) unset($set['id_parent']);
     165                                        if ($perma) $set['perma'] = true;
     166                                        return url_insert($set, $confirmer, $separateur);
    161167                                }
    162                         } while ($redate AND @sql_insertq('spip_urls', $set) <= 0);
    163 
     168                        }
    164169                }
    165170        }
     
    173178                sql_update('spip_urls', array('perma' => "($where_thisurl)"), "type=".sql_quote($set['type'])." AND id_objet=".intval($set['id_objet']));
    174179       
    175         spip_log("Creation de l'url propre '" . $set['url'] . "' pour ".$set['type']." ".$set['id_objet']." (parent ".$set['id_parent']." perma $perma)","urls");
     180        spip_log("Creation de l'url propre '" . $set['url'] . "' pour ".$set['type']." ".$set['id_objet']." (parent ".$set['id_parent']." perma ".($perma?"1":"0").")","urls");
    176181        return true;
    177182}
Note: See TracChangeset for help on using the changeset viewer.