search

chargebee / chargebee-php

PHP library for the Chargebee API.
https://apidocs.chargebee.com/docs/api?lang=php
MIT License
72 stars 62 forks source link

Composer autoload #11

Closed RobertBoes closed 3 years ago

RobertBoes commented 8 years ago

Since #4 is already open for more than a year, I decided to open a new issue with more or less the same problem. When using the Chargebee autoload with Laravel (composer), all files are being loaded because Chargebee.php requires every single class used in this project.

dolcy commented 8 years ago

+1 This entire library needs a revamp in fact.

ashleydw commented 8 years ago

+1 for the total revamp. Code is very out of date - simple things like being unable to get an array of values as in https://github.com/chargebee/chargebee-php/issues/9 is needed

juanstiza commented 8 years ago

+1 for the revamp.

balazsotakomaiya commented 6 years ago

Totally agree.

icetee commented 5 years ago

Man.. please use namespaces

frankarensmeier commented 5 years ago

This is still an issue after more than 4 years. This is a great pain point for us right now.

icetee commented 5 years ago

This is still an issue after more than 4 years. This is a great pain point for us right now.

Try use my fork: https://github.com/chargebee/chargebee-php/pull/23

frankarensmeier commented 5 years ago

@icetee Thank you for for the link - looks very good. Any idea if / when this can be merged into master? This is becoming more and more annoying. I did some profiling on the Chargebee code base with and without proper PSR-0 autoloading and the difference is quite significant. On my local docker container, Chargebee stands for 25% of total execution time whenever a page gets rendered. All files are loaded no matter what (only a small fraction of all pages actually use CB). This is not good!

cb-rakesh commented 3 years ago

This has been fixed in v3.0.0