Age | Commit message (Collapse) | Author | Files | Lines |
|
It doesn't make sense to do a load->database() call
but not connect to the database. IIRC there was more
stuff in CI_DB_driver::initialize() at some point,
so that was probably the reason why the setting
existed in the first place. However, now it only
results in users making invalid bug reports because
they don't understand the feature ...
Examples during just the past 2 weeks: #3571 #3601 #3607
|
|
|
|
|
|
|
|
'Class Loaded' type of messages flood log files when
log_threshold is set to 2 (debug). They're now logged
as 'info' level.
This is manually applying PR #1528, which was created
to do the same thing, but became outdated.
|
|
- Remove PHP version from license notices
- Bump year number in copyright notices
- Recommend PHP 5.4 or newer to be used
- Tell Travis-CI to test on PHP 5.3.0 instead of the latest 5.3 version
Related: #3450
|
|
|
|
|
|
First stab at DB driver reference. Methods are listed in order encountered in the source file.
Parameter or return types in DB_driver updated as needed, and reflected here.
TOC entry added to bottom of database index page.
Signed-off-by:James L Parry <jim_parry@bcit.ca>
|
|
|
|
|
|
|
|
|
|
Close #3239
|
|
|
|
|
|
cleaned `return true; return true;` construction
|
|
The core shouldn't depend on constants that are not defined by itself
|
|
|
|
|
|
|
|
Update copyright notices from 2013 to 2014.
And update one calendar example in user_guide from year 2013/2014 to
2014/2015.
|
|
|
|
Looks like an unnecessary commit was made by the author just as I was clicking the Merge button. :)
|
|
|
|
@narfbg "It has parenthesis, so use the IN / NOT IN pattern as a base."
|
|
|
|
|
|
|
|
|
|
|
|
Better entropy, faster.
Also fixed a few "it's" typos.
|
|
|
|
- Session events logging (debug)
- Bug fix for OCI8 method stored_procedure()
|
|
These were the last two calls of class_exists() without the $autoloader = FALSE argument.
|
|
Fixed problem for transaction test mode.
|
|
|
|
|| to OR
remove the empty line at EOF
|
|
Documentation: fix some outdated paths
|
|
|
|
|
|
Signed-off-by: RJ garcia <rj@bighead.net>
|
|
trans_complete function is committed in test mode.
Because any database drivers are set _trans_failure in test_mode,
And trans_complete function is not evaluate _trans_failure.
|
|
Re-allocated exit status codes according to three references, which follow:
BSD sysexits.h:http://www.gsp.com/cgi-bin/man.cgi?section=3&topic=sysexits
GNU recomendations:http://www.gnu.org/software/libc/manual/html_node/Exit-Status.html
Bash scripting:http://tldp.org/LDP/abs/html/exitcodes.html
The GNU recommendations stem from and expand upon the standard C/C++ library (stdlibc)
definitions, while also suggesting some best-practice conventions which happen to prevent
exit status code collisions with bash, and probably other shells.
The re-allocated codes are now mapped to constant values, set in *application/config/constants.php*,
and used throughout the CodeIgniter core. They would additionally be used in *index.php*,
but the constants file hasn't been loaded at that point, so the integer values are used
instead, and a comment follows each such use with amplifying information on why that
particular value was selected.
Finally, the errors documentation has been updated accordingly.
Signed-off-by: Daniel Hunsaker <danhunsaker@gmail.com>
|
|
feature/exit-status
|
|
DB_driver's trans_complete exception fix
|
|
checks ...
|
|
Note: The Driver libary tests seem to depend on that, so one occurence in CI_Loader is left until we resolve that.
|
|
|
|
Specific codes are as follows, but can easily be changed if a different order/breakdown makes more sense:
- 0: Success; everything executed as planned
- 1: Configuration Error; something is wrong with/in the configuration file(s)
- 2: Class Not Found; what it says
- 3: Driver Method Unsupported; the method you're trying to use on a Driver doesn't exist
- 4: File Not Found; 404 error
- 5: Database Error; something is broken in the database somewhere
- 6: Invalid Input; the user attempted to submit a request with invlaid characters in 1+ key names
7 through 26 are reserved for future use
- 27: Generic Error; generated by show_error() when the status code is >= 100
28 through 127 are errors generated by user applications, normally by using show_error() with a status code below 100
128 through 254 should not be used by applications, as they are reserved by system-level functions
- 255: PHP Fatal Error; automatically generated by PHP for fatal errors, and therefore not allowed for our use
Status codes below 100 are shifted up by 28 to place them in the user error range. It may make more sense to have these codes
left alone and instead shift the CI errors into the 101 through 127 space, but that's not what I opted for here.
It would probably also be a good idea to replace the hard-coded numbers with constants or some such, but I was in a bit of a
hurry when I made these changes, so I didn't look around for the best place to do this. With proper guidance, I could
easily amend this commit with another that uses such constant values.
Signed-off-by: Daniel Hunsaker <danhunsaker@gmail.com>
|