[Bug 1222264] Review Request: php-ocramius-proxy-manager - OOP proxy wrappers utilities

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1222264



--- Comment #3 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> ---
Some comments about autoloader
- isn't ZendXml namespace needed ?
- isn't GeneratedHydrator needed if optional dep is installed ?
- as you have ZF2 in the dependencies, could make sense to use ZF2 autoloader,
something like:


<?php
require_once ('Zend/Loader/StandardAutoloader.php');
$option = array(
  Zend\Loader\StandardAutoloader::LOAD_NS => array(
    'ProxyManager' => __DIR__,
    'GeneratedHydrator' => '/usr/share/php/GeneratedHydrator',
  ),
  Zend\Loader\StandardAutoloader::AUTOREGISTER_ZF => true,
);
$loader = new Zend\Loader\StandardAutoloader($option);
$loader->register();


Notice: using above test suite fails...
Fatal error: Declaration of generated55582f3846c13::__construct() must be
compatible with Zend\Loader\SplAutoloader::__construct($options = NULL) in
/dev/shm/extras/BUILDROOT/php-ocramius-proxy-manager-1.0.0-1.fc21.remi.x86_64/usr/share/php/ProxyManager/GeneratorStrategy/EvaluatingGeneratorStrategy.php(65)
: eval()'d code on line 1

Which seems interesting bug....

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]