gaurav-spacreo / minify

Automatically exported from code.google.com/p/minify
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

All classes should be in Minify namespace #161

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Not really a defect.
Will there be an overhaul of the class structure in the near future?
My problem is that I'm only using the compressor classes (Minify_CSS and
JSMin), I'm serving the files for myself.
Minify's class structure makes it a little hard to include as an
svn:external right now, since I have to use the whole lib folder.
It would be nice if Minify would follow the regulations that all major PHP
projects keep right now, that the project name opens a namespace and
_every_ class is place under this namespace.
So there would only be Minify_* classes.
Doctrine and Zend Framework (and several others, like SabreDav, Yadif, ...)
do this already.
So JSMin and JSMinPlus would have to be renamed to Minify_JS and
Minify_JSPlus or something alike.

Any plans of doing this?

Minify version: release_2.1.3
PHP version: 5.3.1

Original issue reported on code.google.com by stephan....@gmail.com on 4 Feb 2010 at 11:22

GoogleCodeExporter commented 9 years ago
Yes, that's a good idea, but "near future" is unknown. I just haven't had a lot 
of
time to work at home lately and the desired items to overhaul are piling up. I 
really
need to do some thinking and define the next milestone.

I know the current Minify class will end up as Minify_Server. I've considered 
moving
the HTTP_ classes to a separate project, but then is it worth the maintenance 
headache...

Original comment by mrclay....@gmail.com on 5 Feb 2010 at 12:38

GoogleCodeExporter commented 9 years ago
That'd be really awesome.
I think there was a ticket regarding proposal as a Zend Framework component, if
you'll do this you will have to deal with this anyway (since it will have a 
prefix
like Zend_Minify_).

Original comment by stephan....@gmail.com on 11 Feb 2010 at 12:45