search

srs81 / CakePHP-AjaxMultiUpload

CakePHP 2 plugin to allow for easy multi-file upload with AJAX
Other
65 stars 33 forks source link

A few suggestions to improve cake integration #31

Closed destinydriven closed 11 years ago

destinydriven commented 11 years ago
  1. Allowing CakePlugin bootstrap option to load the plugin's bootstrap file to avoid duplicate code through the repeated usage of require_once(). Removed instances where bootstrap was loaded.
  2. Allowing the component to load the helper automatically since both are needed anyway. No need to explicitly load helper once the component is added to controllers. Added visibility keywords.
  3. Updated README.md. Added instruction to use CakePlugin::load('AjaxMultiUpload', array('bootstrap' => true)); so bootstrap is loaded automatically.
srs81 commented 11 years ago

Thanks! I would like to merge this, is this final?

destinydriven commented 11 years ago

Yes, this is final. I made sure that everything is working as expected. :)

srs81 commented 11 years ago

Thanks! :+1: