I spent a couple hours hunting for a bug in which fossil does not properly
apply changes to my custom fields to the ticket table.  The root cause
appears to be these lines from cgi.c's add_param_list():

====
    if( fossil_islower(zName[0]) ){
      cgi_set_parameter_nocopy(zName, zValue);
    }
====

... i.e. silently ignore capitalized POST argument names.  Is there a
reason for that?

It appears to have been like that since the very first check-in of the
file.  I can't possibly be the only person in six years to have been hit by
this.

Fossil should have rejected my attempt to create capitalized ticket table
column names if it was planning on ignoring edits to those fields later.

And it should warn me on the ticket table edit page that capitalized names
do not work.

Or, better, it should just allow capitalized ticket table column names.

Also, I managed to hose my repo by changing my ticket column names from
upper case to lower case, hoping that fossil would have some heuristics to
carry my data over, or else would warn me that it was going to blow away my
values.  I hoped in vain.  There should be a warning about that, too.
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to