Исправляем ошибку меню для ссылок из url_rewrite.php. Хак.

Доброго всем времени суток!
Есть проблема, я описывал ее здесь Проблема активного меню с custom_rewrite
Вкратце:
создаем в меню пункт Вход, ставим тип "ссылка", сама ссылка вот такая /login.
Теперь у нас по клику по пункту меню Вход — можно зайти на сайт.
Кликаем по ней...
Но пункт меню ВХОД не стал активным!
Налицо ошибка в разборе меню, т.е. ссылки вида /login, /passremind b прочие не обрабатываются!
Вы скажете да зачем нам это нужно, у нас эти ссылки висят отдельно. Да, на самом деле, смысла вносить изменения ради этих ссылок нет никакого.
Смысл есть, если мы будем использовать custom_rewrite для своих ~~низменных~~ важных целей. Например, сделать так, чтобы урл контента в каком-либо компоненте был не пятнадцать слов а два

Итак поехали.

сегодня правим файл cms.php

добавляем в начале файла строку

    private         $rules;

там закешируем правила rewrite

берем функцию loadMenuStruct (загрузка меню в память)
меняем цикл

 while ($item = $inDB->fetch_assoc($result)){
            $this->menu_struct[$item['id']] = $item;
        }

на

 while ($item = $inDB->fetch_assoc($result)){
            $this->menu_struct[$item['id']] = $item;
            $realink='/'.$this->detectorMenuURI($item['link']);
            $this->menu_struct[$item['id']]['realink'] = $realink;
        }

это мы посчитали "настоящую" ссылку в меню

добавляем новую функцию -

   /////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
    private function detectorMenuURI($muri){
 
        $uri    = $muri;
        $uri    = ltrim($uri, '/');
        $rules  = array();
 
        $folder = rtrim($uri, '/');
 
        if (in_array($folder, array('admin', 'install', 'migrate'))) { return; }
        if (!$this->rules) {
          if(file_exists(PATH.'/url_rewrite.php')) {
              //подключаем список rewrite-правил
               $this->includeFile('url_rewrite.php');
               if(function_exists('rewrite_rules')){
                   //получаем правила
                   $rules = rewrite_rules();
               }
           }
 
           if(file_exists(PATH.'/custom_rewrite.php')) {
               //подключаем список пользовательских rewrite-правил
               $this->includeFile('custom_rewrite.php');
               if(function_exists('custom_rewrite_rules')){
                   //добавляем к полученным ранее правилам пользовательские
                   $rules = array_merge($rules, custom_rewrite_rules());
               }
           }
           if ($rules){  $this->rules=$rules;}
        }
        else {
          $rules=$this->rules;
        }
 
        $found = false;
 
        if ($rules){
 
            //перебираем правила
            foreach($rules as $rule_id=>$rule) {
 
                //небольшая валидация правила
                if (!$rule['source'] || !$rule['target'] || !$rule['action']) { continue; }
                //проверяем совпадение выражения source с текущим uri
                if (preg_match($rule['source'], $uri, $matches)){
                    //перебираем совпавшие сегменты и добавляем их в target
                    //чтобы сохранить параметры из $uri в новом адресе
                    foreach($matches as $key=>$value){
                        if (!$key) { continue; }
                        if (strstr($rule['target'], '{'.$key.'}')){
                            $rule['target'] = str_replace('{'.$key.'}', $value, $rule['target']);
                        }
                    }
 
                    //действие по-умолчанию: rewrite
                    if (!$rule['action']) { $rule['action'] = 'rewrite'; }
 
                    //выполняем действие
                    switch($rule['action']){
                        case 'rewrite'      : $uri = $rule['target']; $found = true; break;
                        case 'redirect'     : $this->redirect($rule['target']); break;
                        case 'redirect-301' : $this->redirect($rule['target'], '301'); break;
                        case 'alias'        : $this->includeFile($rule['target']); $this->halt();break;
                    }
 
                }
 
                if ($found) { break; }
 
            }
        }
 
        return $uri;
 
    }
 
    /////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

А теперь в функции menuID() меняем цикл foreach

foreach($menu as $item){
 
            if (!$item['link']) { continue; }
 
            //полное совпадение ссылки и адреса?
            if ($uri == $item['link']){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                break;
            }
            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = substr($uri, 0, strlen($item['link']));
            if ($uri_first_part == $item['link']){
                $menuid = $item['id'];
                break;
            }
        }

на

