Bug 223

Summary: [SECTStudio-Classifier] Demote Button does not work
Product: MapLab Reporter: Chris Thorne <cthorne@dmsolutions.ca>
Component: SECT (Studio)Assignee: maplab-dev <maplab-dev@lists.maptools.org>
Status: VERIFIED FIXED    
Severity: normal    
Priority: P2    
Version: 3.0   
Target Milestone: ---   
Hardware: PC   
OS: Windows 2000   
Whiteboard:

Description From 2004-02-18 15:57:26 
The Demote Button does not work. I have only been able to Promote a class.
------- Comment #1 From 2004-02-18 22:01:09 ------- 
Zak:  you took this one, but I found the problem (same issue in the symbolizer)
... I won't fix it because you may have found it already, but the issue is in
the test

$nSelectedClass != null && $nSelectedClass > 0 && $nSelectedClass <
$oLayer->numclasses - 1

this should read

$nSelectedClass !== null && $nSelectedClass >= 0 && $nSelectedClass <
$oLayer->numclasses - 1

Note the double equals on the first condition, null is the same as 0 unless you
ask for 'exactly equals' ... so fixing the second condition doesn't work because
the first class (0) will fail on the first test until you make this change.
------- Comment #2 From 2004-02-19 08:04:58 ------- 
I made the change, checked the logic of the other operations, and tested. Seems
ok now.

It's easier when someone else diagnoses the problem and writes the code. ;)
------- Comment #3 From 2004-02-19 08:07:29 ------- 
*** Bug 224 has been marked as a duplicate of this bug. ***