[ http://issues.apache.org/jira/browse/OFBIZ-413?page=all ]

Si Chen updated OFBIZ-413:
--------------------------

    Attachment: seqid.patch

Here is the file of the change, and it is very simple:
Index: framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java
===================================================================
--- framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java        
(revision 710)
+++ framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java        
(working copy)
@@ -203,7 +203,7 @@
                             rs = stmt.executeQuery(sql);
                             boolean gotVal1 = false;
                             if (rs.next()) {
-                                val1 = rs.getInt(parentUtil.idColName);
+                                val1 = rs.getLong(parentUtil.idColName);
                                 gotVal1 = true;
                             }
                             rs.close();
@@ -226,7 +226,7 @@
                             rs = stmt.executeQuery(sql);
                             boolean gotVal2 = false;
                             if (rs.next()) {
-                                val2 = rs.getInt(parentUtil.idColName);
+                                val2 = rs.getLong(parentUtil.idColName);
                                 gotVal2 = true;
                             }
 


> Sequenced ID value limited to Integer not Long
> ----------------------------------------------
>
>                 Key: OFBIZ-413
>                 URL: http://issues.apache.org/jira/browse/OFBIZ-413
>             Project: OFBiz (The Open for Business Project)
>          Issue Type: Bug
>          Components: framework
>            Reporter: Si Chen
>            Priority: Minor
>         Attachments: seqid.patch
>
>
> I just  found a small problem with SequenceUtil - it calls the getInt instead 
> of the getLong method on the ResultSet query, so large sequence values 
> (exceeding the Java integer limit of 2147483647 will not work.
> I made this change to SequenceUtil and now it works fine.  Just wanted to 
> check to make sure nobody has any objections before committing this...

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to