[Pagekit-devel] Error message from missing fields

[Bill K. <bi...@ka...>]

Hi,

May I suggest a change to the reporting of missing mandatory fields?  The 
error report for missing fields is hard-coded to describe the mandatory 
fields as "red".  This limits the HTML designer to match color schemes, and 
also confuses users if I have set errorspan_begin_tag to something other 
than the color red.  I want to establish a different visual cue than simply 
color.

I have changed Model.pm line 171:

     $model->pkit_gettext_message(qq{You did not fill out all the required 
fields. Please fill the <font color="<PKIT_ERRORSTR>">red</font> fields.});

To this code (based on pkit_set_errorfont()):

     my $color_str = $model->pkit_get_config_attr( GLOBAL => 
'default_errorstr' ) || "#ff0000";
     my $errorspan_begin_tag = $model->pkit_get_config_attr( GLOBAL => 
'errorspan_begin_tag' ) || qq{<font color="$color_str">};
     my $errorspan_end_tag = $model->pkit_get_config_attr( GLOBAL => 
'errorspan_end_tag' ) || q{</font>};
     $model->pkit_gettext_message('You did not fill out all the ' . 
$errorspan_begin_tag . 'mandatory' . $errorspan_end_tag . ' fields.');

Now I can use the Config.xml GLOBAL attributes default_errorstr, 
errorspan_begin_tag, and errorspan_end_tag to set the style of the error 
message, without the message describing the style.

Regards,
Bill Karwin