Closed lonnieezell closed 4 years ago
Well, damn, that's probably why I hadn't had it listed here before. :)
@jim-parry What do you think? I've actually already included the Typography library since I think it's handy enough of the time to keep around. I don't have problems with them being in core, personally, and don't really recall the reasons for keeping them out.
Well if we were to include them in core I'd be more than happy to port the FTP Library over :)
@coderste Tell you what - while we determine whether they're being included in core, if you want to go ahead and port, that would be great. Even if we remove it to its own repo later, it still needs to be ported.
Basically it's just a straight port to use more current PHP7 standards, and to fit in with how CI4 is currently working, which might require a new Service method added in system/Config/Services
if it has any dependencies. Also - any tests that you can make work with it (check the CI3 repo to see if those tests are worth using.) and docs, please.
Thanks!
@lonnieezell hello, i'm finished 1~2 months ago FTP library. Push requested.
@byazrail ha I see - any Tests written for that? If not I don't mind writing some and opening a PR also I think we may need to update the documentation :)
@coderste oo I see - testing documentation. testing after request update.
@byazrail I believe they mean PHP Unit Test and Document Updates :)
Ok, now we have two FTP ports ? Shows the wisdom of discussing it at the issue level, and "assigning" someone, to prevent duplicaiton. "Porting" something from CI3 means fitting it into the CI4 "structure", with a Services method if needed, namespacing, conforming to the CI4 style guide (class, metho & variable naming, for instance, that would probably differ from CI3), updating or creating unit tests, and updating the user guide.
We can work on this PR for that now and just remove the other issue, I am also trying to revert my commit on the other branch
As for the libraries to remove or not, I have no problem with typography and ftp making their way here.
The Zip library sounds like it might suit a Compresser service, with adapters (er, handlers) for some of the compression extensions supported by PHP (zip, phar, bzip2, etc). It could start out as a straight port, but we want to keep the future in mind.
XML-RPC is a beast unto itself, and should be a separate addin. The CI3 implementation is overly complicated, and lacking some functionality (eg using TLS), and needs re-thinking, not just re-porting. I see it as part of a larger group of client/server services (eg SOAP, REST), that are better addressed once we get the basics done.
As per the features of CI4, FTP will not be a part of framework https://forum.codeigniter.com/thread-71705.html
Confirmed there will be no FTP
class in CI4.
I can do this if we're just porting it over and updating it to match the PHP7 Standards. Let me know exactly what I need to do and I'll get to it over the weekend.
Edit: From reading the forum post on CodeIgniter4 it says:
Is this still the case or are we going to include that Library with CodeIgniter4?