jte-intellij

jte-intellij copied to clipboard

IntelliJ Plugin @import x/y/z in .kte file missing the @

1

In the IntelliJ Idea Plugin, working with kotlin (kte) templates the @ for @import is missing, if i use the import completion from IntelliJ. 

schowave

Hi, (there is no "discussion" section hence creating an issue) Would it be possible to make Kotlin dependency optional? I'm not really using it and while trying to make IDEA...

woj-tek

java.lang.IllegalArgumentException: Argument for @NotNull parameter 'file' of com/intellij/openapi/fileEditor/impl/FileDocumentManagerBase.getDocument must not be null at com.intellij.openapi.fileEditor.impl.FileDocumentManagerBase.$$$reportNull$$$0(FileDocumentManagerBase.java) at com.intellij.openapi.fileEditor.impl.FileDocumentManagerBase.getDocument(FileDocumentManagerBase.java) at org.jusecase.jte.intellij.language.JteJavaContentManipulator.handleContentChange(JteJavaContentManipulator.java:19) at org.jusecase.jte.intellij.language.JteJavaContentManipulator.handleContentChange(JteJavaContentManipulator.java:15) at com.intellij.psi.ElementManipulators.handleContentChange(ElementManipulators.java:65) at com.intellij.psi.impl.source.tree.injected.changesHandler.CommonInjectedFileChangesHandler.updateHostElement(CommonInjectedFileChangesHandler.kt:170) at com.intellij.psi.impl.source.tree.injected.changesHandler.CommonInjectedFileChangesHandler.updateHostOrFail(CommonInjectedFileChangesHandler.kt:149) at com.intellij.psi.impl.source.tree.injected.changesHandler.CommonInjectedFileChangesHandler.commitToOriginal(CommonInjectedFileChangesHandler.kt:122) at com.intellij.codeInsight.intention.impl.InjectionEditServiceImpl$1.documentChanged(InjectionEditServiceImpl.java:34)...

jankod

Add Import for nested classes does not work

1

Noticed in IntelliJ 2023.2.2, when having a nested class inside a jte template, for instance @param Foo.Bar bar and alt+enter => "Add import for Foo.Bar" does not add an import...

casid

I'm not sure if that would also happen with non-eap builds, but this is shown in 2024.1 eap4: ``` com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'org.jusecase.jte.intellij.language.convert.JteConvertFromJspAction'...

jansorg

Cannot type closing bracket

1

Example: ``` @template.foo( name = "name", label = localize("name.label"), readonly = action.getName() == null || action.getName().isEmpty() ) ``` Typing a closing bracket after isEmpty() doesn't do anything.

casid

``` @template.foo( name = "name", label = localize("name.label"), readonly = action.getName() != null ) ``` `action.getName() !` is highlighted as error.

casid

Exception on "Add import for"

1

Noticed in IntelliJ 2023.2.2, when having a nested class inside a jte template, for instance `@param Foo.Bar bar` and alt+enter => "Add import for Foo.Bar" causes an exception and does...

casid

``` @template.core.components.paragraph(content = @` ${localize("foo"}${localize("bar")}${baz()} `) ``` Misses a closing brace after `localize("foo"`. Correct is: ``` @template.core.components.paragraph(content = @` ${localize("foo")}${localize("bar")}${baz()} `) ``` This should be highlighted by the plugin.

casid

Tagged templates are not supported by IntelliJ plugin

3

[Tagged templates](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals#tagged_templates) seems to be unrecognized by the IntelliJ plugin:  The code compiles properly tho. To be fair, I'm not sure if it interferes with one of the jte...

dzikoysk