Давайте вместе красиво в ocStore 2.3 поставим seo pro :)

[AlexDW]

а нафига его патчить, если речь про сборку? :ugeek:

сразу и менять

Змінено 3 жовтня 2016 користувачем AlexDW

[chukcha]

потому что тогда нужно сделать следующее

seo_pro

     public function index() {

        // Add rewrite to url class
        if ($this->config->get('config_seo_url') && $this->config->get('seo_pro_status')) {
            $this->url->addRewrite($this);
        } else {
                $action = new Action('startup/seo_url');
                $action->execute($this->registry);

        }

[chukcha]

А че, дифайны в конфиге уже отменили? :)

ага..

 

Но хочется ж красиво!!!

[Otvet]

а нафига его патчить, если речь про сборку? :ugeek:

сразу и менять

потому что так можно вкл/выкл делать

[sv2109]

Можно попробовать так:

 

в  system/framework.php мы не можем подменить 'pre_action' c 'startup/seo_url' потому что на этом этапе еще нету полноценного конфига и нельзя проверить какой тип seo_url_type выбран в админке

 

поэтому делать это нужно в фронт контроллере на этапе вызова 'pre_action's потому что тогда уже есть полноценный конфиг. 

В файле system/engine/front.php

после

foreach ($this->pre_action as $pre_action) {

 

добавить:

if ($pre_action->getId() == 'startup/seo_url' && $this->registry->get('config')->get('config_seo_url_type') == 'seo_pro') {

  $pre_action = new Action('startup/seo_pro');

}

 

как-то так пробовать. 

[chukcha]

Есть один нюанс, который может все сломать

 

Это приоритет index преобразователя

 

вот как доп решение

 

 

seo_url не самая сильная сторона

 

Т.е. если rewrite еще как-то можно поставить в очередь, то index - никак

 

Поэтому..

 

только идея

 

        if ($this->config->get('config_seo_url') && $this->config->get('sla_seo_pro_status')) {
            $this->url->addRewrite($this);

 

$action = new Action('startup/seo_url');

$this->url->addRewrite($action);

 

        } else {
$action = new Action('startup/seo_url');
$action->execute($this->registry);

 

        }

 

и..

 

 

Т.е. все идея заключается в том чтобы построить очередь преобразователей

 

Мы зашли с _route_

Если нашли, то дальше не вызывается seo_url

то выполнили метод $this->load->controller('startup/seo_url');

 

Но если есть метод addRwrite, но нет removeRewrite.

 

Вот я и говорю, что это недороутер
 

[SooR]

Не пойдет, к сожалению

Это можно т олько встроить сразу в двиг.. тогда подойдет

 

Хорошо.

А так?

startup/seo_url.php

 

search

$this->url->addRewrite($this);

replace

if ($this->config->get('config_seo_url_type') == 'seo_pro') {
  $this->url->addRewrite($this->load->controller('startup/seo_pro/rewriteWrapper'));
} else {
  $this->url->addRewrite($this);
}

startup/seo_pro/rewriteWrapper

return $this;

Ничего из этого не проверял.

[AlexDW]

потому что так можно вкл/выкл делать

имелось ввиду что можно условие сразу в файле прописать

речь же не про отдельный модуль, а про сборку

[AlexDW]

А так? startup/seo_url.php

 

вариант с патчем через seo_url - еще более хреновый, чем с запросом к БД, имхо

встречаются некоторые чЮдо-мега-сео дополнения, которые не то что его патчат - целиком заменяют :ugeek:

 

больше склоняюсь к варианту sv2109

[andrus]

Без разницы кто там будет и что ( seo-pro или seo-url) но самое главное, это просьба №1: уберите вы наконец плодирование несуразных дублей, которые поисковики спустя время, таки находят на сайте и тащат их в индекс:

Вот страница на категорию  -     http://demo.myopencart.com/desktops/mac
А вот дубль той же страницы, но в конце чёртов производитель: http://demo.myopencart.com/desktops/mac/apple

в Яндексе и та, и та страницы доступны, он взял и закинул в индекс себе. дальше идём... см. в адресной строке внимательно!

Вот страница статья/информация -    http://demo.myopencart.com/delivery
А вот дубль категории, но с урлом от стр. информация: http://demo.myopencart.com/delivery/desktops/mac
                                                                                             http://demo.myopencart.com/delivery/mac
____________________________

Т.е. поэкспериментируйте у себя на рабочих сайтах:
1. Попробуйте подставить алиас производителя в конец любого урла категории, и вы получите доступный для индекса дубль категории.
2. Попробуйте подставить алиас категории в конце урла статьи любой, и вы получите дубль категории. 

Поднимал данный вопрос ранее, но многие ответили что это только у вас такое, так не бывает. Фиг с ним, удалил я всё через панель Вебмастер Гугл и Яндекс. Но через время снова эти несуразые варианты страниц, нахожу в индексе поисковиков. Помогите избавится от этого недуга, когда нет никакого 301 редиректа но одно что-то... пожалуйста!

Змінено 3 жовтня 2016 користувачем andrus

[chukcha]

Поднимал данный вопрос ранее, но многие ответили что это только у вас такое, так не бывает.

 

Да, только у вас

 

http://demo2302.slasoft.kharkov.ua/delivery/desktops/mac

Что?

[andrus]

? )) 
это ссылки из старой темы? Здесь вот вроде от этого примера всё - http://demo.myopencart.com
Сути не меняет. Своими силами, помню так и не смог склеить с помощью 301, то что тут выше примером представил (см. мой коммент выше). Удалял вручную в панели GWT. Но недавно снова появилось, вот чего-то решил сообщить специалистам, дабы обратили внимание на этот... немного баг. 

