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

[BUGFIX] Access class property closure correctly #1085

Merged
merged 1 commit into from Dec 20, 2022
Merged

[BUGFIX] Access class property closure correctly #1085

merged 1 commit into from Dec 20, 2022

Conversation

eliashaeussler
Copy link
Contributor

Accessing the property closure directly throws an exception:

> typo3cms install:generatepackagestates
                                                                               
  [ Error ]                                                                    
  Call to undefined method Helhum\Typo3Console\CompatibilityClassLoader::includeFile()                                                                    
                                                                               
Exception trace:
#0 ()
   app/vendor/helhum/typo3-console/Classes/Compatibility/CompatibilityClassLoader.php:66
#1 Helhum\Typo3Console\CompatibilityClassLoader->loadClass()
#2 spl_autoload_call()
   app/vendor/helhum/typo3-console/Classes/Console/Core/Kernel.php:94
#3 Helhum\Typo3Console\Core\Kernel->handle()
   app/vendor/helhum/typo3-console/Scripts/typo3-console.php:31
#4 {closure}()
   app/vendor/helhum/typo3-console/Scripts/typo3-console.php:33
#5 require()
   app/vendor/helhum/typo3-console/typo3cms:2
#6 include()
   app/vendor/bin/typo3cms:115

Related: #1082
Resolves: #1081

@helhum
Copy link
Member

helhum commented Dec 20, 2022

Uhm yes, sorry. I should have fixed the CI for 6.x branch...

@helhum helhum merged commit aa4bf52 into TYPO3-Console:main Dec 20, 2022
@eliashaeussler eliashaeussler deleted the fix/closure branch December 20, 2022 16:31
eliashaeussler added a commit to eliashaeussler/typo3-form-consent that referenced this pull request Jan 9, 2023
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.

Not compatible with composer 2.5.0?
2 participants