search

Emagister / zend-form-decorators-bootstrap

Zend_Form decorators for Twitter's Bootstrap UI
http://emagister.github.com/zend-form-decorators-bootstrap
168 stars 79 forks source link

add include-path for ZF 1.11 #49

Closed dready closed 12 years ago

dready commented 12 years ago

Hi,

sorry for the mess. I needed to add the include-path tag so composer will add the Twitter Library into it's vendor/composer/include_path.php.

This is needed for Zend Autoloader to work in ZF 1.11. (At least in my projects)

thx, dready

theUniC commented 12 years ago

Are you running into problems with the autoloader provided by Composer? By requiring the autoload.php from the vendors folder does not fix your issue?

dready commented 12 years ago

unfortunately not. we are using several projects (doctrine, zend, ..) and have been working a while with composer now.

I tried to debug where the problem comes from. Somehow the classloader from composer is not triggered in our form implementation for the Horizontal Twitter Form and the quick fix would be adding it into the include-path. (i finally run out of time for debugging)

theUniC commented 12 years ago

Although will not disappear to use the include-path on Composer it's deprecated in favour of the autoloader. Give me a chance to try to reproduce the issue and check if there is no other way to integrate Composer autoload with Zend_Loader_Autoloader.

dready commented 12 years ago

ok, will do the same on my end this week, i keep you posted if i find the problem.

Maks3w commented 12 years ago

The problem is that the directory must point to the PSR-0 root dir. Actually is pointing to the vendor subdirectory which is wrong. #53 fix this problem

Maks3w commented 12 years ago

PHP classes load correctly after apply #53. However form decorators only can load through include_path so it's needed include the root dir in the include_path

theUniC commented 12 years ago

Done. Closing this as it has been fixed.