Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace require with require_once in entryPoint.php #554

Open
wants to merge 1 commit into
base: hotfix
Choose a base branch
from

Conversation

JoshMcCullough
Copy link

Description

I am not a "PHP Guy", so this could be wrong. However, it fixes error messages such as:

PHP Fatal error: Cannot declare class TemplateText, because the name is already in use

Same with LanguageManager.php and VardefManager.php, and likely but not confirmed, TemplateText.php.

Motivation and Context

I noticed the error while authenticating via the API, when the server returned 500.

How To Test This

Execute the below command and verify that the above error message does not appear in the log.

curl --location --request GET 'localhost/service/v4_1/rest.php' \
--header 'Content-Type: application/x-www-form-urlencoded' \
--data-urlencode 'method=login' \
--data-urlencode 'input_type=JSON' \
--data-urlencode 'response_type=JSON' \
--data-urlencode 'rest_data={"user_auth":{"user_name":"admin","password":"password"},"application_name":"test"}'

Types of changes

  • [ x ] Bug fix (non-breaking change which fixes an issue)
  • [ _ ] New feature (non-breaking change which adds functionality)
  • [ _ ] Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

  • [ x ] My code follows the code style of this project found here.
  • [ _ ] My change requires a change to the documentation.
  • [ x ] I have read the How to Contribute guidelines.

@SuiteBot
Copy link

SuiteBot commented Oct 10, 2024

CLA assistant check
All committers have signed the CLA.

@JoshMcCullough JoshMcCullough changed the title replace require with require_once in entryPoint.php replace require with require_once in entryPoint.php Oct 11, 2024
@JoshMcCullough
Copy link
Author

Can we get this simple fix merged soon, please?

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