Grinyaha / editDocs

Edit fields of database & tv from module of Evolution CMS
13 stars 14 forks source link

Рефактор кода #2

Open poh-am opened 7 years ago

poh-am commented 7 years ago

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

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

Pathologic commented 7 years ago

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

poh-am commented 7 years ago

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

Pathologic commented 7 years ago

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

AgelxNash commented 7 years ago

Если это бекендовый 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 commented 7 years ago

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