Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Path fixed#5507
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Path fixed#5507
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Path fixed since to TwigBridge v2.7 changed the folder structure
Good catch@carlosreig. However, I think we should do better than hardcoding paths in the What about using reflection here to determine the path to the |
Cool! Good idea. I will work on it. |
Changed the hardcoded path by a ReflectionClass of the AppVariable class located in the TwigBridge library. |
👍 |
| // form_div_layout.html.twig file | ||
| $vendorTwigBridgeDir = | ||
| $vendorDir.'/symfony/twig-bridge/Symfony/Bridge/Twig'; | ||
| $appVariableReflection = new \ReflectionClass( '\Symfony\Bridge\Twig\AppVariable' ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Can you remove the spaces around the argument?
Done! The spaces around the parameter are removed. |
Thank you Carlos! This is a very flexible solution to fix the hardcoded path :) |
Path fixed since TwigBridge v2.7 changed the folder structure