Змінено 3 жовтня 2016 користувачем andrus

[chukcha]

Без разницы кто там будет и что ( seo-pro или seo-url) но самое главное, это просьба №1: уберите вы наконец плодирование несуразных дублей, которые поисковики спустя время, таки находят на сайте и тащат их в индекс:

 

Это ваши слова?

 

Вы показали пример, я вам его повторил, где дубли?

 

Причем здесь редиректы?

 

Активируйте seo_pro и будет вам счастье

[AlexDW]

это ссылки из старой темы? Здесь вот вроде от этого примера всё - http://demo.myopencart.com

там на демосайте просто seopro отключен и как раз используется родной кривой seourl ;)

[andrus]

 

 

Активируйте seo_pro и будет вам счастье

Скиньте мне урл любого магазина на ОС, и я покажу о чём речь. У всех активирован seo-pro, но мало кто обращали внимание на наличие подобных дублей. 

 

 

 

там на демосайте просто seopro отключен и как раз используется родной кривой seourl

ладно... Может направите мысль, что можно подправить на своём сайте в файле seo-pro, чтобы срастить всё в одну категорию, где и должно быть? Не пойму что такое с моим сайтом, но и на версии ОС 1.5 и на версии ОС 2,2 такое наблюдается, при активном сео-про

Змінено 3 жовтня 2016 користувачем andrus

[chukcha]

я вам дал ссылку на демо

 

вот вам еще

http://demo2.slasoft.kharkov.ua/

 

и еще

http://demo.slasoft.kharkov.ua/

 

Найдете - 100$ ваши

 

Не найдете -

 

Здесь публично извинитесь за свои слова

 

 

Без разницы кто там будет и что ( seo-pro или seo-url) но самое главное, это просьба №1: уберите вы наконец плодирование несуразных дублей, которые поисковики спустя время, таки находят на сайте и тащат их в индекс:

[Otvet]

Скиньте мне урл любого магазина на ОС, и я покажу о чём речь. У всех активирован seo-pro, но мало кто обращали внимание на наличие подобных дублей. 

если не бегать и не выкладывать ссылки на них на форуме, ПС о них не узнают

 

либо очень кривые модули которые тянут подобные ссылки в данных посредством XHR (тогда гугл видит)

Змінено 3 жовтня 2016 користувачем Otvet

[chukcha]

Тут надо заметить что это  демо сайты, и у них нет задачи что-то ЧПУтить

[andrus]

я вам дал ссылку на демо

Если там по этому демо включён seo-url, то тогда ошибки на лицо. Но думаю у вас там активный как раз seo-pro:

категория - http://demo2.slasoft.kharkov.ua/mac/

а вот её дубль, но с алиасом от производителя в конце - http://demo2.slasoft.kharkov.ua/mac/canon

Змінено 3 жовтня 2016 користувачем andrus

[Otvet]

https://www.yandex.ru/?эксклюзивно-для-andrus-c-опенкартфорум-миллиарды-дублей-1234567890

[chukcha]

Номер кошелька в личку.

[SooR]

вариант с патчем через seo_url - еще более хреновый, чем с запросом к БД, имхо

встречаются некоторые чЮдо-мега-сео дополнения, которые не то что его патчат - целиком заменяют :ugeek:

 

больше склоняюсь к варианту sv2109

 

1. Модификация получается минималистичной.

2. Для чего тогда нужен OCmod?

3. Вариант от sv2109 предложил и я, но это ручное вмешательство, как оказалось.

[Otvet]

категория - http://demo2.slasoft.kharkov.ua/mac/

а вот её дубль, но с алиасом от производителя в конце - http://demo2.slasoft.kharkov.ua/mac/canon

а теперь найдите в выдаче это...

 

для этого есть своя ниша

http://22.seo-page.ru/

[chukcha]

 

Вариант от sv2109 предложил и я, но это ручное вмешательство, как оказалось.

 

Ни в коем случае, что отвергли, просто было озвучено, и попытка найти более оптимальное решение.

[andrus]

)
ладно. думаю мои комменты выше, они не без внимания будут у разработчиков. И тот кто создавал seo-pro (спасибо за этот файл), то учтёт обнаруженное, немного поэкспериментирует и внесёт ряд улучшений. 

Змінено 3 жовтня 2016 користувачем andrus