Fix: PHP spam errors in file_manager

[Squidly271]

Summary by CodeRabbit

• Bug Fixes
  • Improved initialization handling in the core processing loop to ensure proper state management during system startup and recovery scenarios, preventing potential edge-case issues.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 98b21056-8a02-4a10-afaa-b790b35bacd7

📥 Commits

Reviewing files that changed from the base of the PR and between 583b8a1 and 6b44813.

📒 Files selected for processing (1)

• emhttp/plugins/dynamix/nchan/file_manager

---

Walkthrough

A safety adjustment in the file_manager script ensures the $pid variable is properly initialized on every iteration of the main loop. The change introduces a guarded initialization condition that affects control flow only on the first iteration after restart or when the variable is undefined.

Changes

Cohort / File(s)	Summary
Variable Initialization emhttp/plugins/dynamix/nchan/file_manager	Added guarded initialization of $pid variable in main loop to ensure it is defined on every iteration without altering action handling or processing logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A tiny hop, a guard so fine,
Where $pid learns to shine,
One line added, one removed with care,
Safety blooms in the morning air! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: fixing PHP errors (specifically spam errors related to undefined $pid) in the file_manager script.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

🔧 PR Test Plugin Available

A test plugin has been generated for this PR that includes the modified files.

Version: 2026.03.11.1845
Build: View Workflow Run

📥 Installation Instructions:

Install via Unraid Web UI:

1. Go to Plugins → Install Plugin
2. Copy and paste this URL:

https://preview.dl.unraid.net/pr-plugins/pr-2569/webgui-pr-2569.plg

3. Click Install

Alternative: Direct Download

• 📦 Download PLG
• 📦 Download TXZ

⚠️ Important Notes:

• Testing only: This plugin is for testing PR changes
• Backup included: Original files are automatically backed up
• Easy removal: Files are restored when plugin is removed
• Conflicts: Remove this plugin before installing production updates

📝 Modified Files:

Click to expand file list

emhttp/plugins/dynamix/nchan/file_manager

🔄 To Remove:

Navigate to Plugins → Installed Plugins and remove webgui-pr-2569, or run:

plugin remove webgui-pr-2569

---

_{🤖 This comment is automatically generated and will be updated with each new push to this PR.}