[#IDEADEV-22587] no completion/validation for some attributes of the 'lang' namespace

IDEA: Development

no completion/validation for some attributes of the 'lang' namespace

Created: 08 Oct 07 20:04 Updated: 06 Dec 07 18:52

Component/s:

J2EE.Spring

Fix Version/s:

Selena 7.0.2

Original Estimate:

Unknown

Remaining Estimate:

Unknown

Time Spent:

Unknown

File Attachments:

lang-namespace-various.patch (6 kb)

Issue Links:

Duplicate

This issue is duplicated by:
	~~IDEA-15716~~ Support for property name completion ...

Build:	7,330
Fixed in build:	7,508

Description

« Hide

no resolving for 'scope' attribute (all)
no resolving for 'script-interfaces' attribute (jruby,bsh)
no resolving for 'script-source' attribute (all)
attribute 'refresh-check-delay' is declared as 'xsd:long' but modeled as 'Integer'
no resolving for 'customizer-ref' attribute (groovy)

Attached patch solved the first 3.
It also removes from methods from com.intellij.spring.model.xml.lang.DynamicScript which were duplicate to methods on com.intellij.spring.model.xml.lang.SimpleScript.

I assume the fix for (4) should be made in DOM internals: at the moment it doesn't support Long type.

(5) can be easily fixed using the annotation-driven Converter from the patch attached to ~~IDEADEV-17510~~: the specified type should be 'org.springframework.scripting.groovy.GroovyObjectCustomizer'

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Taras Tielkes - 08 Oct 07 20:23 - edited

In addition, for <lang:jruby> and <lang:bsh>, the value of the 'script-interfaces' attribute could be used to implement getBeanClass(). While only the first specified interface can be returned through this method, it would allow bean reference completion from other beans (IDEADEV-18981) to work in most cases.

[ Permlink | « Hide ]

t800t8 - 15 Oct 07 05:31

No resolving for "name" attribute of <lang:property>

[ Permlink | « Hide ]

Taras Tielkes - 15 Oct 07 11:41

To support "name", IDEA (or a plugin) would have to parse the groovy/jruby/bsh language.

The other attributes mentioned in the issue description and first comment do not require such extensive support. In other words they could be supported in a bugfix release without much risk.

[ Permlink | « Hide ]

Dmitry Avdeev - 17 Oct 07 14:31

Thanks Taras. Your patch is applied.