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

ITKDev: Fixes must not be accessed before initialization #16

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

cableman
Copy link
Contributor

@cableman cableman commented Nov 27, 2024

Link to ticket

https://leantime.itkdev.dk/?tab=timesheet#/tickets/showTicket/3103

Description

Error: Typed property Drupal\os2web_datalookup\LookupResult\CprLookupResult::$name must not be accessed before initialization in Drupal\os2web_datalookup\LookupResult\CprLookupResult->getFieldValue() (line 676 of /app/web/modules/contrib/os2web_datalookup/src/LookupResult/CprLookupResult.php)

This results in a 500 back to the user or service in PHP >= 8.3 (PHP message: Uncaught PHP Exception).

Screenshot of the result

N/A

Checklist

  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

If your code does not pass all the requirements on the checklist you have to add a comment explaining why this change
should be exempt from the list.

Additional comments or questions

N/A

@cableman cableman requested a review from jekuaitk November 27, 2024 11:43
@jekuaitk jekuaitk merged commit cadc545 into OS2web:main Dec 6, 2024
3 of 4 checks passed
@jekuaitk jekuaitk deleted the feature/no-init-error branch December 6, 2024 09:00
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