Improve clarity of order of reading of configuration options in readme - Githubissues

startersclan / docker-hlstatsxce-daemon

Dockerized HLStatsX:CE perl daemon 🐳

GNU General Public License v3.0

4 stars 1 forks source link

Improve clarity of order of reading of configuration options in readme #2

Closed leojonathanoh closed 4 years ago

leojonathanoh commented 4 years ago

Bug

Readme isn't clear about the order of reading of configuration options.

Suggested

Actual order of reading of configuration options:

Default config file ./hlstats.conf
- https://github.com/startersclan/hlstatsx-community-edition/blob/965453909e1d28aed3abfca7f93b6c1b27a7f75d/scripts/hlstats.pl#L1739-L1784
Command line parameters
- https://github.com/startersclan/hlstatsx-community-edition/blob/965453909e1d28aed3abfca7f93b6c1b27a7f75d/scripts/hlstats.pl#L1792-L1829
(N.A. since bugged) Custom config file specified by command line parameter --configfile.
- Doesn't work because of a bug explained in https://github.com/startersclan/hlstatsx-community-edition/blob/965453909e1d28aed3abfca7f93b6c1b27a7f75d/scripts/hlstats.pl#L1821-L1826
Database configuration from hlstats_options table:
- directives: https://github.com/startersclan/hlstatsx-community-edition/blob/965453909e1d28aed3abfca7f93b6c1b27a7f75d/scripts/hlstats.pl#L1755-L1781)
- call: https://github.com/startersclan/hlstatsx-community-edition/blob/965453909e1d28aed3abfca7f93b6c1b27a7f75d/scripts/hlstats.pl#L1882)