Let's leave this commented code it, we actually should be doing it but it's wrong this way ... we need to check that the charlength doe not equal 0 and there was a recent tracker issue where we fixed this in JCategories though I don't think it has been merged yet.  So, let's leave it and put on the list of things to fix .