Changeset 1425

Show
Ignore:
Timestamp:
17/12/09 17:14:06 (3 months ago)
Author:
bruno
Message:

* fixing wrong year usage

  • in some cases the current year was used instead of the selected_year, leading to problems working with other years than the current
  • after saving, the 'selected week' is now correctly highlighted (before it always jumped back to the current week)

* selected_week and selected_year:

  • are now only initialized once on page load
  • always contain the numeric values now (I hope)
Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • tupper/trunk/tupper-site/src/main/kauri/static/js/time.js

    r1422 r1425  
    22var dataURI = ""; 
    33var performancesForm = null; 
    4 var selected_week = 0
     4var selected_week = currentWeek
    55var selected_year = currentYear; 
    66var editChild = null; 
     
    1818        $.getJSON(urlPerf, fillPerformances); 
    1919 
    20         var current_week = week.split("y")[0].substring(1, week.split("y")[0].length); 
    21         //alert(current_week); 
    22         var year = new Date().getFullYear(); 
    2320        var firstweek = selected_week - 6; 
    2421        var lastweek = firstweek + 11; 
    2522 
    2623        var url = baseUri + "week/" + selectedEmpl +"/"; 
    27         url  = url + "w" + firstweek + "y" + year + "-w" + lastweek + "y" + year; 
     24        url  = url + "w" + firstweek + "y" + selected_year + "-w" + lastweek + "y" + selected_year; 
    2825 
    2926        $.getJSON(url, fillWeeks); 
     
    109106 
    110107    $("#copy_previous_data").click(function(){ 
    111         var week = "w" + (selected_week*1 - 1) + "y" + selected_year; 
     108        var week = "w" + (selected_week - 1) + "y" + selected_year; 
    112109        url = baseUri + "performance/" + selectedEmpl + "/" + week; 
    113110        copiedFromPrevious = true; 
     
    273270     
    274271function getInitWeeks(){ 
    275     selected_week = currentWeek; 
    276     var year = new Date().getFullYear(); 
    277272    var firstweek = selected_week - 6; 
    278273    var lastweek = firstweek + 11; 
    279274 
    280275    // server does lenient interpretation of the week range 
    281     var weekRange = "w" + firstweek + "y" + year + "-w" + lastweek + "y" + year; 
     276    var weekRange = "w" + firstweek + "y" + selected_year + "-w" + lastweek + "y" + selected_year; 
    282277    var url = baseUri + "week/" + selectedEmpl +"/" + weekRange; 
    283278 
     
    286281     
    287282function getInitPerformances(){ 
    288     selected_week = currentWeek; 
    289     var url = baseUri + "performance/" + selectedEmpl + "/w"; //----------------------------------------// 
    290     var current_week = currentWeek; 
    291     var year = new Date().getFullYear(); 
    292       
    293     url = url + current_week + "y" + year + "/"; 
    294     week = "w"+ current_week + "y" + year; 
     283    week = "w"+ selected_week + "y" + selected_year; 
     284 
     285    var url = baseUri + "performance/" + selectedEmpl + "/" + week; 
    295286    performancesForm.initDataURIs({ 
    296287        "dataURI": url 
     
    323314    //        $(".pager td:first").text("Previous Weeks Incomplete"); 
    324315    //      } 
    325     selected_week = currentWeek; 
    326316    var data = json[0].data; 
    327317    $("#weeks tbody").empty(); 
     
    338328            line = line + " this_week"; 
    339329        } 
    340         if (selected_week == dataWeek && dataYear == week.split("y")[1]){ 
     330        if (selected_week == dataWeek && selected_year == dataYear){ 
    341331            line = line + " selected "; 
    342332        } 
     
    377367            selected_year = week.substring(3,week.length); 
    378368        }else{ 
    379             selected_year = week.substring(4,week.length) 
    380         } 
     369            selected_year = week.substring(4,week.length); 
     370        } 
     371 
     372        selected_week = selected_week*1; 
     373        selected_year = selected_year*1; 
     374 
    381375        url = baseUri + "performance/" + selectedEmpl + "/" + week; //--------------------------// 
    382376        $.getJSON(url,fillPerformances);