织梦侠
WWW.2L3.NET

升级到dedecms 5.7文档关键词代码部分细节警示

织梦侠主站从dedecms 5.6 升级到5.7, 两周之后百度权重直接消失,还被惩罚了 。找了几天总算是发现到了问题,有个细节地方忘记了,文章内的描文本成全站所有文章都有,部分页还有错误。  原来dedecms 升级把 文档关键词换站全站了,害死人!

1. 原本的dedecms是文章内描文本必须是设定的关键词与文档关键词相应对称,两边有才出现描文本。
代码:
  1. /**
  2.          * 高亮问题修正, 排除alt title <a></a>直接的字符替换
  3.          *
  4.          * @param string $kw
  5.          * @param string $body
  6.          * @return string
  7.          */
  8.         function ReplaceKeyword($kw,&$body)
  9.         {
  10.                 global $cfg_cmspath;
  11.                 $maxkey = 5;
  12.                 $kws = explode(",",trim($kw));        //以分好为间隔符
  13.                 $i=0;
  14.                 $karr = $kaarr = $GLOBALS['replaced'] = array();
  15.  
  16.                 //暂时屏蔽超链接
  17.                 $body = preg_replace("/(<a(.*))(>)(.*)(<)(\/a>)/isU", '\\1-]-\\4-[-\\6', $body);
  18.  
  19.                 foreach($kws as $k)
  20.                 {
  21.                         $k = trim($k);
  22.                         if($k!="")
  23.                         {
  24.                                 if($i > $maxkey)
  25.                                 {
  26.                                         break;
  27.                                 }
  28.                                 $myrow = $this->dsql->GetOne("select * from zuimoban_keywords where keyword='$k' And rpurl<>'' ");
  29.                                 if(is_array($myrow))
  30.                                 {
  31.                                         $karr[] = $k;
  32.                                         $GLOBALS['replaced'][$k] = 0;
  33.                                         $kaarr[] = "<a href='{$myrow['rpurl']}'><u>$k</u></a>";
  34.                                 }
  35.                                 $i++;
  36.                         }
  37.                 }
  38.                 $body = preg_replace("/(^|>)([^<]+)(?=<|$)/sUe", "_highlight('\\2', \$karr, \$kaarr, '\\1')", $body);
  39.  
  40.                 //恢复超链接
  41.                 $body = preg_replace("/(<a(.*))-\]-(.*)-\[-(\/a>)/isU", '\\1>\\3<\\4', $body);
  42.                 return $body;
  43.         }
  44.  
2.  dedecms 5.7  直接就修改成不管文章关键词有没有,只要文档内关键词存在,文章展示全部描文本。更可怕的是dedecms有bug,若同个文章有两个关键词,有包含关系,如 ecshop 与ecshop模板,描文本就加的混乱,页面还显示出代码来。  这不轻易发现 网站出现大量的描文本,不K站才怪!  
代码部分:
  1. function ReplaceKeyword($kw,&$body)
  2.     {
  3.         global $cfg_cmspath;
  4.         $maxkey = 5;
  5.         $kws = explode(",",trim($kw));    //以分好为间隔符
  6.         $i=0;
  7.         $karr = $kaarr = $GLOBALS['replaced'] = array();
  8.  
  9.         //暂时屏蔽超链接
  10.         $body = preg_replace("#(<a(.*))(>)(.*)(<)(\/a>)#isU", '\\1-]-\\4-[-\\6', $body);
  11.  
  12.         /*
  13.         foreach($kws as $k)
  14.         {
  15.             $k = trim($k);
  16.             if($k!="")
  17.             {
  18.                 if($i > $maxkey)
  19.                 {
  20.                     break;
  21.                 }
  22.                 $myrow = $this->dsql->GetOne("SELECT * FROM zuimoban_keywords WHERE keyword='$k' AND rpurl<>'' ");
  23.                 if(is_array($myrow))
  24.                 {
  25.                     $karr[] = $k;
  26.                     $GLOBALS['replaced'][$k] = 0;
  27.                     $kaarr[] = "<a href='{$myrow['rpurl']}'><u>$k</u></a>";
  28.                 }
  29.                 $i++;
  30.             }
  31.         }
  32.         */
  33.         $query = "SELECT * FROM zuimoban_keywords WHERE rpurl<>'' ORDER BY rank DESC"; 
  34.         $this->dsql->SetQuery($query);
  35.         $this->dsql->Execute();
  36.         while($row = $this->dsql->GetArray())
  37.         {
  38.             $key = trim($row['keyword']);
  39.             $key_url=trim($row['rpurl']);
  40.             $karr[] = $key;
  41.             $kaarr[] = "<a href='$key_url' target='_blank'><u>$key</u></a>";
  42.         }
  43.  
  44.         // 这里可能会有错误
  45.         $body = @preg_replace("#(^|>)([^<]+)(?=<|$)#sUe", "_highlight('\\2', \$karr, \$kaarr, '\\1')", $body);
  46.  
  47.         //恢复超链接
  48.         $body = preg_replace("#(<a(.*))-\]-(.*)-\[-(\/a>)#isU", '\\1>\\3<\\4', $body);
  49.         return $body;
  50.     }
  51.  
以上代码里居然有注释: 这里可能会有错误。 作为程序代码来说,还可能有错误? 不可预见? 明明已经预见,为什么不修正呢? 不负责任!

从5.6 到5.7升级,如果不注意,很容易忽略到这个问题,进而蜘蛛来抓取,感觉页面又变化了,描写文本还有作弊嫌疑。K站呀,减权重呀!DEDECMS也不特别提醒下!   不是DEDECMS经常出漏洞,还真不想升级!  这次倒霉到了!  你呢?
转载请注明文章来自织梦侠[秩名]作者的-升级到dedecms 5.7文档关键词代码部分细节警示
联系作者
取消
这个作者很懒什么也没留下!
打赏作者
取消

本文作者无偿奉献,就打赏给我们织梦侠吧!

扫码支付
扫码打赏,建议金额1-10元

打开支付宝扫一扫,即可进行扫码打赏哦

提醒:打赏金额将直接进入对方账号,无法退款,请您谨慎操作。