ligasgr / intellij-xquery

Plugin to support XQuery in Intellij Idea
Apache License 2.0
35 stars 23 forks source link

Annotations with literals don't work #73

Closed dirkk closed 10 years ago

dirkk commented 10 years ago

First of all: Thanks for this wonderful plugin. Looks very promising and I would very, very much love to finally have a useful XQuery tool.

The following code shows the error (at function) '%', function or variable expected, got 'function'

declare
  %rest:path("/test")
  function local:login() {
      <test />
  };

Without the string literal within the annotation (i.e. stripping "/test") everything works like a charm.

I would also love to fix this myself, but I'll first to have a look at idea plugin development, so this might take a while...

ligasgr commented 10 years ago

Hi, thanks for these very kind words and your feedback! Indeed this doesn't work. It's going to take some time before the release is ready so in the meanwhile you should be able to download and install (install plugin from disk) current version from master. It's available under this link: https://ligasgr.ci.cloudbees.com/job/intellij-xquery-master-prepare-distribution/ws/build/distributions/intellij-xquery.zip

dirkk commented 10 years ago

Wow, that was a really quick fix. The reported bug was properly fixed (thanks!). However, I would vote for a reopening as annotation with multiple string literals (e.g. declare %rest:path("/test", "test2") function local:login() {<test/>};) still don't work.

ligasgr commented 10 years ago

Ahh. Right. I forgot to check grammar while fixing it :) Thanks for pointing that out.

ligasgr commented 10 years ago

As before current version from master is available under this link: https://ligasgr.ci.cloudbees.com/job/intellij-xquery-master-prepare-distribution/ws/build/distributions/intellij-xquery.zip

dirkk commented 10 years ago

Thanks, works like a charm!