Changeset dd8b4487012abde6396680a0aa2a55b9385a4d4f

Show
Ignore:
Timestamp:
05/10/12 12:21:33 (2 years ago)
Author:
Theo Schlossnagle <jesus@lethargy.org>
git-committer:
Theo Schlossnagle <jesus@lethargy.org> 1336652493 -0700
git-parent:

[f5979a3536dff652504b40af656f4484ca42a331], [8c7268cc2fa5e944f69acbb2db8383c0d4283d15]

git-author:
Theo Schlossnagle <jesus@lethargy.org> 1336652493 -0700
Message:

Merge pull request #72 from gsandie/composite_fix

Handle nil values gracefully in composites

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • ui/web/htdocs/js/recon.js

    r4fceaf1 r8c7268c  
    545545                        if (data[mat] && (data[mat].metric_type == 'numeric') && data[mat].data[i][1])  
    546546                            return data[mat].data[i][1]; 
    547                         else  
     547                        //If it is a numeric dataset but contains a null value, return the null 
     548                        else if (data[mat] && (data[mat].metric_type == 'numeric') && (data[mat].data[i][1] == "")) 
     549                            return null; 
     550                        else 
    548551                            return "BAD"; 
    549552                    }); 
    550                     if (nexpr.match(/BAD/))  
     553 
     554                    var val; 
     555                    if (nexpr.match(/BAD/))  { 
    551556                        return false; 
    552                     var val = my_rpn_eval(nexpr, {}); 
     557                    } 
     558                    // any nulls that were returned should be replaced with a 0 for calculation 
     559                    else if (nexpr.match(/null/)) { 
     560                        nexpr = nexpr.replace(/null/g,'0.0'); 
     561                    } 
     562                    val = my_rpn_eval(nexpr, {}); 
    553563                    data[cindex].data.push([data[nindex].data[i][0], val]); 
    554564                }