[GERONIMO-715] 'StringIndexOutOfBoundsException' in 'org.openejb.corba.compiler.PortableStubCompiler'

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0-M4
• Fix Version/s: 1.0-M4
• Component/s: OpenEJB

Environment

up to date Geronimo snapshot (03.07.2005)

• Assignee: Aaron Mulder
• Reporter: M.Oliver Scheele
• Created: 07/Jul/05 03:55
• Updated: 07/Aug/06 19:05
• Resolved: 13/Jul/05 15:36

Description

Just played around the first time with Geronimo and the OpenEJB service.
First of all: Thanks for this great project and the hard work!

During deployment of a simple CMP EJB project I got the following exception:
15:58:00,318 DEBUG [Deployer] Deployment failed: plan=null, module=D:\projects\geronimo\deploy_tests\target\geronimo\myproject.ear
..........
Caused by: java.lang.StringIndexOutOfBoundsException: String index out of range: 5
at java.lang.String.charAt(String.java:444)
at org.openejb.corba.compiler.PortableStubCompiler.createIiopOperations(PortableStubCompiler.java:198)
........

During the last patch there seems to be introduced a small bug.
Lock at line 198 in class 'org.openejb.corba.compiler.PortableStubCompiler' inside the 'openejb-core' project:

>> if (methodName.length() > 4 && Character.isUpperCase(methodName.charAt(4 + 1))) {

That's not very friendly when using 5-character strings.
It should be something like this:
>> if (methodName.length() > 5 && Character.isUpperCase(methodName.charAt(4 + 1))) {