foreach($menu as $item){
 
            if (!$item['link']) { continue; }
 
            //полное совпадение ссылки и адреса?
            if ($uri == $item['link']){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                break;
            }
////////////////  Определяем правильную ссылку для пункта меню
            if ($uri == $item['realink']){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                break;
            }
/////////////////////////////////////////////////////////////////////////////
            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = substr($uri, 0, strlen($item['link']));
            if ($uri_first_part == $item['link']){
                $menuid = $item['id'];
                break;
            }
////////////////  Определяем правильную ссылку для пункта меню  /////////////
            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = substr($uri, 0, strlen($item['realink']));
            if ($uri_first_part == $item['realink']){
                $menuid = $item['id'];
                break;
            }
//////////////////////////////////////////////////////////////////////////////
        }

теперь меню работает правильно
делалось под 1.8 для 1.9 думаю будет аналогично
решение не претендует на красоту, но общая идея понятна
а по поводу "зачем все это нужно" я расскажу в следующих постах

Компонент Инвайтер v.1.5 stable

Замена sendmail для Denwer

custom_rewrite url_rewrite

18 комментариев

universe 14 лет назад #

"Например, сделать так, чтобы урл контента в каком-либо компоненте был не пятнадцать слов а два"
не понял про что это=)

Крот 14 лет назад #

здесь очень много желающих чтобы ссылка до какого-либо объекта в каком-либо компоненте была не
http://сайт.ру/категория1/категория2/категория3/.../объект.html
а просто
http://сайт.ру/категория/объект.html вне зависимости от вложенности
это просто пример, через custom_rewrite много интересного сделать можно

universe 14 лет назад #

тоесть что то общего в нем есть с http://instantcms.ru/blogs/moi-blog-360/komponent-i-plagin-seo-pages.html ?

Крот 14 лет назад #

не могу сказать, я еще не смотрел SEO компонент, может просто совпало так))

Крот 14 лет назад #

но подозреваю, что есть smile

-3

Алексей Т 14 лет назад #

Спасибо крот...

Крот 14 лет назад #

поправил функцию loadMenuStruct - была небольшая ошибка... если кто поставил, проверьте

Tarhun 14 лет назад #

на 1.9 не работает...

Tarhun 14 лет назад #

как отредактировать под 1.9?
После пункта "добавляем новую функцию"
Перестает вообще открываться сайт, происходит редирект - http://сайт.ру/сайт.ру/

BSB 14 лет назад #

Хм, я в растерянности... Поставил апач на голую убунту 12.04, поправил Override None на Override All в конфиге, убедился, что установлен mod_rewrite - у меня icms 1.9.1 по ссылке на /login вываливается с ошибкой 404.
Есть, правда, вероятность, что версию неполноценную выкачал (критерий был - utf-8). Кто-нить, дайте сцыль на .htaccess правильный? А то у меня такое впечатление, что в транках он левый.

Крот 14 лет назад #

для 1.9 надо
функция detectorMenuURI() - на самом деле это просто подправленная копия detectURI - сделано для ускорения работы
можно использовать оригинальный detectURI - но тогда при каждом клике будет пересчитываться список uri
соотв-но при изменении списка в custom_rewrite нужно как-то сбрасывать закэшированное значение

Код PHP:

 private function detectorMenuURI($muri){

     $uri    = $muri;
     $uri    = ltrim($uri, '/');
     $rules  = array();

     $folder = rtrim($uri, '/');

if (strstr($uri, "?") && !preg_match('/^admin\/(.*)/i', $uri) && !strstr($uri, 'go/url=')){
            $query_str = substr($uri, strpos($uri, "?")+1);
            $uri = substr($uri, 0, strpos($uri, "?"));
            parse_str($query_str, $temp_request);
			$_REQUEST = array_merge($_REQUEST, $temp_request);
        }

        if (in_array($folder, array('admin', 'install', 'migrate', 'index.php'))) { return; }
     
        //специальный хак для поиска по сайту, для совместимости со старыми шаблонами
        if (strstr($_SERVER['QUERY_STRING'], 'view=search')){ $uri = 'search'; }

if (!$this->rules) {
       if(file_exists(PATH.'/url_rewrite.php')) {
           //подключаем список rewrite-правил
            self::includeFile('url_rewrite.php');
            if(function_exists('rewrite_rules')){
                //получаем правила
                $rules = rewrite_rules();
            }
        }

        if(file_exists(PATH.'/custom_rewrite.php')) {
            //подключаем список пользовательских rewrite-правил
            self::includeFile('custom_rewrite.php');
            if(function_exists('custom_rewrite_rules')){
                //добавляем к полученным ранее правилам пользовательские
                $rules = array_merge($rules, custom_rewrite_rules());
            }
        }
        if ($rules){  $this->rules=$rules;}
     }
     else {
       $rules=$this->rules;
     }

     $found = false;

     if ($rules){

         //перебираем правила
         foreach($rules as $rule_id=>$rule) {

             //небольшая валидация правила
             if (!$rule['source'] || !$rule['target'] || !$rule['action']) { continue; }
             //проверяем совпадение выражения source с текущим uri
             if (preg_match($rule['source'], $uri, $matches)){
                 //перебираем совпавшие сегменты и добавляем их в target
                 //чтобы сохранить параметры из $uri в новом адресе
                 foreach($matches as $key=>$value){
                     if (!$key) { continue; }
                     if (strstr($rule['target'], '{'.$key.'}')){
                         $rule['target'] = str_replace('{'.$key.'}', $value, $rule['target']);
                     }
                 }

                 //действие по-умолчанию: rewrite
                 if (!$rule['action']) { $rule['action'] = 'rewrite'; }

                 //выполняем действие
                 switch($rule['action']){
                     case 'rewrite'      : $uri = $rule['target']; $found = true; break;
                     case 'redirect'     : $this->redirect($rule['target']); break;
                     case 'redirect-301' : $this->redirect($rule['target'], '301'); break;
                        case 'alias'        : self::includeFile($rule['target']); $this->halt();break;
                 }

             }

             if ($found) { break; }

         }
     }

     return $uri;
     
 }

