Ticket #525 (closed defect: fixed)

Opened 3 years ago

Last modified 18 months ago

dojo.string.endsWith not working

Reported by: guioum Owned by: anonymous
Priority: high Milestone:
Component: Core Version: 0.2
Severity: major Keywords:
Cc:

Description

In "dojo.string.extras"

Test case:

dojo.string.endsWith("mode", "value", false) // returns true, must be false
// "mode".length-"value".length = -1
// "mode".lastIndexOf("value") = -1
// -1 = -1

Is:

    return str.lastIndexOf(end) == str.length - end.length;

Should be:

    var startPos = str.length - end.length;
    if (startPos < 0) {
      return false;
    }
    return (str.lastIndexOf(end, startPos) == startPos);

Thanks. Guioum.

Change History

Changed 3 years ago by alex

  • status changed from new to closed
  • resolution set to fixed
  • milestone set to 0.3release

fixed in [3353]

Changed 18 months ago by anonymous

  • milestone deleted

Milestone 0.3release deleted

Note: See TracTickets for help on using tickets.