[#GRAILS-5386] cannot generate-all on a domain with nested enum (needs Groovy 1.7 nested class support) - Grails JIRA

Details

Type: Bug
Status: Reopened
Priority: Minor
Resolution: Unresolved
Affects Version/s: 1.2-M4
Fix Version/s: None
Component/s: Scaffolding
Labels:
None
Environment:
1.2-M4

Description

grails generate-all com.test.ChatResource fails with the following domain class on 1.2M4:

package com.test

class ChatResource {

String name
String value
Type type

enum Type { SCRIPT, LINK; }
}

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Graeme Rocher added a comment - 23/Nov/09 9:15 AM

works for me. Note your domain should be:

class ChatResource {

	String name
	String value
	Type type

}
enum Type { SCRIPT, LINK; }

Show

Graeme Rocher added a comment - 23/Nov/09 9:15 AM works for me. Note your domain should be:

class ChatResource {

	String name
	String value
	Type type

}
enum Type { SCRIPT, LINK; }

Hide

Permalink

justin friedl added a comment - 23/Nov/09 9:43 AM

Ok, so having the enum defined outside of the domain class works, but having it defined inside fails with:

"Domain class not found in grails-app/domain, trying hibernate mapped classes...
No domain class found for name com.test.ChatResource. Please try again and enter a valid domain class name"

This works in 1.1.1 so it still seems like a bug to me.

Show

justin friedl added a comment - 23/Nov/09 9:43 AM Ok, so having the enum defined outside of the domain class works, but having it defined inside fails with: "Domain class not found in grails-app/domain, trying hibernate mapped classes... No domain class found for name com.test.ChatResource. Please try again and enter a valid domain class name" This works in 1.1.1 so it still seems like a bug to me.

Hide

Permalink

Graeme Rocher added a comment - 23/Nov/09 9:48 AM

It is probably pure luck that it worked as the construct you are using in the example is "nested" classes which Groovy hasn't supported and won't support until Groovy 1.7 (it is one of the main features of that release). Reducing priority to minor since workaround is available.

Show

Graeme Rocher added a comment - 23/Nov/09 9:48 AM It is probably pure luck that it worked as the construct you are using in the example is "nested" classes which Groovy hasn't supported and won't support until Groovy 1.7 (it is one of the main features of that release). Reducing priority to minor since workaround is available.

Hide

Permalink

Graeme Rocher added a comment - 23/Nov/09 9:50 AM

Grails 1.3 will support Groovy 1.7 which will support this feature (nesting classes). See http://jira.codehaus.org/browse/GROOVY-3080 for reference

Show

Graeme Rocher added a comment - 23/Nov/09 9:50 AM Grails 1.3 will support Groovy 1.7 which will support this feature (nesting classes). See http://jira.codehaus.org/browse/GROOVY-3080 for reference

People

Assignee:

Unassigned

Reporter:

justin friedl

Vote (0)

Watch (2)

Dates

Created:

17/Nov/09 1:14 PM

Updated:

20/Jan/12 3:00 AM

Last Reviewed:

01/Jan/10