menuID

Код PHP:

    public function menuId(){

        //если menu_id был определен ранее, то вернем и выйдем
        if ($this->menu_id) { return $this->menu_id; }

        $view       = self::request('view', 'str', '');
        
        if ($this->is_content){
            $uri = substr($this->uri, strlen('content/'));
        } else {
            $uri = $this->uri;
        }

        $uri        = '/'.$uri;

        //флаг, показывающий было совпадение URI и ссылки пунта меню
        //полным или частичным
        $is_strict  = false;

        //главная страница?
        $menuid     = ($uri == '/' ? 1 : 0);
        if ($menuid == 1) {
            $this->is_menu_id_strict = 1;
            return $menuid;
        }

        //перевернем массив меню чтобы перебирать от последнего пункта к первому
        $menu = array_reverse($this->menu_struct);

        //перебираем меню в поисках текущего пункта
        foreach($menu as $item){

            if (!$item['link']) { continue; }

			// uri с учетом имени хоста
			$full_uri = HOST . $uri;

            //полное совпадение ссылки и адреса?
            if ($uri == $item['link'] || $full_uri == $item['link']){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                break;
            }

////////////////  Определяем правильную ссылку для пункта меню
            if ($uri == $item['realink']){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                break;
            }

            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = substr($uri, 0, strlen($item['link']));
            if ($uri_first_part == $item['link']){
                $menuid = $item['id'];
                break;
            }
////////////////  Определяем правильную ссылку для пункта меню  /////////////
            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = substr($uri, 0, strlen($item['realink']));
            if ($uri_first_part == $item['realink']){
                $menuid = $item['id'];
                break;
            }
           
        }

        $this->menu_id              = $menuid;
        $this->is_menu_id_strict    = $is_strict;

        return $menuid;

    }

сравните со своим cms.php

Fuze 14 лет назад #

Более простое решение, файл core.php

После

Код PHP:

private         $uri;

вставляем

Код PHP:

private         $real_uri;

Далее в методе detectURI() перед

Код PHP:

 
        if ($rules){
            //перебираем правила

Вставляем

Код PHP:

$this->real_uri = $uri;

Далее в методе menuId() после

Код PHP:

$uri      = '/'.$uri;

вставляем

Код PHP:

$real_uri = '/'.$this->real_uri;

И далее условия по полному или частичному совпадению меняем на

Код PHP:

            //полное совпадение ссылки и адреса?
            if (in_array($item['link'], array($uri, $full_uri, $real_uri))){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                break;
            }
 
            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = mb_substr($uri, 0, mb_strlen($item['link']));
            $real_uri_first_part = mb_substr($real_uri, 0, mb_strlen($item['link']));
            if (in_array($item['link'], array($uri_first_part, $real_uri_first_part))){
                $menuid = $item['id'];
                break;
            }

Крот 14 лет назад #

посмотрел, да так проще
НО:
второе решение, также как и мое содержит один нюанс -
есть два пункта меню
1) /catalog/1001
2) /catalog

при клике на первое - активным становится второе. решается сначала перебором полных путей, а затем, если не найдено перебор частичных путей.
вот теперь меня метод menuID() устраивает полностью )))

