Skip to content

Update symfony usage #87

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

Closed
wants to merge 1 commit into from
Closed

Update symfony usage #87

wants to merge 1 commit into from

Conversation

andig
Copy link
Contributor

@andig andig commented Jan 28, 2018

No description provided.

Make sure `httpkernel` can access the Symfony session service. Add this to `services.yml`:

session.handler:
public: true

Choose a reason for hiding this comment

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

Does it work? I've tried it on symfony 4 and only solution that works was - #60 (comment)

Choose a reason for hiding this comment

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

after adding these lines, following error appears:

In CheckDefinitionValidityPass.php line 60:
                                                                                                                                                                                                                    
  The definition for "session.handler" has no class. If you intend to inject this service dynamically at runtime, please mark it as synthetic=true. If this is an abstract definition solely used by child definit  
  ions, please add abstract=true, otherwise specify a class to get rid of this error.    

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could you check if declaring the service as synthetic works? See https://symfony.com/doc/current/service_container/synthetic_services.html for how to.

@andig
Copy link
Contributor Author

andig commented Jan 29, 2018

Does it work? I've tried it on symfony 4 and only solution that works was - #60 (comment)

@bronek89 that implementation is on the application side, not php-pm, right? Then we'd need to add this description to this PR?

@marcj
Copy link
Member

marcj commented Feb 10, 2018

I've fixed that in master, no need for a workaround like that anymore.

@marcj marcj closed this Feb 10, 2018
@andig andig deleted the s4 branch February 10, 2018 09:53
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.

3 participants