tomblachut / svelte-intellij

Svelte components in WebStorm and friends
MIT License
485 stars 38 forks source link

Update SvelteJSReferenceExpressionResolver for 223 intellij branch #293

Closed ulitink closed 2 years ago

ulitink commented 2 years ago

After changes 05f3877df7ea0b8f4a8c95a8cc3ba588a57cbb2e and 549174d83c3b5015755b0dfab5b4b5e480b8e394 SvelteJSReferenceExpressionResolver and SvelteTypeScriptReferenceExpressionResolver should be adapted to the new API.

To keep them working in the previous way, one of the option is to remove override from SvelteTypeScriptReferenceExpressionResolver.createLocalResolveProcessor, and in SvelteJSReferenceExpressionResolver.resolve make it work similar to how it's done in SvelteTypeScriptReferenceExpressionResolver, i.e. if resolve is local, call createLocalResolveProcessor (also need to remove override there).

ulitink commented 2 years ago

Fixed in https://github.com/tomblachut/svelte-intellij/commit/29ed585cd7cd8765774507da3e56e6ebe0c5848c