PayA Connect - Backend checklist

- link này có thể chưa tồn tại, check sự tồn tại: app/code/Magenest/PayaConnect/view/adminhtml/templates/system/config/fieldset/version.phtml:20

- không dùng object manager:
app/code/Magenest/PayaConnect/Controller/Card.php:52

- response mà lỗi thì không trả gì về à ? Controller này nên luôn có 1 return
app/code/Magenest/PayaConnect/Controller/Card/Save.php:148

- biến chưa khởi tạo:
app/code/Magenest/PayaConnect/Controller/Adminhtml/Transaction/Index.php:36

- các file controller đều chưa có acl, có thể tạo 1 abstract controller để dùng chung acl:
app/code/Magenest/PayaConnect/Controller/Adminhtml/Transaction/Index.php:57

- ko cần viết else nữa: app/code/Magenest/PayaConnect/Helper/ConfigHelper.php:96

- hàm nên viết vào model:
\Magenest\PayaConnect\Helper\DataHelper::saveTransactionOrder
\Magenest\PayaConnect\Helper\DataHelper::editStatusTransaction
\Magenest\PayaConnect\Helper\DataHelper::getDataCard

- Mấy đoạn code này xem xem có thể tách ra thành hàm và sử dụng lại đc ko, đang viết lại code gần như giống hệt:
app/code/Magenest/PayaConnect/Model/PayaConnect.php:404
app/code/Magenest/PayaConnect/Model/PayaConnect.php:381
app/code/Magenest/PayaConnect/Model/PayaConnect.php:362

- tách ra thành function riêng cho 1 version. mà mới là version 1.0 mà:
app/code/Magenest/PayaConnect/Setup/InstallSchema.php:34

- chưa có changelog
- các text, message thông báo ra màn hình nên để vào __(), với js thi $.mage.__()