Status: New
Owner: ----

New issue 3446 by elpochodelagente: VMMakerTool load config button doesn't work
http://code.google.com/p/pharo/issues/detail?id=3446

Pharo image: dev 1.1.1
Pharo core version: Pharo-1.1-11411
Virtual machine used: custom linux, allmost latest VMMaker, classic (not cog)
Class browser used if applicable:  OBSystemBrowserAdaptor

Steps to reproduce:
1. Load VMMaker with
2. Open the VMMakerTool with: "VMMakerTool openInWorld"
3. Push load config button, you get an error message

The problem is that VMMaker tool makes use of StandardFileDialog, which isn't present in Pharo anymore. The method that raises the error is

VMMakerTool>>#loadConfig
    | fileResult file |
fileResult := (StandardFileMenu oldFileMenu: FileDirectory default withPattern: '*.config') startUpWithCaption: 'Select VMMaker configuration...'.
    fileResult
        ifNotNil: [
            file := fileResult directory fullNameFor: fileResult name.
            [
                vmMaker := VMMaker forConfigurationFile: file.
                vmMaker logger: logger.
                vmMaker platformDirectory
            ]
            on: Error
do: [self inform: 'Possible problem with path settings or platform name?'].
            self updateAllViews]


Changing

fileResult := (StandardFileMenu oldFileMenu: FileDirectory default withPattern: '*.config') startUpWithCaption: 'Select VMMaker configuration...'.

for

fileResult := UIManager default chooseFileMatching: nil label: 'Select VMMaker configuration...'.

and

file := fileResult directory fullNameFor: fileResult name.

for

file := fileResult

Fixes it almost completely. It would be better to use chooseFileMatching: #('*.config') but it didn't work well.



Reply via email to