[#258] Refactor @Parameter to include iri instead of (urlPrefix, name)

[blcham]

Fixes #258

[blcham]

Build fails due to:

Note it is related to mvn module:

  <name>SPipes Module Annotation Processor</name>
  <artifactId>s-pipes-module-creator-maven-plugin</artifactId>

[palagdan]

@blcham
There is an error here because the annotation has only a 'comment' parameter in CKAN module, but it should also include an 'iri' parameter. I'll try to investigate.

I don't understand. readConstraintsFromClass traverses the fields and retrieves the annotations, but the annotations already lack the 'iri' field...

Maybe the issue is due to an outdated dependency in the s-pipes-module-creator-maven-plugin, which uses a previous implementation of @Parameter with urlPrefix and name?
:

  <dependency>
            <groupId>cz.cvut.kbss</groupId>
            <artifactId>s-pipes-core</artifactId>
        </dependency>

[blcham]

@palagdan

Thanks for the hint, I commit-ed fix of the issue ... not going to search which was the biggest issue but I assume the most important part of what I did was:

Please review the PR.

[palagdan]

I've reviewed and noticed only one issue with raw values. I will do this.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[palagdan]

I suggest using constants instead of raw values here.

[blcham]

Suggestions above will be resolved in subsequent task -- #275