lmi
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi] Unable to rename employee class--fixed


From: Greg Chicares
Subject: [lmi] Unable to rename employee class--fixed
Date: Fri, 31 Oct 2014 01:42:22 +0000
User-agent: Mozilla/5.0 (Windows NT 5.1; rv:24.0) Gecko/20100101 Thunderbird/24.6.0

If you edit class defaults in a census, and change the value of
"Employee class" (which determines which class's defaults you're
editing), then nothing happens--and that's wrong. To reproduce:

alt-F N C
ctrl-alt-E
  type "x" in "Employee class"
OK
Yes
alt-F S [save as "eraseme.cns"]
alt-F X

grep EmployeeClass eraseme.cns

I'd expect every line to match
  <EmployeeClass>x</EmployeeClass>
except the first, which represents case default parameters.
Instead, none did.

Fixed in 'census_view.cpp' 20141031T0141Z, revision 6010.

The comment correctly says to apply changes...
///   if 'for_this_class_only' is specified, to all cells in the
///     employee class of the old parameters;
                              ^^^

but the code applied changes to all cells in the employee class
of the *new* parameters:
    if((*j)["EmployeeClass"] == new_parms["EmployeeClass"])
                                ^^^

Curiously, the incorrect code predates the lmi epoch, but the
correct comment was added 20080327T0257Z.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]