- Resolve randome apache segfault due to recursion.
- Also, remove additional references to export user
- Change the way the guest_account id is gathered.
- Change elseif format
- Remove trailing white-spaces
If the plugin architecture does not exist, install will fail. Also
found php function version_compare not working in PHP 5.3.x.
There were additionally issues with the upgrade functions as well.
Resolving those too.
-feature: CLOG timestamp is now formatted as defined in settings
-feature: CLOG can show loginformation from rotated logfiles
-Added GDC_DOT to global_constants.php and global_arrays.php, and refactored the usage of global $datechar
- Optimisations of query calls and function calls in loops
- Removed unused function calls and variables
- Refactored large if statements into switch statements
- Applied early return
- Fixed alignments and removed unnecessary parentheses
- Added missing docblock to new function
- Added configurable max_display_rows to settings panel
- Applied max_display_rows to code instead of MAX_DISPLAY_ROWS
- Removed define MAX_DISPLAY_ROWS
- CLOG is now paginated
- System Utilities - Log administration is now paginated
- Bugfix in classic template paginator; when previous option appears, the paginator grows in height
- Fixes memory usage; in the past the whole logfile would be read, now only the needed amount of rows
- Removed some unused variables
- Small optimisations in showing CLOG
- Added and moved code to CACTI_VERSION constant
- Merged cacti versions and hash codes array to have one place to add versions
- Moved cacti version to file to facitate automation needs
- Cleaned up cli database upgrade script
- Data collection warnings when using cmd.php
- Replace <path_php_binary> in data input methods
- Spaces adjacent to double quotes are eliminated durate data input
method import
- Horor the column setting in graph tree view mode
- Replaced deprecated each() for foreach()
- Removed unnecessary reset() calls used for resetting pointers of an array before iterating by a foreach() loop
Alignment for:
- array_rekey()
- is_hex_string()
- get_graph_group(()
- calculate_percentiles()
- cacti_oid_numeric_format()
- CactiErrorHandler()
read_default_user_setting() and read_default_config_option()
- refactored each() info foreach()
- removed unnecessary parentheses
tail_file()
- small optimisiational refactoring
get_full_script_path()
- alignment
- optimised str_replace() into one execution
stri_replace()
- strlen() into variable
clean_up_path()
- optimised if statement by reordering
generate_data_source_path()
- Removed unneeded variable declarations
- Limited query result (because of fetch cell returning one cell)
generate_graph_def_name()
- put strval($graph_item_id) into variable
move_graph_group()
- moved query to lower position
- alignment
- foreach instead of each() (each() will be deprecated in php 7.2.0)
exec_into_array()
- I believe this code was a self-implemented array_values()
draw_login_status()
- declared only necessary fields in querystatement
draw_navigation_text()
- removed/repositioned unnecessary declaration of variables
- declared only necessary fields in querystatement
resolve_navigation_variables()
- refactored heavy function call stored into variable
get_hash_graph_template()
get_hash_data_template()
get_hash_data_input()
get_hash_cdef()
get_hash_vdef()
get_hash_data_query()
- refactored into switch statement (33% faster)
- added default case
generate_hash()
- removed global $config;
Cleanup of unused functions:
- is_valid_email()
- timer_start()
- timer_end()
- strip_newlines()
Small refactor in cacti_log():
- reduce function calls for read_config_option()
- refactor of loglevel if statement
- unnecessary parentheses
- syslog section refactor
- strpos() is 33% quicker than substr_count() in this case
draw_navigation_text():
- get result in 1 query instead of 2
auth.php:
- placed result of get_selected_theme() in var
functions.php:
- mailer():
- fetch date from setting by caching method
- read_config_option('settings_from_email') unnecessary
- 2x undefined function $mail->replyTo() (should be addReplyTo())
- calculate_percentiles()
- Final return statement has undeclared variable; I believe this should be results instead of result, but could not reproduce
index.php:
- only fetch the fields needed for querying external_links
- 2x </td> instead of </tr>
patrick