[Bug 661] New: comparison bug in loop macro in HopObject.js

View: New views
3 Messages — Rating Filter:   Alert me  

[Bug 661] New: comparison bug in loop macro in HopObject.js

by Bugzilla from bugzilla-daemon@helma.at :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://dev.helma.org/bugs/show_bug.cgi?id=661

           Summary: comparison bug in loop macro in HopObject.js
           Product: Helma
           Version: 1.6.3
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: major
          Priority: P5
         Component: helmaLib, helmaTools
        AssignedTo: helma-dev@...
        ReportedBy: maksim.lin@...


On line 139, the comparison:
if (!test || isNaN(test)) {

fails incorrectly if the first elements property that you are using is numeric
and happens to be value of zero (thanks to js type coercion).
This fixes the problem:
((test === undefined) || (test === null) || isNaN(test)) {

--
Configure bugmail: http://dev.helma.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Helma-dev mailing list
Helma-dev@...
http://helma.org/mailman/listinfo/helma-dev

[Bug 661] comparison bug in loop macro in HopObject.js

by Bugzilla from bugzilla-daemon@helma.at :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://dev.helma.org/bugs/show_bug.cgi?id=661


Hannes Wallnoefer <hannes@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |hannes@...
         Resolution|                            |FIXED




--- Comment #1 from Hannes Wallnoefer <hannes@...>  2009-07-06 14:32:18 ---
Fixed in svn.

--
Configure bugmail: http://dev.helma.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Helma-dev mailing list
Helma-dev@...
http://helma.org/mailman/listinfo/helma-dev

[Bug 661] comparison bug in loop macro in HopObject.js

by Bugzilla from bugzilla-daemon@helma.at :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://dev.helma.org/bugs/show_bug.cgi?id=661


Hannes Wallnoefer <hannes@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |1.7.0




--
Configure bugmail: http://dev.helma.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Helma-dev mailing list
Helma-dev@...
http://helma.org/mailman/listinfo/helma-dev