
|
If you were logged in you would be able to see more operations.
|
|
|
| Severity: |
Minor
|
| Fixed in Change#: |
6,151
|
| Runtime: |
N/A
|
| Flags: |
External
|
| Fix in hand: |
True
|
|
As you see, "target_element" will be of type "LzDataElement", and as you also see it will be compared against a "LzView" ( "immediateparent.subviews[0] == target_element " ):
"datalistselector.lzx":
[code]
<method name="findIndex" args="s"> <![CDATA[
var target_element;
if (s instanceof LzView) {
target_element = s.datapath.p;
} else {
target_element = s.p;
}
if ( ! immediateparent.subviews[0].cloneManager ) {
return (immediateparent.subviews[0] == target_element ? 0 : -1);
}
...
</method>
[/code]
That's like saying "true == false" and asking yourself why this condition will never be true...
Fixed version must be:
[code]
if ( ! immediateparent.subviews[0].cloneManager ) {
return (immediateparent.subviews[0] == s ? 0 : -1);
}
[/code]
|
|
Description
|
As you see, "target_element" will be of type "LzDataElement", and as you also see it will be compared against a "LzView" ( "immediateparent.subviews[0] == target_element " ):
"datalistselector.lzx":
[code]
<method name="findIndex" args="s"> <![CDATA[
var target_element;
if (s instanceof LzView) {
target_element = s.datapath.p;
} else {
target_element = s.p;
}
if ( ! immediateparent.subviews[0].cloneManager ) {
return (immediateparent.subviews[0] == target_element ? 0 : -1);
}
...
</method>
[/code]
That's like saying "true == false" and asking yourself why this condition will never be true...
Fixed version must be:
[code]
if ( ! immediateparent.subviews[0].cloneManager ) {
return (immediateparent.subviews[0] == s ? 0 : -1);
}
[/code] |
Show » |
| There are no comments yet on this issue.
|
|