Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

pharo
Status: Accepted
Owner: [hidden email]
Labels: Type-Bug

New issue 5885 by [hidden email]: FileBrowser/FileList cannot reset  
the search pattern
http://code.google.com/p/pharo/issues/detail?id=5885

When you accept an empty pattern (which means no filter), then the file  
list is empty..

Here is what's going on:

FileList>>pattern: aString
        pattern := aString  asString trimBoth.
        pattern := pattern isEmptyOrNil
                ifTrue: [ nil ]
                ifFalse: [ pattern asRegex ].
        self updateFileList.

Unfortunately, this message receive nil when we accept an empty pattern  
text field...

nil asString -> 'nil', we then search files matching 'nil'



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

pharo
Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5885 by [hidden email]: FileBrowser/FileList  
cannot reset the search pattern
http://code.google.com/p/pharo/issues/detail?id=5885

Here is a fix attached

Attachments:
        FileList-pattern.st  345 bytes


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

pharo
Updates:
        Labels: Milestone-2.0

Comment #2 on issue 5885 by [hidden email]: FileBrowser/FileList  
cannot reset the search pattern
http://code.google.com/p/pharo/issues/detail?id=5885

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

pharo
Updates:
        Status: FixToInclude

Comment #3 on issue 5885 by [hidden email]: FileBrowser/FileList  
cannot reset the search pattern
http://code.google.com/p/pharo/issues/detail?id=5885

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

pharo

Comment #4 on issue 5885 by [hidden email]: FileBrowser/FileList  
cannot reset the search pattern
http://code.google.com/p/pharo/issues/detail?id=5885

in 20081


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5885 in pharo: FileBrowser/FileList cannot reset the search pattern

pharo
Updates:
        Status: Integrated

Comment #5 on issue 5885 by [hidden email]: FileBrowser/FileList  
cannot reset the search pattern
http://code.google.com/p/pharo/issues/detail?id=5885

(No comment was entered for this change.)


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