[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.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [lmi] Unable to rename employee class--fixed,
Greg Chicares <=