Channels
#tornadofx
- p
permalmberg
01/21/2017, 12:45 PMit says it can't access because it is private - e
edvin
01/21/2017, 12:45 PMI don't see any problems? - p
permalmberg
01/21/2017, 12:46 PMreally? - e
edvin
01/21/2017, 12:47 PMWhat exactly is the error message you receive? - p
permalmberg
01/21/2017, 12:47 PMError:(89, 92) Kotlin: Cannot access 'vm': it is private in 'OutputTextView' - p
permalmberg
01/21/2017, 12:47 PMwithprivate val vm: OutputVM by param() - e
edvin
01/21/2017, 12:47 PMis that at the call site where you configure the parameter? - p
permalmberg
01/21/2017, 12:47 PMat the callsite - p
permalmberg
01/21/2017, 12:48 PMi.e.this += find<OutputTextView>(mapOf(OutputTextView::vm to outVM)) - e
edvin
01/21/2017, 12:48 PMI see... Nothing we can do about that I'm afraid. - p
permalmberg
01/21/2017, 12:49 PMNo, didn't expect there to be. I'm a stickler to keeping things contained inside my classes...I suppose no one elese has declared their params as private... - e
edvin
01/21/2017, 12:50 PMYou might be able to use internal if stuff is in the same package. Not the same though. Irritating indeed. - p
permalmberg
01/21/2017, 12:52 PMYeah...only other option is to create the view at the callsite using constructor parameters, but then we get problem of the init-stuff not being called - e
edvin
01/21/2017, 12:53 PMThat could be fixed actually.. Whenever a view is added or accessed by the framework, we could make sure that init() is run. - e
edvin
01/21/2017, 12:53 PMDon't know why I didn't think of that the last time. - p
permalmberg
01/21/2017, 12:54 PMDunno 🙂 - p
permalmberg
01/21/2017, 12:54 PMI'd actually prefer that instead of the find()...much cleaner - e
edvin
01/21/2017, 12:54 PMLet me have a look. - e
edvin
01/21/2017, 12:56 PM@permalmberg I committed a fix that will (probably) allow you to instantiate views manually. Check it out 🙂 - p
permalmberg
01/21/2017, 12:57 PMOk, have to update manually then (I really need to create a real maven project instead of this intellij-only version) - e
edvin
01/21/2017, 12:58 PMYes you do 🙂 - p
permalmberg
01/21/2017, 1:00 PMIs there such a thing as intellij to maven converter? 😄 - e
edvin
01/21/2017, 1:02 PMIt's very easy to do at least, but you'll have to add your dependencies manually. - p
permalmberg
01/21/2017, 1:03 PMyeah, don't have that many deps yet, luckily - e
edvin
01/21/2017, 1:04 PM@permalmberg Right click your project and choose Add Framework, then Maven, and you're set 🙂 - p
permalmberg
01/21/2017, 1:06 PM@edvin really? I thought I'd have to remake them all - e
edvin
01/21/2017, 1:06 PMNah 🙂 - p
permalmberg
01/21/2017, 1:07 PMso you mean it'll build based on the pom file now instead of the intellij project settings? - e
edvin
01/21/2017, 1:08 PMYes - p
permalmberg
01/21/2017, 1:09 PMyou sure about that? My pom is empty, except the default data and it still builds...