Обновление PdoPage

Ранее описал как убрать зависимость от jQuery в PdoPage
Затем залез в issues и увидел там давольно интересное предложение от @Руслан Алеев и описание ошибок с которыми тоже сталкивался не раз
Самая бесячая для меня
PHP warning: Undefined variable $showLog
решение которой начинается сразу после установки

Поэтому решил немного исправить ошибки и добавить новые фишки

Скажу сразу что я не гуру кода, часть делал через нейронку

Тестил на php начиная с 7.4 до 8.4
MODX 3.1.2
То что «вроде» получилось сделать PR кинул
https://github.com/vitos88/pdoTools3.0.3

  • убрал зависимость от jQuery (#315) файл pdopage.js и pdopage.min.js
  • убрал ошибку Undefined variable $showLog (#351) сниппет pdoPage
  • исправил &outputSeparator при включеном
    &ajaxMode=`button`
    (#354) сниппет pdoPage
  • убрал ошибку "[] operator not supported for strings in pdoCrumbs" (#367) сниппет pdoCrumbs
  • По умолчанию &sortby=`pagetitle`, но если другой класс будет то где нет pagetitle логика ломалась и нужно было дабавлять свое поле в sortby
    Понемял логику в 'core/components/pdotools/src/Fetch.php'
    Теперь если publishedon нет у обьекта будет приотитет такой:
    publishedon, createdon, menuindex, pagetitle, username, id
    вроде везде будет, id то точно (#368)
  • исправил (#370)
    исправил 'core/components/pdotools/src/Support/Paginator.php'
  • добавлен &limitOnPage в PdoPage (#374) сниппет pdoPage
    Теперь если в PdoPage задан limitOnPage он имеет высокий приоритет и limit игнорируется полностью
    limit из post/get средний приоритет
    если оставить как обычно limit это будет самый низкий преоритет
У меня не получилось повторить ошибки из issues
#366
#369
#381

Не получилось исправить
#372
#375
ВитОс
7 часов назад
modx.pro
33
0

Комментарии: 0

Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.
0