Updated Code Review 10 - Pluf::loadClass (pluf)

[no-r...@ceondo.com]

Hello,

The following review has been updated:

10 - Pluf::loadClass
Project: Pluf Framework
Status: Accepted
Reported by: William Martin
URL: http://projects.ceondo.com/p/pluf/review/10/
General comments (last first):

# By Thomas Riché, Jun 14, 2011:
Indeed !!!

I was trying to integrate PHPExcel to an existing app.
I saw that this lib is using the same duck typing conventions as we do in Pluf...

however, i ran into class loading errors.
I then remember reading about this revue.

YOU SAVED MY DAY !!!

Thanks

# By Thomas Keller, Apr 7, 2011:
Looks fine.

Detailed file comments (last first):

# By Thomas Keller, Apr 7, 2011, on file:
b/src/Pluf.php

I'm fine with that, the $error could however be improved to say "Impossible to load the class / interface: ".

# By Thomas Riché, Jun 14, 2011, on file:
b/src/Pluf.php

Perfect

--
Review: http://projects.ceondo.com/p/pluf/review/10/