Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-20 Thread Jakob Korherr
Sounds great. But with regard to your third point, what about keeping it more general? Maybe we later want to extend it to something we don't want to check in production that has nothing to do with IDs whatsoever? Maybe org.apache.myfaces.DISABLE_PRODUCTION_CHECKS or

Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-20 Thread Matt Benson
On Fri, May 20, 2011 at 2:40 AM, Jakob Korherr jakob.korh...@gmail.com wrote: Sounds great. But with regard to your third point, what about keeping it more general? Maybe we later want to extend it to something we don't want to check in production that has nothing to do with IDs whatsoever?

Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-20 Thread Mike Kienenberger
The point of the options is to customize your behavior from the standard expected configuration. I think you are better off keeping things as specific as possible rather than generalizing. If someone has a special reason why they need to turn on duplicate id warnings (like in this case), that

Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-19 Thread Martin Koci
org.apache.myfaces.CHECK_ID_IN_PRODUCTION (default true) and when false: 1) skip duplicate id check 2) skip id validity check (in UIComponent.setId) 3) ... (something we found later) ... WDYT? Jakob Korherr píše v So 14. 05. 2011 v 12:26 +0200: +1 for a MyFaces specific parameter.

Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-14 Thread Jakob Korherr
+1 for a MyFaces specific parameter. Regards, Jakob 2011/5/11 Martin Koci martin.kocicak.k...@gmail.com: +1 for specific parameter (in one project I build view dynamically from DB and want this ids check) Gerhard Petracek píše v St 11. 05. 2011 v 07:52 +0200: hi, i would combine it - +1

Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-11 Thread Martin Koci
+1 for specific parameter (in one project I build view dynamically from DB and want this ids check) Gerhard Petracek píše v St 11. 05. 2011 v 07:52 +0200: hi, i would combine it - +1 for a myfaces specific parameter which gets evaluated in case of project-stage production. regards,

[core] do not check for duplicate ids when saving view on production stage

2011-05-10 Thread Leonardo Uribe
Hi Checking the state saving algorithm I have seen that every time StateManager.saveView is called, it checks for duplicate ids, scanning the whole component tree. The documentation of StateManager.saveView says this: ...This method must also enforce the rule that, for components with non-null

Re: [core] do not check for duplicate ids when saving view on production stage

2011-05-10 Thread Gerhard Petracek
hi, i would combine it - +1 for a myfaces specific parameter which gets evaluated in case of project-stage production. regards, gerhard http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces 2011/5/11