Renaming @ViewRef config property?
Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
+1 -- Bruno Leonardo Gonçalves 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
-0.5 for now once we add more, you get the same and it would be not that expressive. that was the reason for changing it (compared to codi). regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
+1, same for @ConfigProperty btw Le 3 oct. 2013 20:49, Gerhard Petracek gerhard.petra...@gmail.com a écrit : -0.5 for now once we add more, you get the same and it would be not that expressive. that was the reason for changing it (compared to codi). regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
@Gerhard: Are there any expected properties on @ViewRef in the future? 2013/10/3 Romain Manni-Bucau rmannibu...@gmail.com +1, same for @ConfigProperty btw Le 3 oct. 2013 20:49, Gerhard Petracek gerhard.petra...@gmail.com a écrit : -0.5 for now once we add more, you get the same and it would be not that expressive. that was the reason for changing it (compared to codi). regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
hi thomas, yes - we had something in codi and we might add something like the payload in bv. regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com @Gerhard: Are there any expected properties on @ViewRef in the future? 2013/10/3 Romain Manni-Bucau rmannibu...@gmail.com +1, same for @ConfigProperty btw Le 3 oct. 2013 20:49, Gerhard Petracek gerhard.petra...@gmail.com a écrit : -0.5 for now once we add more, you get the same and it would be not that expressive. that was the reason for changing it (compared to codi). regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
If config is the unique mandatory attr it should be value imo Le 3 oct. 2013 22:39, Gerhard Petracek gerhard.petra...@gmail.com a écrit : hi thomas, yes - we had something in codi and we might add something like the payload in bv. regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com @Gerhard: Are there any expected properties on @ViewRef in the future? 2013/10/3 Romain Manni-Bucau rmannibu...@gmail.com +1, same for @ConfigProperty btw Le 3 oct. 2013 20:49, Gerhard Petracek gerhard.petra...@gmail.com a écrit : -0.5 for now once we add more, you get the same and it would be not that expressive. that was the reason for changing it (compared to codi). regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas
Re: Renaming @ViewRef config property?
I didnt use viewref enough to say but typically it is boring to need to write name in configproperty each time while you know what it is. Moreover for viewref, config doesnt sound really right, metadata or marker sounds as right as config depending where you are coming from. Well this doesnt hold features so i dont want to loose time in it but now you know why i like value ;) Le 3 oct. 2013 23:21, Gerhard Petracek gerhard.petra...@gmail.com a écrit : -0.5 (instead of -1), because i used value in codi back then and there is nothing wrong with it. however, that was one of the lessons learned from using it in projects and explaining it in trainings for almost three years. what we have right now just reflects the feedback. regards, gerhard 2013/10/3 Romain Manni-Bucau rmannibu...@gmail.com If config is the unique mandatory attr it should be value imo Le 3 oct. 2013 22:39, Gerhard Petracek gerhard.petra...@gmail.com a écrit : hi thomas, yes - we had something in codi and we might add something like the payload in bv. regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com @Gerhard: Are there any expected properties on @ViewRef in the future? 2013/10/3 Romain Manni-Bucau rmannibu...@gmail.com +1, same for @ConfigProperty btw Le 3 oct. 2013 20:49, Gerhard Petracek gerhard.petra...@gmail.com a écrit : -0.5 for now once we add more, you get the same and it would be not that expressive. that was the reason for changing it (compared to codi). regards, gerhard 2013/10/3 Thomas Andraschko andraschko.tho...@gmail.com Hi, currently @ViewRef has only one property called config. So the current usage is: @ViewRef(config = Views.Logout.class) What about renaming it to value? - @ViewRef(Views.Logout.class) Regards, Thomas