Re: [Wonder-disc] ERD2WPropertyName.displayRequiredMarker

[Anjo K. <kr...@lo...>]

Hi Dennis,

I'll choose door #2, the washing machine:

> (task = 'edit' and isMandatory = '0') => displayRequiredMarker = true

Also, I changed it to use display on !hidePropertyName, which can now  
also be configured by rules, as the former displayNameForProperty  
would also be used by error messages, so you couldn't really set to "".

Thanks for noticing it. All you others out there: you may need check  
your apps!

Cheers, Anjo

Am 14.05.2007 um 18:08 schrieb Denis Frolov:

> Hi,
>
> ERD2WPropertyName.displayRequiredMarker seems to be duplicating  
> logic in rules now. This prevents overriding of "isMandatory" via  
> rules. Such overriding can be useful in cases like:
>
> (task = 'edit' and componentName = 'D2WEditBoolean') => isMandatory  
> = false, since it doesn't make sense to show a required mark for  
> checkbox.
>
> Attached is a patch that removes duplication and allows overriding  
> of isMandatory. Alternatively it probably makes sense to move  
> "displayRequiredMarker" to rules and define a rule like:
>
> (task = 'edit' and isMandatory = '0') => displayRequiredMarker = true
>
> Thanks in advance for reviewing this suggestion and patch.
>
> --
> Denis Frolov
> Design Maximum MA
>
> Tel: +7 863 2648211
> Fax: +7 863 2645229
> Web: http://www.designmaximum.com
>
> <displayRequiredMarker.txt>
>
>