[eluser]skunkbad[/eluser]
[quote author="DJMOHNL" date="1280561717"]next time you try create an MY_Input.php youll see that you need to add all the functions inside this one, wich isnt very smart to do.. Yust add the system "get patch" and it gives you a far better patch (nothing gets broken, its tested by more than a dosen people here)[/quote]
I know that patching the system libraries works, because I tried it too. I was simply providing a solution that shows extending the classes "the CI way", because this is the way most people should do it. If they ever upgrade their installation to a new version of CI, they won't wipe out a custom functionality of their site that they "patched" or "hacked".
Heck, I don't even use $_GET. I'm just here trying to help solve an ongoing complaint about CI's $_GET array deficiency. If you needed to have other extended functionality of the Input class, yes you would have to change MY_Input to suit your needs. I am smart enough to do it.
In the case of WanWizard and his complaint about extending classes and performance issues, this is a legitimate complaint. Your hacking of core CI classes, and not extending those classes because you might fumble the next time they require modification is just a lame excuse.
In what way does your beloved patched method provide a far better patch? It's result is the same as what I have done by extending the classes, but I haven't changed core files. Prove me wrong...
