Comments (7)
Looks like a bug to me. Either we need to change the catch logic or or pass the name to the method. I don't know the context of this logic off the top of my head. @sneakyvv?
from ux.
Same thing @kbond :)
I'd code that, but i have no idea how to test that correctly, with LiveComponents, ...
$compiler->write('$this->extensions[')
->string(ComponentExtension::class)
->raw(']->finishEmbeddedComponentRender(')
->string($this->getAttribute('component'))
->string(')')
->raw(";\n")
;
from ux.
Did you hit this try...catch and get an \ArgumentCountError
thrown?
from ux.
Hmm where ? Around this one or in throwRuntimeError ? Sorry i'm not sure how to handle that.. :|
from ux.
I'm just wondering how you stumbled upon this bug. Just by looking at code or did an app actually run this and throw an exception.
(I'm trying to gauge how important it is to require the name here - can we just remove the try catch and let the caught exception be thrown as normal?)
from ux.
Reading code, and PhpStorm alerted me with some red wavy line :)
from ux.
I have to leave, i can try your solution in a couple of hours
from ux.
Related Issues (20)
- [stimulus-bundle] Symfony 6.4 and Assetmapper (importmap:require @hotwired/stimulus fails) HOT 1
- [Autocomplete] Security considerations HOT 1
- [Autocomplete] Performance considerations HOT 1
- [AssetMapper 6.4] Question about bundle paths for import HOT 5
- [Autocomplete] SortableJS Sorting Issue HOT 10
- Unable to find custom template for UX components with <twig:component> syntax HOT 6
- bin/console config:dump not supported? HOT 2
- [Live] Dependent fields demo: add dependency requirement HOT 2
- [LiveComponent] Force POST method when model is changed HOT 4
- [StimulusBundle] Error with AssetMapper 6.4 & StimulusBundle HOT 4
- [LiveComponent] Normalize true/false model values? HOT 1
- [LiveComponent] Execute model:set hook even for props changed on the server HOT 2
- [StimulusBundle] Failed to resolve module specifier "@symfony/stimulus-bridge". HOT 3
- [AssetMapper] link tags not generated in 6.4-dev HOT 1
- [TurboBundle] Incompatible WebpackEncoreBundle on PHP 8.2 ?
- [Translator] Filter dumped domains HOT 2
- [Translator] Filter dumped locales HOT 3
- Demo with both AssetMapper and Webpack (6.4/7) HOT 7
- Symfony/UX-autocomplete npm package not installable in docker container HOT 3
- [LiveCollection] Live collection breaks autocomplete input
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ux.