ok. Regards Felix
Am Mittwoch, den 12.09.2007, 08:21 +0200 schrieb Christophe Lombart: > You can apply the patch and after I will start the refactoting. > > Thanks, > Christophe > > > On 9/12/07, Felix Meschberger <[EMAIL PROTECTED]> wrote: > > > > Hi, > > > > Alright then, I will create the JIRA and try to setup a patch. > > > > Christophe, as you are currently refactoring the code, shall I just > > apply the patch or do you want to integrate this in your refactoring > > work ? > > > > Regards > > Felix > > > > > > Am Dienstag, den 11.09.2007, 22:32 +0200 schrieb Christophe Lombart: > > > On 9/11/07, Felix Meschberger <[EMAIL PROTECTED]> wrote: > > > > > > > > Hi all, > > > > > > > > The SimpleFieldsHelper.retrieveSimpleField method is used to load JCR > > > > properties into "simple" fields. If a node does not have a mapped > > > > property, a warning message is emited. I am not sure, whether it is > > > > actually a problematic situation if a property, which is not mandatory > > > > does not exist ? In fact, if a property is mandatory, it may or may > > not > > > > exist and thus the mapping may be supplied or not. > > > > > > > > What do you think of lowering the severity of this situation to debug > > ? > > > > Alternatively (or in addition) the descriptor might be extended such > > > > that the mapping may declared as mandatory without rquiring the > > property > > > > to be mandatory or there might even be some default value specified or > > > > the SimpleFieldsHelper might access the property definition to see > > > > whether there is a default value defined .... > > > > > > > > > > > > +2 (+1 for the debug, +1 for the changing a little bit the code and/or > > > the descriptor). > > > >
