Skip to content

Conversation

@simonLeary42
Copy link
Collaborator

@simonLeary42 simonLeary42 commented Sep 24, 2025

UnitySite contains all HTTP related utilities, except for testValidSSHKey. I moved testValidSSHKey to utils.php.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the UnitySite class to UnityHTTPD and moves the testValidSSHKey method from UnitySite to the global utils.php file. The change improves the organization of HTTP-related utilities under a more descriptive class name.

  • Renamed UnitySite class to UnityHTTPD throughout the codebase
  • Moved testValidSSHKey function from the class to global scope in utils.php
  • Updated all import statements, class references, and method calls accordingly

Reviewed Changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
Multiple PHP files Updated import statements from UnitySite to UnityHTTPD
Multiple PHP files Changed static method calls from UnitySite:: to UnityHTTPD::
resources/lib/UnityHTTPD.php Renamed class and removed testValidSSHKey method
resources/lib/utils.php Added testValidSSHKey function and required import
webroot/panel/account.php Updated testValidSSHKey call to use global function
Test files Updated class names and method references in tests
Documentation files Updated references to new class name

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

@simonLeary42 simonLeary42 merged commit fdaa31b into main Sep 26, 2025
2 checks passed
@simonLeary42 simonLeary42 deleted the UnityHTTPD branch September 26, 2025 21:18
@simonLeary42
Copy link
Collaborator Author

58fbaa1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants