review reward point 10/4

- vẫn sử dụng object manager:
 app/code/Magenest/RewardPoints/Block/Adminhtml/Order/RedeemPoints.php:81
 app/code/Magenest/RewardPoints/Block/Customer/Friend.php:246
 app/code/Magenest/RewardPoints/Helper/Data.php:1636

- override cha nhưng code lại y hệt:
 app/code/Magenest/RewardPoints/Controller/Adminhtml/Transaction/Index.php:27
 app/code/Magenest/RewardPoints/Controller/Adminhtml/Rule/Index.php:28
 app/code/Magenest/RewardPoints/Controller/Adminhtml/Account/Index.php:28

- class backend lại kế thừa class của front end, nên extend Abstract Transaction. Chưa có acl app/code/Magenest/RewardPoints/Controller/Adminhtml/Transaction/Save.php:15
các controller trong Quote


- Check lại các controller trong Rule, chưa có acl, nên kế thừa abstract Rule để không phải viết lại hàm _isAllowed()

- thêm comment các trường hợp nào chạy vào các hàm if, vẫn sử dụng object manager
 app/code/Magenest/RewardPoints/Controller/Customer/ApplyReferralCode.php

- biến chưa đc khai báo app/code/Magenest/RewardPoints/Controller/Quote/Add.php:81


- xử lý try catch:
 app/code/Magenest/RewardPoints/Controller/Quote/Add.php:102

- Naming convention: tên ko nên sử dụng 1 động từ
 app/code/Magenest/RewardPoints/Cookie/ReferralCode.php:60


- Nên tách ra thành hàm riêng, 1 hàm xử lý 1 việc cụ thể:
 app/code/Magenest/RewardPoints/Cron/DailyUpdate.php:155

- xóa hàm không còn sử dụng:
  app/code/Magenest/RewardPoints/Helper/Data.php:658
  app/code/Magenest/RewardPoints/Helper/Data.php:789
  app/code/Magenest/RewardPoints/Helper/Data.php:796


- sửa các trường hợp return
  app/code/Magenest/RewardPoints/Helper/Data.php:889