Skip to content

Conversation

@poh-am
Copy link

@poh-am poh-am commented Feb 21, 2017

Немножко переписал код, вынес логику в класс, теперь появилась возможность добавить модуль в exstras, но желательно протестировать его еще.
Из того, что увидил, пофиксил:

  • Dropzone ошибка постоянно выскакивала;
  • шаблон модуля и чанк перенес в папку install;
  • добавил по умолчанию папку upload, куда загружаются xls файлы.

@Grinyaha, @Pathologic, @AgelxNash пересмотрите пожалуйста код на наличие ошибок свежим взглядом, если есть время, все ли там ок, так было б немножко удобнее организовать этот модуль.
Если будет время, хорошо бы придумать вынести PHPExcel и загружать ее через автозагрузчик.

@Pathologic
Copy link

В классе совсем не используются переменные 😄

@poh-am
Copy link
Author

poh-am commented Feb 21, 2017

Это я сегодня еще сяду, уберу лишний код, перепишу под MODxAPI
Но само направление правильное?

@Pathologic
Copy link

Ну, раз работает, то правильное. Еще не мешало бы убрать этот чанк paginateEditDocs. А загрузку файлов я бы не стал делать, проще использовать файловый менеджер.

@AgelxNash
Copy link

Если это бекендовый ajax, то почему там используется Web событие инициализации?
https://github.com/poh-am/editDocs/blob/e637d0b603c11182b0f41a074846b80db62594bd/assets/modules/editdocs/ajax.php#L12

glob на папках с большим кол-вом файлов отваливается. Лучше использовать итератор. Ну или воспользоваться готовым https://github.com/AgelxNash/DocLister/blob/master/assets/lib/Helpers/FS.php#L318-L333

@Grinyaha
Copy link
Owner

Grinyaha commented Mar 2, 2017

Спасибо что посмотрели моего первенца :) Пока честно говоря завал с работой, некогда вникать. Как в extras добавлять не знаю, Диме писать? Он даже вникать не захотел.

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.

4 participants