2.9.2 114 employees change pass

[BeHuK]

No description provided.

[roomote-v0]

Rooviewer   See task

I've reviewed this PR containing 30 commits with various bug fixes and improvements. Overall, the code quality is reasonable with appropriate fixes for SQL errors, display issues, and enhancements to API documentation generation.

Review Summary

Key Changes:

• Fixed SQL column name bug in LDAP authentication (id → u_id)
• Improved checkbox2 control with "select all" functionality
• Enhanced OpenAPI schema generation with better module support
• Added custom sequence interface for drag-and-drop sorting
• Fixed import field template assignment
• Added getFilesNormalize() method for file upload handling
• Improved error messages for missing API endpoints

Potential Issues:

While most changes are solid improvements, there's one potential issue worth noting:

In inc/classes/OpenApiSpec.php around lines 254-259, the variable $schema is accessed outside its definition scope. The variable is only defined inside the if (file_exists($controller_path)) block (line 246), but lines 254-259 attempt to use it unconditionally. This could cause an "Undefined variable" error if the controller file doesn't exist. Consider initializing $schema = [] before line 232 to prevent this issue.

Recommendation: ✅ Approve with minor suggestion to initialize $schema variable.

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}