Skip to content

Update seopro.php#72

Open
leonidukg wants to merge 1 commit intoocStore:3.0.x.xfrom
leonidukg:patch-6
Open

Update seopro.php#72
leonidukg wants to merge 1 commit intoocStore:3.0.x.xfrom
leonidukg:patch-6

Conversation

@leonidukg
Copy link
Copy Markdown

  1. Неправильно формировалась ссылка на запись в блоге, пропадала категория. Хотя в настройках было написано, чтобы её сохранять. Из-за этого canonnical был правильный, а адрес нет.
  2. Неправильная проверка переменной для кэширования. Проверка идёт на массив ли переменная, хотя выше и так она объявляется массивом. Из-за этого кэш всегда сбрасывается
  3. Немного поправлен синтаксис

1. Неправильно формировалась ссылка на запись в блоге, пропадала категория. Хотя в настройках было написано, чтобы её сохранять. Из-за этого canonnical был правильный, а адрес нет.
2. Неправильная проверка переменной для кэширования. Проверка идёт на массив ли переменная, хотя выше и так она объявляется массивом. Из-за этого кэш всегда сбрасывается
3. Немного поправлен синтаксис
@AlektroNik
Copy link
Copy Markdown

AlektroNik commented Apr 4, 2022

Добрый день. Вы не сталкивались с проблемой, что SeoPro конвертирует знаки плюс в get запросах в пробелы обычные?
А хотелось, чтобы он либо их не трогал вообще, либо в %20 делал.
Я знаю, что для php это обычная практика + ковертить в пробел. И, тем не менее, задача актуальная. На чистом opencart без seopro нет проблем.

@leonidukg
Copy link
Copy Markdown
Author

Добрый день. Вы не сталкивались с проблемой, что SeoPro конвертирует знаки плюс в get запросах в пробелы обычные? А хотелось, чтобы он либо их не трогал вообще, либо в %20 делал. Я знаю, что для php это обычная практика + ковертить в пробел. И, тем не менее, задача актуальная. На чистом opencart без seopro нет проблем.

В телеграмм мне напишите: https://t.me/leonidukg
Задач таких никогда не стояло. Нужен сам сайт и уже там смотреть, что происходит.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants