String#substr() and String#substring() are the two lesser known legacy ways to slice a string. It's better to use String#slice() as it's a more popular option with clearer behavior that has a consistent Array counterpart.
This rule is fixible when no arguments are passed to the method.
foo.substr(1, 2);
foo.substring(1, 3);foo.slice(1, 3);