Код PHP:

    /**
     * Возвращает ID текущего пункта меню
     * @return int
     */
    public function menuId(){
 
        //если menu_id был определен ранее, то вернем и выйдем
        if ($this->menu_id) { return $this->menu_id; }
 
        $view       = self::request('view', 'str', '');
 
        if ($this->is_content){
            $uri = substr($this->uri, strlen('content/'));
        } else {
            $uri = $this->uri;
        }
 
        $uri        = '/'.$uri;
 
        $real_uri = '/'.$this->real_uri;
        //флаг, показывающий было совпадение URI и ссылки пунта меню
        //полным или частичным
        $is_strict  = false;
 
        //главная страница?
        $menuid     = ($uri == '/' ? 1 : 0);
        if ($menuid == 1) {
            $this->is_menu_id_strict = 1;
            return $menuid;
        }
 
        //перевернем массив меню чтобы перебирать от последнего пункта к первому
        $menu = array_reverse($this->menu_struct);
        $fnd=false;
        //перебираем меню в поисках текущего пункта
        foreach($menu as $item){
 
            if (!$item['link']) { continue; }
 
			// uri с учетом имени хоста
			$full_uri = HOST . $uri;
 
           if (in_array($item['link'], array($uri, $full_uri, $real_uri))){
                $menuid = $item['id'];
                $is_strict = true; //полное совпадение
                $fnd=true;
                break;
            }
       } 
        //перебираем меню в поисках текущего пункта
      if (!$fnd)
        foreach($menu as $item){
 
            if (!$item['link']) { continue; }
 
			// uri с учетом имени хоста
			$full_uri = HOST . $uri;
 
            //частичное совпадение ссылки и адреса (по началу строки)?
            $uri_first_part = mb_substr($uri, 0, mb_strlen($item['link']));
            $real_uri_first_part = mb_substr($real_uri, 0, mb_strlen($item['link']));
            if (in_array($item['link'], array($uri_first_part, $real_uri_first_part))){
                $menuid = $item['id'];
                break;
            }
        }
 
        $this->menu_id              = $menuid;
        $this->is_menu_id_strict    = $is_strict;
 
        return $menuid;
 
    }

Tarhun 14 лет назад #

Firefox определил, что сервер перенаправляет запрос на этот адрес таким образом, что он никогда не завершится.

Можете скинуть сам файл cms.php для 1.9?

Олег Васильевич я 14 лет назад #

И я был бы весьма благодарен за этот файл.
Спасибо!

Bubble Gumoff 14 лет назад #

А как можно без rewrite разбить seolink, чтобы формировались ссылки так
ссылка статьи была host/statya
ссылка категории host/kategoriya
ссылка подкатегории host/podkategoriya

wildbeez 14 лет назад #

пишу в надежде, что ответят

Крот 14 лет назад #

в 1.10 ошибки больше нет)

Исправляем ошибку меню для ссылок из url_rewrite.php. Хак.

universe 14 лет назад #

Крот 14 лет назад #

universe 14 лет назад #

Крот 14 лет назад #

Крот 14 лет назад #

Алексей Т 14 лет назад #

Крот 14 лет назад #

Tarhun 14 лет назад #

Tarhun 14 лет назад #

BSB 14 лет назад #

Крот 14 лет назад #

Fuze 14 лет назад #

Крот 14 лет назад #

Tarhun 14 лет назад #

Олег Васильевич я 14 лет назад #

Bubble Gumoff 14 лет назад #

wildbeez 14 лет назад #

Крот 14 лет назад #

Еще от автора

Инвайтер 1.9 для 1.10.6

Инстант "по взрослому". Часть 2. Авторизация. Аякс. v1.10

Инстант "по взрослому". Часть 1. Авторизация. Счетчик неверных входов. v1.10

InstantCMS Team

О проекте

Поддержка

Дополнения

Исправляем ошибку меню для ссылок из url_rewrite.php. Хак.

universe 14 лет назад #

Крот 14 лет назад #

universe 14 лет назад #

Крот 14 лет назад #

Крот 14 лет назад #

Алексей Т 14 лет назад #

Крот 14 лет назад #

Tarhun 14 лет назад #

Tarhun 14 лет назад #

BSB 14 лет назад #

Крот 14 лет назад #

Fuze 14 лет назад #

Крот 14 лет назад #

Tarhun 14 лет назад #

Олег Васильевич я 14 лет назад #

Bubble Gumoff 14 лет назад #

*wildbeez* 14 лет назад #

Крот 14 лет назад #

Еще от автора

Инвайтер 1.9 для 1.10.6

Инстант "по взрослому". Часть 2. Авторизация. Аякс. v1.10

Инстант "по взрослому". Часть 1. Авторизация. Счетчик неверных входов. v1.10

InstantCMS Team

О проекте

Поддержка

Дополнения

wildbeez 14 лет назад #