|
From: <caw...@us...> - 2007-05-29 13:11:52
|
Revision: 2541
http://svn.sourceforge.net/rubyeclipse/?rev=2541&view=rev
Author: cawilliams
Date: 2007-05-29 06:11:51 -0700 (Tue, 29 May 2007)
Log Message:
-----------
add ability to validate constants
Modified Paths:
--------------
trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/core/RubyConventions.java
Modified: trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/core/RubyConventions.java
===================================================================
--- trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/core/RubyConventions.java 2007-05-29 13:11:31 UTC (rev 2540)
+++ trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/core/RubyConventions.java 2007-05-29 13:11:51 UTC (rev 2541)
@@ -85,10 +85,24 @@
if (!isConstant(typeName)) {
return new Status(IStatus.ERROR, RubyCore.PLUGIN_ID, -1, "Class name must be a constant. It must begin with a capital letter, and contain only letters, digits, or underscores.", null);
}
- return new Status(IStatus.OK, RubyCore.PLUGIN_ID, -1, null, null);
+ return RubyModelStatus.VERIFIED_OK;
}
+ public static IStatus validateConstant(String constantName) {
+ if (constantName == null) {
+ return new Status(IStatus.ERROR, RubyCore.PLUGIN_ID, -1, Messages.convention_type_nullName, null);
+ }
+ if (constantName.length() == 0) {
+ return new Status(IStatus.ERROR, RubyCore.PLUGIN_ID, -1, Messages.bind(Messages.convention_type_invalidName, constantName), null);
+ }
+ if (!isConstant(constantName)) {
+ return new Status(IStatus.ERROR, RubyCore.PLUGIN_ID, -1, "Name must be a constant. It must begin with a capital letter, and contain only letters, digits, or underscores.", null);
+ }
+ return RubyModelStatus.VERIFIED_OK;
+
+ }
+
private static boolean isConstant(String className) {
if (className == null || className.length() == 0) return false;
if (!Character.isLowerCase(className.charAt(0)) && !Character.isLetter(className.charAt(0)))
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
|