Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added driver extension support #2029

Merged
merged 1 commit into from
Nov 25, 2012
Merged

Added driver extension support #2029

merged 1 commit into from
Nov 25, 2012

Conversation

dchill42
Copy link
Contributor

Retooling of #1787

Enables mix-and-match extension of Driver library and child classes. Extend library without requiring child extensions and vice-versa.

Session and Cache drivers updated to work with changes to valid_drivers list, which now lists child class names without parent class name included. Driver child class and file names still require parent name prefix.

Adds unit testing of Driver library base class.

Reverts loading CI_Driver in _ci_load_class() - all drivers (including Session!) must be loaded with CI_Loader->driver() NOT CI_Loader->library().

Signed-off-by: dchill42 dchill42@gmail.com

… tests

Signed-off-by: dchill42 <dchill42@gmail.com>
narfbg added a commit that referenced this pull request Nov 25, 2012
Added driver extension support
@narfbg narfbg merged commit 89ecc8a into bcit-ci:develop Nov 25, 2012
narfbg added a commit that referenced this pull request Nov 25, 2012
We have file naming conventions and any extension filename needs to match MY_<orig_filename>,
so we don't need to check for lowercase equivalents.
nonchip pushed a commit to nonchip/CodeIgniter that referenced this pull request Jun 29, 2013
nonchip pushed a commit to nonchip/CodeIgniter that referenced this pull request Jun 29, 2013
We have file naming conventions and any extension filename needs to match MY_<orig_filename>,
so we don't need to check for lowercase equivalents.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants