saguziel / Kryptose

A password manager
2 stars 0 forks source link

For JONATHAN TO FIX #31

Closed AMarcedone closed 9 years ago

AMarcedone commented 9 years ago

Hi, from Controller.java

//TODO Jonathan //model.setLastException(new RecoverableException("Please select a Credential before editing"));

I was displaying an error when the user tried to hit edit before selecting a credential. But now setLastException is not available and I do not know how to do it. Not the end of the world though.

AMarcedone commented 9 years ago

This problem both appears in addCredentialAction and editCredentialAction.

private Action editCredentialAction = new AbstractAction("Done") {
    @Override
    public void actionPerformed(ActionEvent ev) {

        Exception e = null; 

            try {
                e = control.set(setType.EDIT).get();
            } catch (InterruptedException | ExecutionException e1) {
                //TODO Jonathan: what do I do here?
            }

        if(e == null ){//No exception, edit succeeded
            control.requestViewState(ViewState.MANAGING);
        }else {
            //TODO Jonathan: we should probably display an alert-box about the exception here.
        }           
    }
};