I dont think `file` is a great parameter. It doesn't make sense to me that a shortcut manager can backup any file on the users system. If the shortcut manager is going to have back something up, it should be the shortcuts file and the shortcuts file only. I would prefer file was removed as a parameter and these two lines were changed to

```
if not self.shortcuts_file:
    return
````