• Ребят, я с трудом сдерживаю маты, я постараюсь успокоиться и дать конструктивный комментарий.

    # В коде бардак, необходим полноценный рефакторинг
    # Уже писали но я повторюсь жесткая завязка на стороннем плагине, который уже больше 5 лет не поддерживается.
    # Куча магических цифр и строк (Ex. add_action( ‘init’, ‘wpshop_init_signon’, 7); // Что значит цифра 7?)
    # Нет проверок на существование переменных или элементов массива я понимаю что язык позволяет так наглеть но это сильно усложняет дебаг.
    # Нет четкой структуры, такое ощущение что “хоп-хоп и продакшн”

    Вам конечно решать что делать дальше с плагином.
    Я рекомендую все же структуризовать (или по крайней мере описать документацию для разработв), отрефакторить (включите отображение всех ошибок в том числе и warning-ов в PHP, поймете) и создать свои формы избавиться от привязки к устаревшему плагину.

    И Я не рекоммендую устанавливать данный плагин хоть, в будущем возникнут проблемы, которые разработчик решить не сможет.

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author wpshop

    (@wpshop)

    add_action( ‘init’, ‘wpshop_init_signon’, 7) где 7 это приоритет, на init вешают функции и другие плагины.
    Cforms нас устраивает прежде всего как конструктор форм, если завести “свои формы” то не будет гибкости, а делать свой конструктор накладно.
    Рефакторинг кода и доки для разработчиков планируем но пока другие задачи в приоритете (функционал). В любом случае спасибо за критику будем исправлять.

    Plugin Contributor shurupp

    (@shurupp)

    Мы теперь сами поддерживаем плагин cForms, планируем его полностью интегрировать внутрь WP Shop, чтобы был только 1 плагин для работы.
    По коду возможно есть какие-то недочеты. Подключайтесь к нашей команде, поможете нам структурировать все ??

    Ага.. мне сейчас попал сайт на расширение, который базируется на этом “плагине”. И мы тут с коллегой повеселились будь здоров. За это конечно спасибо, но за то, что пополняете репозиторий малограмотными, полными ошибок продуктами – большое негодование.

    Постарайтесь в течении месяца хотя бы переверстать Вашу “профессиональную тему”, предварительно ознакомившись с HTML и CSS, хотя бы. Например, с различиями id и class в тегах.

    К сожалению, если этого не произойдет и Ваш продукт будет оставлен в публичном доступе, то я буду вынужден разместить обзор этого продукта в сети, для того, чтобы ограничить дальнейшее его распространение.

    Я очень надеюсь, что вы меня правильно поймете – все когда-то были начинающими, но разница в том, что не было попыток свои “эксперименты начинающего” оформлять в продукт и распространяться в сети под видом модуля. Спасибо.

    Plugin Contributor shurupp

    (@shurupp)

    Спасибо за комментарий, Артемий Лебедев, не иначе. Может покажете нам образцы своего мастерства? Обзор он разместит, держите его семеро
    Не нравится наш скрипт – напиши свой, а мы посмотрим. Если есть комменты по делу – спасибо, а нет так просто тролль

    1. Отправьте Вашу “профессиональную” тему на валидацию (validator.w3.org).

    2. Почему товары интернет магазина находятся в обычных записях. У WP есть функционал для создания отдельного типа записи и отдельных таксономий, предназначенный как раз для этих целей (используя стандартные в процессе работы создается путаница).

    3. Магазин на 2500 товаров загружается с этим плагином около 10 секунд )

    Plugin Author wpshop

    (@wpshop)

    1. ну проверили и что:
    id не уникально на странице, свойство onclick в ссылке, свойство border внутри тега table. Да не по стандарту, но что это меняет для конечного пользователя?
    2. Так мы специально так сделали. Вынести в отдельный тип записей не проблема, но большинство сторонних плагинов (к примеру для SEO) работают именно с этим типом записей (post). А если вам нужно отделить контент от магазина вот и создайте отдельные типы записей (например новости, вакансии и т.д.)
    3.Значит у вас такой хостинг, или конфликт плагинов (например wp-shop конфликтует с cyr-to-lat вместо него рекомендуем ставить rus-to-lat) в общем нужно смотреть. Вот примеры с немалым количеством товаров у них работает нормально https://piterplay.com/,https://vkplitka.ru/ (первое что попало под руку). Есть магазины с 10000 товара и работают нормально.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Это что то с чем то…’ is closed to new replies.