Issue 4098 in pharo: Fileout - "choose another name" cancel crashs

Status: New
Owner: ----

New issue 4098 by [hidden email]: Fileout - "choose another name"  
cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098

Pharo image: Pharo-core
Pharo core version: Pharo1.3a
Latest update: #13172
Virtual machine used: <Pharo4.1.1 on Win32>
Class browser used (if applicable): "SystemBrowser default"

This bug apparently was in 1.2 also.

To reproduce.
1. Open a browser, select a package, right click, select FileOut.
2. Do it again, so the system announces the file already exists
3. When the 1st popup opens, select 'choose another file name'
4. When the 2nd popup opens, click on cancel.

"receiver of asVmPathName is nil"




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

Updates:
        Status: Started

Comment #1 on issue 4098 by marianopeck: Fileout - "choose another name"  
cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098

Thanks.I could reproduce it. Of course, adding a " fileName ifNil: [ ^ nil  
]."  at the beginning fix it...but I don't know the real/clean solution. Do  
you?


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

Comment #2 on issue 4098 by [hidden email]: Fileout - "choose  
another name" cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098

The problem is in StandardFileStream>>fileExistsUserHandling:

For choice 2 the user is asked a second time with a dialog in the line
   newName := UIManager default request: 'Enter a new file name'  
initialAnswer: fullFileName.

Here temp "newName" could be nil if the user clicks cancel and an Abort  
exception should be signaled similar to choice 3 from the first menu.

The attached file fixes the issue.


Attachments:
        StandardFileStream class-fileExistsUserHandling.st  1004 bytes


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

Updates:
        Status: FixProposed

Comment #3 on issue 4098 by [hidden email]: Fileout - "choose  
another name" cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098

SLICE-Issue-4098-Fileout-choose-another-name-cancel-crash-tbn.1 in inbox


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

Updates:
        Labels: Milestone-1.3

Comment #4 on issue 4098 by [hidden email]: Fileout - "choose  
another name" cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098

Checked the slice again in an updated Pharo1.3a#13176 and it works without  
a problem....


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

Updates:
        Status: Closed

Comment #5 on issue 4098 by [hidden email]: Fileout - "choose  
another name" cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098

in 13177


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

Updates:
        Labels: MigratedToFogBugz

Comment #6 on issue 4098 by [hidden email]: Fileout - "choose  
another name" cancel crashs
http://code.google.com/p/pharo/issues/detail?id=4098#c6

Issue migrated to https://pharo.fogbugz.com/f/cases/4144

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

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