fix for Cannot use bool as array

[mambax7]

No description provided.

[Copilot]

Pull Request Overview

This PR fixes a PHP error "Cannot use bool as array" by updating array destructuring patterns in while loops. The issue occurs when fetchRow() returns false (indicating no more rows) but the code attempts to destructure it as an array.

• Replaced direct array destructuring assignments in while loop conditions with proper null checks
• Added intermediate variable assignments to safely handle fetchRow() return values
• Maintained existing logic flow while preventing the type error

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
htdocs/xoops_lib/modules/protector/admin/center.php	Fixed array destructuring in database result loops and corrected control flow logic
htdocs/pda.php	Updated while loop to safely handle fetchRow return value
htdocs/modules/system/include/update.php	Fixed array destructuring in template ID collection loop
htdocs/modules/system/class/maintenance.php	Added proper null check for fetchRow in database dump method
htdocs/modules/profile/class/visibility.php	Updated field ID collection loop to handle fetchRow safely
htdocs/kernel/session.php	Fixed session data retrieval with proper null checking
htdocs/class/xoopstree.php	Updated multiple tree traversal methods to handle fetchRow safely
htdocs/class/model/stats.php	Fixed array destructuring in statistics counting methods
htdocs/class/model/joint.php	Updated link count retrieval to handle fetchRow safely
htdocs/banners.php	Fixed multiple banner statistics loops to handle fetchRow safely

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The logic is inverted. The condition should check !$db->isResultSet($result) as in the original code. isResultSet() returns true for valid result sets, but you want to process the data when you have a valid result set, not when you don't.