CodeIgniter Forums
IDEA: Default core class extensions - Printable Version

+- CodeIgniter Forums (https://forum.codeigniter.com)
+-- Forum: CodeIgniter 4 (https://forum.codeigniter.com/forumdisplay.php?fid=28)
+--- Forum: CodeIgniter 4 Discussion (https://forum.codeigniter.com/forumdisplay.php?fid=31)
+--- Thread: IDEA: Default core class extensions (/showthread.php?tid=73142)



IDEA: Default core class extensions - MGatner - 03-23-2019

Hi all! So after this thread we decided to create BaseController, an extended controller shipping by default in /app/Controllers (see the pull request). I'm now working on my next library which includes extending the Model and Entity classes, which got me thinking...  Should we be providing default extensions in /app for the common core classes? It seems a win-win for everyone:
  • the new/casual user won't extend anyways, it won't make a difference
  • the intermediate user (or one needing to extend later in development) will appreciate having it ready and their classes already using the /app versions
  • the power user will implement the provided classes, make their own extended versions, or bypass it and extend the /system version if they are worried about the extra parent class
Controller, Model, and Entity are the ones that stick out to me, but maybe also Filters? others?
Curious everyone's thoughts on this...


RE: IDEA: Default core class extensions - MGatner - 03-24-2019

No objections, splendid! Smile
Since it won’t be much work I might just go ahead and submit a pull request and we can discuss it there.


RE: IDEA: Default core class extensions - John_Betong - 03-24-2019

Possibly make Url Segments available?