Why have this complicated structure here?  The intent is simply to show the warning dialog, right?  If so, perhaps just:
```js
this.showWarningDialog();
return false;
```