Skip to content

fix: import the fractor configuration if found in fractor.php file#5

Closed
mgerdes-atl wants to merge 3 commits intoandreaswolf:mainfrom
mgerdes-atl:main
Closed

fix: import the fractor configuration if found in fractor.php file#5
mgerdes-atl wants to merge 3 commits intoandreaswolf:mainfrom
mgerdes-atl:main

Conversation

@mgerdes-atl
Copy link
Copy Markdown

Hi @andreaswolf ,

thanks for the inspiring speech at t3chh .
I just wanted to try out the lifter including fractor and came across the error

FRACTOR: Fatal error: Uncaught TypeError: a9f\Fractor\Configuration\FractorConfigurationBuilder::__invoke(): Argument #1 ($containerConfigurator) must be of type Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator, a9f\Fractor\Configuration\FractorConfigurationBuilder given, called in /var/www/html/lifter/vendor/a9f/lifter/res/run_fractor.php on line 19 and defined in /var/www/html/lifter/vendor/a9f/fractor/src/Configuration/FractorConfigurationBuilder.php:49

I change the run_fractor to use import method. Hopefully that should do the trick

mgerdes-atl and others added 3 commits July 24, 2024 13:28
Otherwise there was an error:
```FRACTOR: Fatal error: Uncaught TypeError: a9f\Fractor\Configuration\FractorConfigurationBuilder::__invoke(): Argument andreaswolf#1 ($containerConfigurator) must be of type Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator, a9f\Fractor\Configuration\FractorConfigurationBuilder given, called in /var/www/html/lifter/vendor/a9f/lifter/res/run_fractor.php on line 19 and defined in /var/www/html/lifter/vendor/a9f/fractor/src/Configuration/FractorConfigurationBuilder.php:49```
adjust composer dependency to allow a9f/fractor version ^0.4
@andreaswolf
Copy link
Copy Markdown
Owner

Thanks for the contribution. I reworked the PR branch to remove the merge of main, and adjusted the commit messages; however, I could not push to the PR, so I instead put the branch into my repo as pr-5-reworked. @mgerdes-atl if you like, you can just reset your PR branch to that state, or I'll create a new PR and merge that instead.

@andreaswolf
Copy link
Copy Markdown
Owner

Closing this in favor of #13

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