On 25.01.2013 17:30, Francesco Chicchiriccò wrote: > On 25/01/2013 17:28, Christian Schneider wrote: >> On 25.01.2013 17:18, Francesco Chicchiriccò wrote: >>> I also thought about this and if I should rather wait. My change only >>>> affects the new interface. So the old controller stays the same and >>>> stays active. The change should not affect anyone who uses the current >>>> spring based rest interface. >>>> As we already use the new interface in the console and the tests these >>>> are of course affected but this should be only internal. >>>> >>>> If you prefer I can roll back and wait for the result of the >>>> discussion. >>> For the moment it would be enough for me to have again a working admin >>> console, as it used to be before of this commit. >>> >>> Regards. >>> >> I will take care of this. I hoped to not affect the console. If I cant >> fix it easily I will roll back. > > I am about to commit a temporary workaround: take your time and fix it > properly, please. > I restored the old behaviour of isCreateAllowed which should fix the bug. The bug and the ugly code in Login show that using a header was probably not such a good idea after all. Sorry for breaking the console. I did not test the UI before the commit.
Christian -- Christian Schneider http://www.liquid-reality.de Open Source Architect http://www.talend.com
