fix: address TheWitness review feedback on PR Cacti#283

- Remove $uniqueID filter from syslog_remove query (incorrectly
  filtered removal rules by random batch marker)
- Reorder CI workflow: lint/PHPStan before integration tests
- Switch echo to print in syslog_batch_transfer.php

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>

Author: somethingwithproof

.github/workflows/plugin-ci-workflow.yml

@@ -32,65 +32,7 @@ on:
       - develop
 
 jobs:
-  quality-checks:
-    name: PHP Quality (Lint, Stan, CS)
-    runs-on: ubuntu-latest
-    steps:
-    - name: Checkout Cacti
-      uses: actions/checkout@v4
-      with:
-        repository: Cacti/cacti
-        path: cacti
-    
-    - name: Checkout Syslog Plugin
-      uses: actions/checkout@v4
-      with:
-        path: cacti/plugins/syslog
-    
-    - name: Setup PHP 8.3
-      uses: shivammathur/setup-php@v2
-      with:
-        php-version: '8.3'
-        extensions: intl, mysql, gd, ldap, gmp, xml, curl, json, mbstring
-        tools: php-cs-fixer, phpstan
-
-    - name: Check PHP Syntax (Lint)
-      run: |
-        cd cacti/plugins/syslog
-        if find . -name '*.php' -not -path './vendor/*' -exec php -l {} 2>&1 \; | grep -iv 'no syntax errors detected'; then
-          print "Syntax errors found!"
-          exit 1
-        fi
-
-    - name: Run PHP CS Fixer (Dry Run)
-      run: |
-        cd cacti/plugins/syslog
-        php-cs-fixer fix --dry-run --diff --ansi --config=../../../.php-cs-fixer.php . || true
-
-    - name: Create PHPStan config
-      run: |
-        cd cacti/plugins/syslog
-        cat > phpstan.neon << 'EOF'
-        parameters:
-          level: 5
-          paths:
-            - .
-          excludePaths:
-            - vendor/
-            - locales/
-          ignoreErrors:
-            - '#has invalid return type the\.#'
-          bootstrapFiles:
-            - ../../include/global.php
-        EOF
-
-    - name: Run PHPStan Analysis
-      run: |
-        cd cacti/plugins/syslog
-        phpstan analyse --no-progress --error-format=github || true
-
   integration-test:
-    needs: quality-checks
     runs-on: ${{ matrix.os }}
 
     strategy:
@@ -205,7 +147,7 @@ jobs:
         sed -i "s/'cacti'/'cacti'/g" ${{ github.workspace }}/cacti/plugins/syslog/config.php
         sed -i "s/'cactiuser'/'cactiuser'/g" ${{ github.workspace }}/cacti/plugins/syslog/config.php
         sed -i 's/\/\/\$/\$/g' ${{ github.workspace }}/cacti/plugins/syslog/config.php
-        sudo chmod 664 ${{ github.workspace }}/cacti/include/config.php
+        sudo chmod 664 ${{ github.workspace }}/cacti/plugins/syslog/config.php
     
     - name: Configure Apache
       run: |
@@ -236,6 +178,46 @@ jobs:
       run: |
         cd ${{ github.workspace }}/cacti
         sudo php cli/plugin_manage.php --plugin=syslog --install --enable
+    
+    - name: Check PHP Syntax for Plugin
+      run: |
+        cd ${{ github.workspace }}/cacti/plugins/syslog
+        if find . -name '*.php' -exec php -l {} 2>&1 \; | grep -iv 'no syntax errors detected'; then
+          echo "Syntax errors found!"
+          exit 1
+        fi
+
+    - name: Create PHPStan config
+      run: |
+        cd ${{ github.workspace }}/cacti/plugins/syslog
+        cat > phpstan.neon << 'EOF'
+        parameters:
+          level: 5
+          paths:
+            - .
+          excludePaths:
+            - vendor/
+            - locales/
+          ignoreErrors:
+            - '#has invalid return type the\.#'
+          bootstrapFiles:
+            - ../../include/global.php
+        EOF
+
+    - name: Install PHPStan
+      run: |
+        cd ${{ github.workspace }}/cacti/plugins/syslog
+        composer require --dev phpstan/phpstan --with-all-dependencies || composer global require phpstan/phpstan
+
+    - name: Run PHPStan Analysis
+      run: |
+        cd ${{ github.workspace }}/cacti/plugins/syslog
+        if [ -f vendor/bin/phpstan ]; then
+          vendor/bin/phpstan analyse --no-progress --error-format=github || true
+        else
+          phpstan analyse --no-progress --error-format=github || true
+        fi
+      continue-on-error: true
 
     - name: Run Plugin Regression Tests
       run: |
@@ -246,40 +228,39 @@ jobs:
             php "$test"
           done
         fi
-    
-    
+
     - name: Run Cacti Poller
       run: |
         cd ${{ github.workspace }}/cacti
         sudo php poller.php --poller=1 --force --debug
         if ! grep -q "SYSTEM STATS" log/cacti.log; then
-          print "Cacti poller did not finish successfully"
+          echo "Cacti poller did not finish successfully"
           cat log/cacti.log
           exit 1
         fi
-    
+
     - name: Populate Syslog Test Data
       run: |
         sudo chmod +x ${{ github.workspace }}/cacti/plugins/syslog/.github/workflows/populate_syslog_incoming.sh
         cd ${{ github.workspace }}/cacti/plugins/syslog/.github/workflows
         sudo ./populate_syslog_incoming.sh
 
-
     - name: force Syslog Plugin Poller
       run: |
         cd ${{ github.workspace }}/cacti
         sudo php plugins/syslog/syslog_process.php  --debug
         if ! grep -q "SYSTEM SYSLOG STATS" log/cacti.log; then
-          print "Syslog plugin poller did not finish successfully"
+          echo "Syslog plugin poller did not finish successfully"
           cat log/cacti.log
           exit 1
         fi
 
-    
     - name: View Cacti Logs
       if: always()
       run: |
         if [ -f ${{ github.workspace }}/cacti/log/cacti.log ]; then
-          print "=== Cacti Log ==="
+          echo "=== Cacti Log ==="
           sudo cat ${{ github.workspace }}/cacti/log/cacti.log
         fi
+
+