Issue 7275 in pharo: Sanitize String with/without selectors

Status: Accepted
Owner: ----
Labels: Milestone-2.0 Type-Feature

New issue 7275 by [hidden email]: Sanitize String with/without  
selectors
http://code.google.com/p/pharo/issues/detail?id=7275

Some String with/without selectors have bugs in them

http://code.google.com/p/pharo/issues/detail?id=7272

#withoutWrappingDoubleQuotes is faulty

Sanitize and fix, add some more unit tests


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 7275 by [hidden email]: Sanitize String  
with/without selectors
http://code.google.com/p/pharo/issues/detail?id=7275

In PharoInbox:

Name:  
SLICE-Issue-7275-Sanitize-String-withwithout-selectors-SvenVanCaekenberghe.1
Author: SvenVanCaekenberghe
Time: 17 January 2013, 11:07:14.877 am
UUID: fabfce02-7bf4-4eb5-a42c-c5512ddbc2d7
Ancestors:
Dependencies: Graphics-Fonts-SvenVanCaekenberghe.66,  
CollectionsTests-SvenVanCaekenberghe.582,  
Collections-Strings-SvenVanCaekenberghe.237

Removed from String
   #unparentically (what the hell ?)
   #withoutWrappingDoubleQuotes (not used, wrong, no users, we have  
#withoutQuoting)
   #withBlanksTrimmed (deprecated in 1.4)
   #withoutLeadingBlanks (deprecated in 1.4)
   #withoutTrailingBlanks (deprecated in 1.4)
   #withoutJustTrailingDigits (wrong, unlined in its only user,  
StringFont>>#familyName)

Fixed reported bug in String>>#withoutTrailingDigits (thx Davide Varvello)

Improved comments and code formatting

Added to StringTest
   #testWithBlanksCondensed
   #testWithSeparatorsCompacted
   #testWithoutLeadingDigits
   #testWithoutTrailingDigits
   #testWithoutPeriodSuffix


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #2 on issue 7275 by [hidden email]: Sanitize String  
with/without selectors
http://code.google.com/p/pharo/issues/detail?id=7275

looks good!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #3 on issue 7275 by [hidden email]: Sanitize String  
with/without selectors
http://code.google.com/p/pharo/issues/detail?id=7275

in 2.0 481


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker