jbaron / cats

Code Assistant for TypeScript
Apache License 2.0
408 stars 66 forks source link

Can't create/open a file/folder #120

Closed MangelMaxime closed 9 years ago

MangelMaxime commented 9 years ago

Hi guys, when I try to create or open a file or folder i get the following error on OS X and windows. With the stable and the unstable release of CATS.

6:15:33 PM TypeError: Cannot read property 'on' of undefined
6:15:33 PM at new EditSession (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/lib/main.js:3731:28)
6:15:33 PM at new SourceEditor (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/lib/main.js:2844:36)
6:15:33 PM at Object.newFile (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/lib/main.js:1999:41)
6:15:33 PM at null.<anonymous> (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/lib/main.js:4560:25)
6:15:33 PM at qx.Class.define.members._dispatchEvent (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/resource/qooxdoo.js:210:456968)
6:15:33 PM at cg.qx.Class.define.members.dispatchEvent (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/resource/qooxdoo.js:210:336489)
6:15:33 PM at qx.Class.define.members.dispatchEvent (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/resource/qooxdoo.js:210:133313)
6:15:33 PM at Object.qx.Class.define.statics.fireEvent (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/resource/qooxdoo.js:210:136975)
6:15:33 PM at qx.Class.define.members.__gw (file:///var/folders/c7/2fkczxvn1l95q9gvx8382ct00000gn/T/.org.chromium.Chromium.J0QJPp/resource/qooxdoo.js:210:378551)
6:15:33 PM at null.<anonymous> 
MangelMaxime commented 9 years ago

In fact, it's seems like I needed to open a project before. Sorry about that.

jbaron commented 9 years ago

Stil, perhaps a good idea to have a simple message about opening a project first instead of trowing an exception.

MangelMaxime commented 9 years ago

Indeed, it's a good idea I think because I was thinking of a bug since 3 days :)

jbaron commented 9 years ago

Added some more sanity checks at unstable version (turned out many errors going on when you don't have a project open)