grinyaha / editdocs Goto Github PK

php 7.4.1

Импорт: Deprecated: Function get_magic_quotes_gpc() is deprecated in /home/.../assets/lib/MODxAPI/MODx.php on line 158
Notice: Undefined index: clear in /home/.../assets/modules/editdocs/ajax.php on line 32
Notice: Undefined index: bigparent in /home/.../assets/modules/editdocs/ajax.php on line 38
Notice: Undefined index: bigparent in /home/.../assets/modules/editdocs/ajax.php on line 41
Notice: Undefined index: edit in /home/.../assets/modules/editdocs/ajax.php on line 41
Notice: Undefined index: id in /home/.../assets/modules/editdocs/ajax.php on line 44
Notice: Undefined index: myfile in /home/.../assets/modules/editdocs/ajax.php on line 50
Notice: Undefined property: editDocs::$snipPrepare in /home/.../assets/modules/editdocs/ajax.php on line 680
Notice: Undefined variable: testInfo in /home/.../assets/modules/editdocs/ajax.php on line 357
Notice: Undefined property: editDocs::$out in /home/.../assets/modules/editdocs/ajax.php on line 466
7 из 7. Готово!

Непонятно, зачем в рамках одного класса подключать 5 раз этот файл, если он сразу подключен в конструкторе - https://github.com/webber12/editDocs/blob/master/ajax.php#L74-L76

Лучше бы:

• переименовать function editDoc($modx) в function __construct($modx) - т.к. по сути использование названия метода, совпадающего с названием класса - это и есть устаревший вариант конструктора
• оставить в нем только инициализацию modResource, а во всех последующих методах обращаться сразу к $this->doc без повторной инициализации класса modResource
• остальное из конструктора вынести в отдельный метод - например save (исходя из того, что там делается).

В header.tpl имеется секция:
<link rel="stylesheet" type="text/css" href="[+manager_path+]/media/style/[+manager_theme+]/style.css" />
В плейсхолдер [+manager_path+] возвращается результат уже с "/" на конце, таким образом лично у меня не подгружались стили, возможно конфликт из-за настройки "Добавлять слэш к контейнеру" или Strict URLs или фиксом удаления лишнего слэша в .htaccess, но точно не уверен.

Лечение удалить / перед media:
<link rel="stylesheet" type="text/css" href="[+manager_path+]media/style/[+manager_theme+]/style.css" />

При импорте если передавать published он попросту игнорируется
приходится повторно заливать и делать апдейт.

Еще не всегда после загрузки файла вылезает форма с настройками приходится убирать у класса setting display: none

А я то все думаю, что за чертовщина с пагинацией :) https://github.com/Grinyaha/editDocs/blob/master/assets/modules/editdocs/ajax.php#L166

Добавить выгрузкку фото из simpleGallery

После обновления EditDocs при экспорте в выборке параметров исчезло поле id http://joxi.ru/1A5QXz7tDJGB5A Очень неудобно, можно его как-то вернуть?

Как найти или где выключить создание алиаса?
Ведет себя странно. Часто генерит алиасы типа "-5" из pagetitle типа "Улица Блаблаблаевская, 5"
Приходится постоянно редактировать вновь созданные документы

1. Установил Evolution 2 отсюда
   https://github.com/evolution-cms/evolution

2. На сервере установлен php 7.1

3. Установил ваш модуль последнею версию.

При попытки открыть модуль, в админки показывается ошибка Error, где должны выводиться модуль.

Ошибка из логов.

2019/08/05 13:04:03 [error] 16414#16414: *94 FastCGI sent in stderr: "PHP message: PHP Fatal error: Cannot declare class DLTemplate, because the name is already in use in /var/www/local-website/my-site-evolution.ru/assets/snippets/DocLister/lib/DLTemplate.class.php on line 8" while reading response header from upstream, client: 10.0.2.2, server: my-site-evolution.ru, request: "GET /manager/?a=112&id=3 HTTP/1.1", upstream: "fastcgi://unix:/run/php/php7.1-fpm.sock:", host: "my-site-evolution.ru", referrer: "http://my-site-evolution.ru/manager/"

1. объединить процесс обновления и добавления новых. Пользователи путаются, да и ни к чему это - проще, если система автоматом из одного файла распознает, что новое, а что существует.
2. оптимизировать процесс сохранения при апдейте - совершенно не нужно создавать документ и сохранять его при редактировании каждого поля - https://github.com/Grinyaha/editDocs/blob/master/assets/modules/editdocs/ajax.php#L270-L272 в цикле перебора ключей, достаточно собрать все ключи и обновить документ один раз
3. добавить генерацию событий, чтобы можно было манипулировать созданными данными также, как теми, что создаются вручную (для этого надо вызывать сохранение как $doc->save(true);

Ради интереса можно глянуть, как это делал я под свои нужды - https://github.com/webber12/editDocs/blob/master/ajax.php#L260-L310 :)

$writer->setPreCalculateFormulas(false); добавить

Как можно подружить EditDocs с MultiCategories?
github.com/Pathologic/MultiCategories

Невозможно задать другой разделитель при экспорте в CSV. ; часто используется в поле content, файл экспорта становится непригодным

При импорте нет возможности проверять совпадение по id (Evo 1.4.11, PHP 5.6)