-
Notifications
You must be signed in to change notification settings - Fork 54
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
Auto Component suggestion dont work with TextBlocks #823
Comments
Ah okay so this is the new JDK17 text blocks with 3 x double quotes. Yeah that may be something special we need to add into the plugin to support. Maybe @essobedo finds some time to work on this |
@davsclaus This kind of syntax is not even supported by Camel core so it is normal that the plugin doesn't accept it. IMHO, I'm not sure that it makes sense to allow to add carriage return characters in a middle of a URI. WDYT? |
Ah yeah before text blocks you would have to concat multiple strings "aaa" But I see a good reason to make end users organize URIs more readable if they use text blocks where parameters can be on multiple lines. So I wonder if we should then parse the text block and remove text blocks, and trim. This is also what you can do in XML where an uri attribute can be in a block |
I'm not sure if my last comment was clear enough so let me reformulate. The text block is not really the problem here, the real problem is the fact that endpoint URIs with carriage return characters are not yet supported by Camel so adding the support in the plugin if you can't start your route would be useless and potentially misleading. So if it makes sense for you to support URIs with carriage return characters then we need to implement it into Camel first and then add the support to the plugin. |
Okay we should add support for this in Camel 4.x since its java 17 based. |
Okay this will require Camel 4.2 onwards. |
Okay 4.2.0 is to be released soon that has text block support. When we have updated, can we then check if this works in this plugin. Otherwise @essobedo if you still want to work a bit on this plugin then you are welcome to help with what to fix in the plugin source for text block support |
ok, I will check it asap |
Hello team , auto suggestion is dont work for text blocks, I think title is very clear I will add a screenshot for this am I missing some topic
The text was updated successfully, but these errors were encountered: