[CLOSED] StringIndexOutOfBoundsException in StandardConfigurationLoader

[tduchateau]

Issue by eruiz from Wednesday Jun 26, 2013 at 14:44 GMT

---

In resolveGroups method at StandardConfigurationLoader the instruction below throws a StringIndexOutOfBoundsException (line ~221):

else {
   ...
                    logger.warn("The property '{}' (inside the '{}' group) doesn't exist",
                            key.substring(groupName.length() + 1), groupName);
}

I think key.substring(groupName.length() + 1) should be replaced by key.substring(key.indexOf(".") + 1)

To reproduce use blank "groupName" and export options "csv,xls,pdf"

[tduchateau]

Comment by tduchateau from Wednesday Jun 26, 2013 at 14:54 GMT

Even if you don't explicitely enable any configuration group in your JSP, the property in datatables.properties should be defined as follows:

global.export.types=csv,xls,pdf

Did you use the belowing format?

Le 26 juin 2013 à 16:44, Enrique Ruiz notifications@github.com a écrit :

In resolveGroups method at StandardConfigurationLoader the instruction below throws a StringIndexOutOfBoundsException (line ~221):

else {
...
logger.warn("The property '{}' (inside the '{}' group) doesn't exist",
key.substring(groupName.length() + 1), groupName);
}
I think key.substring(groupName.length() + 1) should be replaced by key.substring(key.indexOf(".") + 1)

To reproduce use blank "groupName" and export options "csv,xls,pdf"

—
Reply to this email directly or view it on GitHub.

[tduchateau]

Comment by tduchateau from Wednesday Jun 26, 2013 at 15:38 GMT

Damned, you were right.
In one of the recent commits, I made some changes in order to remove the group information from the "groupedProperties" but I forgot to reflect the changes in the logger (else statement).

Thanks Enrique. :-)

[tduchateau]

Comment by tduchateau from Wednesday Jun 26, 2013 at 15:57 GMT

By the way, the 'global' group must indeed be always added.