Add Signal Handling, Cacti process registration, and php-cs-fixer

[TheWitness]

This pull request does the following:

• Marks the files poller_servcheck.php and servcheck_process.php executable and adds shebang
• Add signal handling to both files to properly shut down each other when signaled
• Implement the standard Cacti process control with timeouts. Calculate the timeouts as algorithmically defined in the design.
• Apply Cacti 1.3+ PSR rules to the code for formatting
• Move some code into functions
• Improve debug statistics by adding memory and cpu usage.

[Copilot]

Pull request overview

This pull request implements signal handling, Cacti process registration with algorithmic timeouts, and applies PSR formatting standards to the servcheck plugin code.

Changes:

• Adds signal handlers (SIGTERM, SIGINT, SIGUSR1) to gracefully shut down child processes
• Implements Cacti process control with calculated timeouts based on test parameters
• Adds shebang lines to poller_servcheck.php and servcheck_process.php
• Converts array syntax from array() to [] (PSR-12 standard)
• Removes obsolete plugin_servcheck_processes table
• Adds configurable concurrent process count setting

Reviewed changes

Copilot reviewed 22 out of 23 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
poller_servcheck.php	Adds signal handlers, process registration/management, and concurrent process control
servcheck_process.php	Adds signal handlers, process registration with timeout calculation, and performance statistics
setup.php	Removes obsolete processes table, adds concurrent processes setting, applies PSR formatting
servcheck_test.php	PSR formatting changes (array syntax, spacing, comments)
servcheck_*.php	PSR formatting changes across multiple files
includes/test_*.php	PSR formatting changes in test modules
includes/functions.php	PSR formatting and minor improvements

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 22 out of 23 changed files in this pull request and generated 4 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 22 out of 23 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

SQL syntax error: There's a comma after 'WHERE enabled = "on"' when it should be removed or the SQL statement is incomplete. This will cause a database query error.