FSN#25848 Forse code refactoring. Geen functionele wijzigingen

svn path=/Website/trunk/; revision=19586

APPL/FIN/fin.inc

@@ -924,7 +924,6 @@ fin = { func_enabled_factuur: // Wat mag ik zoal op deze specifieke factuur?
                 oRs.Close();
             }
 
-            result.btwtabel = [];
             if (result.btwtabel_key)
             {
               result.btwtabel = fin.btwtabel_info(result.btwtabel_key);
@@ -1075,13 +1074,13 @@ fin = { func_enabled_factuur: // Wat mag ik zoal op deze specifieke factuur?
 
             var htmlText = "<select type='text' class='fldfinbtw' id='BTW" + rowIndex
                      + "' name='BTW" + rowIndex + "'>"
-            for (var j = 0; j < ref_info.btwtabel.length; j++)
+            for (var j = 0; j < params.btwtabel.length; j++)
             {
-              if (params.btwkey == ref_info.btwtabel[j].key)
-                thisBTW = ref_info.btwtabel[j];
-              htmlText += "<option value='" + ref_info.btwtabel[j].key + "'"
-                        + (params.btwkey == ref_info.btwtabel[j].key? " selected='1'" : "") + ">"
-                        + safe.html(ref_info.btwtabel[j].oms) + "</option>"
+              if (params.btwkey == params.btwtabel[j].key)
+                thisBTW = params.btwtabel[j];
+              htmlText += "<option value='" + params.btwtabel[j].key + "'"
+                        + (params.btwkey == params.btwtabel[j].key? " selected='1'" : "") + ">"
+                        + safe.html(params.btwtabel[j].oms) + "</option>"
             }
             htmlText += "</select>";
             html.push("<td align='right'>" + htmlText + "</td>");
@@ -1109,7 +1108,7 @@ fin = { func_enabled_factuur: // Wat mag ik zoal op deze specifieke factuur?
                    + "     , fr.fin_factuurregel_omschrijving"
                    + "     , fr.fin_factuurregel_referentie"
                    + "     , fr.prs_kostensoort_key"
-                   + "     , ks.prs_kostensoort_oms || '-' || " + lcl.xsql("ks.prs_kostensoort_opmerking", "ks.prs_kostensoort_key") + " kostensoort"
+                   + "     , NVL2(ks.prs_kostensoort_key, ks.prs_kostensoort_oms || '-' || " + lcl.xsql("ks.prs_kostensoort_opmerking", "ks.prs_kostensoort_key") + ", NULL) kostensoort"
                    + "     , (SELECT btw.fin_btwtabelwaarde_verlegd"
                      + "        FROM fin_btwtabelwaarde btw"
                      + "       WHERE btw.fin_btwtabelwaarde_key = fr.fin_btwtabelwaarde_key"
@@ -1122,7 +1121,7 @@ fin = { func_enabled_factuur: // Wat mag ik zoal op deze specifieke factuur?
           },
 
           // Bewerkbare regels
-          generateFINregels: function _generateFINregels()
+          generateFINregels: function _generateFINregels(btwtabel)
           {
                var rowIndex = 0;
                var oRs = Oracle.Execute(fin.regelSQL(fin_key));
@@ -1134,7 +1133,8 @@ fin = { func_enabled_factuur: // Wat mag ik zoal op deze specifieke factuur?
                                        regel_nr: oRs("fin_factuurregel_nr").value,
                                        sum: oRs("fin_factuurregel_totaal").value,
                                        BTW: oRs("BTW").value,
-                                       btwkey: oRs("fin_btwtabelwaarde_key").value ,
+                                       btwkey: oRs("fin_btwtabelwaarde_key").value,
+                                       btwtabel: btwtabel,
                                        sumBTW: oRs("sumBTW").value,
                                        descr: oRs("fin_factuurregel_omschrijving").value,
                                        ref: oRs("fin_factuurregel_referentie").value,

APPL/FIN/fin_edit_factuur.asp

@@ -435,7 +435,7 @@ var fin_btwtabel_key = oRs("fin_btwtabel_key").value; // Standaard btw tabel key
                     <th style="text-align:right"><%=L("lcl_fin_btwtarief")%></th>
                     <th style="text-align:right"><%=L("lcl_fin_total_sum_inBTW")%></th>
                 </tr>
-        <%      var rowIndex = fin.generateFINregels();
+        <%      var rowIndex = fin.generateFINregels(serversidebtwtabel);
         %>
                 </table>
 

APPL/FIN/fin_edit_factuur.js

@@ -153,12 +153,7 @@ function FcltGetRefInfoCallback(json)
     if (json.ref_type)
     {
       // Hoeveel factuurregels zijn er?
-      var count = 0;
-      $("table#sel_items input[id^=sum]").each(function()
-      {
-          if (myParseFloat($(this).val()) != 0)
-            count++;
-      });
+      var count = $("table#sel_items input[id^=sum]").length;
 
       // Als er door de referenties verschillende btw tabellen wordt gebruikt terwijl er factuurregels zijn, dan wijziging niet accepteren en melding geven
       if (count > 0 && window.btwtabel_key != json.btwtabel_key)
@@ -316,7 +311,7 @@ function FcltGetRefInfoCallback(json)
 
         if (count == 0 && json.refchanged && fin_key == -1)
         {
-          AddItem();
+          AddItem(true);
           $("#show_calendar").focus(); // Het eerstvolgende veld
         }
 
@@ -443,7 +438,7 @@ function setPrice(obj)
 }
 
 // Voeg een ieuwe regel toe na het plus-knopje
-function AddItem() // als je hieronder iets aanpast controleer dan ook fin.inc/CreateEditRow
+function AddItem(nofocus) // als je hieronder iets aanpast controleer dan ook fin.inc/CreateEditRow
 {
   var sum = 0;
   var descr = "";
@@ -520,8 +515,8 @@ function AddItem() // als je hieronder iets aanpast controleer dan ook fin.inc/C
                  + "' value='" + currency_pref + num2curr(sumBTW) + "'>";
   cell.align = 'right';
 
-  document.getElementById("descript" + rowIndex).focus();
-
+  if (!nofocus)
+    $("#descript" + rowIndex).focus();
 }
 
 function DeleteOrderLine(obj)