Changeset a91a7822b0b574212b7e50d0cb3c95aa9fc71043

Show
Ignore:
Timestamp:
06/10/09 17:35:30 (5 years ago)
Author:
Umar Farooq <umar@omniti.com>
git-committer:
Umar Farooq <umar@omniti.com> 1244655330 +0000
git-parent:

[a6f44d6b86937e3b7a7f84eea82557562396a359]

git-author:
Umar Farooq <umar@omniti.com> 1244655330 +0000
Message:

styling changes for template browsing, and validation to ensure each SID placeholder and TEXT placeholder have values set before making graphs.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • ui/web/htdocs/css/style.css

    r43db6fd ra91a782  
    530530padding-left:30px; 
    531531} 
     532#browselist ul li ul li span.sidvar { 
     533background:none; 
     534padding-left:0; 
     535} 
     536#browselist ul li ul li span.sidvar span.sidvarspan{ 
     537background:none; 
     538padding-left:15px; 
     539} 
     540#browselist ul li ul li#textvar span{ 
     541background:none; 
     542padding-left:15px; 
     543} 
    532544#browselist ul li ul li span.numeric { 
    533545background:transparent url(../images/icon-data.png) no-repeat scroll 15px 1px; 
     
    552564#browselist ul li ul li ul li span.templatesid { 
    553565padding-left:30px; 
     566} 
     567#browselist ul li ul li#graphcreatebutton span{ 
     568background:none; 
     569padding-left:8px; 
    554570} 
    555571#browselist ul li ul li ul li span.CreateGraph { 
  • ui/web/htdocs/template_browse_json.php

    ra6f44d6 ra91a782  
    108108         foreach($textvars as $tv) { 
    109109           $params = array(); 
    110            $ctext = "<br>".$tv.":"; 
    111            $ctext.="<input type='text' size='15' name='".$tv."' id='textvar'><br>"; 
    112            $jitem = array ('id' => '', 
     110           $ctext = $tv.":"; 
     111           $ctext.="<input type='text' size='15' name='".$tv."' id='textvar'>"; 
     112           $jitem = array ('id' => 'textvar', 
    113113                           'text' => $ctext, 
    114114                           'hasChildren' => false, 
     
    138138sidvars =[]; 
    139139 
     140//if we have selected atleast one sid for each SID placeholder 
     141//and set values for TEXT placeholder, make the graphs 
     142var have_req_txtvals = false; 
     143var have_req_sidvals = false; 
     144 
     145 
    140146var template_e = $("#"+templateid); 
    141147 
    142 template_e.find("#textvar").each ( function (i) { 
    143         textvals+=  "&"+$(this).attr('name')+"="+$(this).val(); 
    144         textvars[i] = $(this).attr('name'); 
     148template_e.find("input[@id='textvar']").each ( function (i) { 
     149        textvars[i] = $(this).attr('name'); 
     150        if($(this).val() != '') { 
     151          have_req_txtvals = true; 
     152          textvals+=  "&"+$(this).attr('name')+"="+$(this).val(); 
     153        } 
     154        else have_req_txtvals = false; 
    145155}); 
    146156 
     
    164174 
    165175for (i=0; i<sidvars.length; i++){ 
    166         sidvar_sid_map[sidvars[i]].sort(); 
    167         sidvals+="&"+sidvars[i]+"="+sidvar_sid_map[sidvars[i]].join(","); 
    168 
    169  
    170 //if we have selected atleast one sid 
    171 if(sids.length > 0){      
     176  if(sidvar_sid_map[sidvars[i]]) { 
     177    sidvar_sid_map[sidvars[i]].sort(); 
     178    have_req_sidvals=true; 
     179    sidvals+="&"+sidvars[i]+"="+sidvar_sid_map[sidvars[i]].join(","); 
     180  } 
     181  else { 
     182    have_req_sidvals=false; 
     183  } 
     184
     185 
     186if(have_req_txtvals && have_req_sidvals){ 
    172187        var dataString = 'templateid='+templateid+'&textvars='+textvars.join(',')+'&sidvars='+sidvars.join(',')+textvals+sidvals; 
    173188        $.ajax({ 
     
    177192        success: function() { } 
    178193        }); 
    179         $(".CreateGraph").html('Graph Saved').fadeIn('slow'); 
    180         $(".CreateGraph").html('Graph Saved').fadeOut('slow'); 
     194        template_e.find(".CreateGraph").html('Graph Saved').fadeIn('slow'); 
     195        template_e.find(".CreateGraph").html('Graph Saved').fadeOut('slow'); 
    181196} 
    182197else { 
    183      modal_warning("Graph creation error", "You need to select atleast one sid!"); 
     198     modal_warning("Graph creation error", "You need to select atleast one value for each SID placeholder ("+sidvars.join(",")+"), and each TEXT place holder ("+textvars.join(",")+")!"); 
    184199} 
    185200} 
     
    187202</script> 
    188203JS; 
    189 $jitem = array ('id' => '', 
     204$jitem = array ('id' => 'graphcreatebutton', 
    190205                           'text' => $ctext, 
    191206                           'hasChildren' => false, 
    192                            'classes' => $tv
     207                           'classes' => ''
    193208                           'params' => array(), 
    194209                           );