This project is archived and is in readonly mode.

#170 ✓invalid
ronin-24111 (at lighthouseapp)

String.contains seperator

Reported by ronin-24111 (at lighthouseapp) | June 26th, 2008 @ 09:01 PM | in 1.2.1

'aba'.contains('b', 'aa');

// -> returns true

(selector + this + selector) is tested, you should check if the first or last characters equal the seperator.

Comments and changes to this ticket

  • Jan Kassens

    Jan Kassens June 26th, 2008 @ 10:49 PM

    • State changed from “new” to “hold”

    i think that problem only exists with multi character seperators and should not be fixed, as its used in speed critical situations and a note in the docs should suffice.

    do you have a real world example?

  • ronin-24111 (at lighthouseapp)

    ronin-24111 (at lighthouseapp) June 27th, 2008 @ 12:42 PM

    Yes, in the real world you don't really need seperators like this!

    But there ist no note in the docs describing this exception, so you could see this as a bug ;)

  • Jan Kassens

    Jan Kassens June 27th, 2008 @ 12:45 PM

    • State changed from “hold” to “open”
    • Tag set to docs
    • Milestone changed from 2.0 to 1.2.1
  • Jan Kassens

    Jan Kassens September 12th, 2008 @ 04:59 PM

    • State changed from “open” to “invalid”
    • Assigned user cleared.

    I don't think its even worth adding it to the docs, as it only adds some noise to them.

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

People watching this ticket