Author: [email protected]
Date: Mon Dec 15 11:28:25 2008
New Revision: 2878
Modified:
trunk/src/ca/sqlpower/architect/SQLIndex.java
Log:
Fixed a findBugs warning. the static string types should be declared final
if they are going to be used as constants. This will avoid any
vulnerability for people trying to modify these strings.
Modified: trunk/src/ca/sqlpower/architect/SQLIndex.java
==============================================================================
--- trunk/src/ca/sqlpower/architect/SQLIndex.java (original)
+++ trunk/src/ca/sqlpower/architect/SQLIndex.java Mon Dec 15 11:28:25 2008
@@ -53,7 +53,7 @@
* This is the property name in the PL.ini file that will indicate
what Index types
* are supported for any specific database.
*/
- public static String INDEX_TYPE_DESCRIPTOR = SQLIndex.class.getName()
+ ".IndexType";
+ public static final String INDEX_TYPE_DESCRIPTOR =
SQLIndex.class.getName() + ".IndexType";
/**
* This is the index type
@@ -64,7 +64,7 @@
* This is the name of the column that will be augmented by the custom
* JDBC wrappers to represent index type;
*/
- public static String RS_INDEX_TYPE_COL = "SPG_INDEX_TYPE";
+ public static final String RS_INDEX_TYPE_COL = "SPG_INDEX_TYPE";
/**
* A simple placeholder for a column. We're not using real SQLColumn
instances here so that the
