AMP for Product Pages не дружит с SeoPro

Тема в разделе "Модули и дополнения", создана пользователем Djon, 6 сен 2016.

  1. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
  2. markimax

    markimax Активный пользователь

    Сообщения:
    210
    Симпатии:
    34
    В помойку такие говно модули сразу.
     
  3. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
    Я ручками поправил нужный файл, VQMOD не использовал.
     
  4. markimax

    markimax Активный пользователь

    Сообщения:
    210
    Симпатии:
    34
    Я про то что если так делают модули, то и модули такие по качеству и настоятельно рекомендую не использовать такие говномодули.
    И судя по вопросам не зря так рекомендую
     
  5. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
    За неимением поварихи, как говорится, ... повара. Аналогов, как я понимаю, нет, поэтому и прошу помочь с доработкой этого.
     
  6. markimax

    markimax Активный пользователь

    Сообщения:
    210
    Симпатии:
    34
    Как это нет - их валом. Модулей микроразметки и кешировщиков валом.
    Толку от этого говномодуля никакого
     
  7. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
    А вы вообще в курсе, что такое AMP? По вашим словам так не скажешь...
     
  8. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
    Плагин то по сути простенький, ничего особенного в его работе нет. Проблема лишь в том, что SeoPro всем страницам, где в урле есть "product_id=(значение)" меняет урл на свой ЧПУ. Не смотрит вообще на то, что стоит перед "product_id=(значение)".

    Как его заставить менять "product/amp_product&product_id=(значение)" на, скажем, чпу/amp или вообще не менять?
     
  9. Vlad

    Vlad Эксперт

    Сообщения:
    4.161
    Симпатии:
    1.013
    Если вам достаточно просто без чпу, то в модификаторе поменяйте product_id на ampproduct_id например.

    product/amp_product', 'ampproduct_id=


    а в контроллере модуля в самом начале т.е после index добавьте
    PHP:
     $this->request->get['product_id'] = $this->request->get['ampproduct_id'];
    Метод конечно через Ж, но должен работать. У меня нет сеопро под рукой (пользуюсь русской сборкой), потому на примере сеопро не могу подсказать..
     
  10. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
    Благодарю! Оставлю пока так, как вы предложили, и подожду следующей версии, автор обещал решить проблему.
     
  11. poni

    poni Новичок

    Сообщения:
    3
    Симпатии:
    0
    @Djon Скажите пожалуйста, решили вопрос с SEO_PRO?
     
  12. Djon

    Djon Новичок

    Сообщения:
    19
    Симпатии:
    12
    Я на это подзабил пока. Автор обещал пофиксить, возможно, проблема уже решена.
     
  13. grey0207

    grey0207 Активный пользователь

    Сообщения:
    126
    Симпатии:
    33
    Согласно change log'а последней версии, пишет автор:
    version 1.0.9 (06-02-2017)
    - seo url fix for products with "amp" in url

    Протестирую сегодня модуль и отпишусь.
     
  14. poni

    poni Новичок

    Сообщения:
    3
    Симпатии:
    0
    Спасибо!
    У меня есть модуль, который может складывать товары в Корзину: http://ampopencart.tk/index.php?route=product/amp_product&product_id=40
    как думаете, насколько сложно/есть желание объединить функционал модуля топикстартера и этот?
     
  15. poni

    poni Новичок

    Сообщения:
    3
    Симпатии:
    0
    @grey0207
    Выпустили версию 1.0.10
    - price meta fixed
    - version 2.1 updated
    - version 2.1 seo url support
     
  16. grey0207

    grey0207 Активный пользователь

    Сообщения:
    126
    Симпатии:
    33
    Я знаю, он там с версиями напутал что-то, после моего сообщения сегодняшнего в support обновил всё)
    P.S.: у меня что-то с seo url не решилось ничего. Отложил затею на потом.
     
  17. goreev

    goreev Новичок

    Сообщения:
    3
    Симпатии:
    0
  18. Chukcha

    Chukcha Специалист

    Сообщения:
    3.013
    Симпатии:
    687
    Самое простое, быстрое решение

    product_id заменить на amp_product_id
     
  19. goreev

    goreev Новичок

    Сообщения:
    3
    Симпатии:
    0
  20. trialon77

    trialon77 Новичок

    Сообщения:
    1
    Симпатии:
    0
    А что за модуль это у вас? Ссылку можно? Это тоже для создания amp страниц?