search

OpenLiberty / liberty-tools-eclipse

Eclipse Public License 2.0
11 stars 8 forks source link

Missing '}' flagged by LSP4MP validation for property referencing inner class name #501

Open scottkurz opened 6 months ago

scottkurz commented 6 months ago

image

From the 'guide-microprofile-reactive-messaging' guide, we have in microprofile-config.properties:

mp.messaging.outgoing.systemLoad.value.serializer=io.openliberty.guides.models.SystemLoad$SystemLoadSerializer

which gets flagged with the error: image

scottkurz commented 6 months ago

mp.messaging.outgoing.systemLoad.value.serializer=io.openliberty.guides.models.SystemLoad$SystemLoadSerializer

This use of an "inner class" as property value is getting flagged as an error by Liberty Tools validation, based on the LSP4MP code.

If I look at the Git history in the LS project: https://github.com/eclipse/lsp4mp/commit/8512c09f4d823e451405eb95aa342d46554ec347 In many MicroProfile implementations, ${other.property} can be used to refer to the value of an already defined property when defining the value of another property.

Does Liberty support something like prop1=${prop2}?