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). > >
