From 60e43a746a8ba62ef4cd14a635ac219ace04c655 Mon Sep 17 00:00:00 2001
From: Jos Groot Lipman <jos.grootlipman@aareon.nl>
Date: Thu, 17 Sep 2015 09:59:40 +0000
Subject: [PATCH] FSN#33592 Verkoop facturen nu met kop

svn path=/Website/trunk/; revision=26369
---
 APPL/API2/model_fin_verkoopfactuur.inc | 54 +++++++++-----------------
 APPL/SCF/scaffolding_print.inc         |  2 +-
 APPL/SCF/scaffolding_show.inc          |  2 +-
 APPL/SCF/scaffolding_wrap.inc          |  4 +-
 4 files changed, 23 insertions(+), 39 deletions(-)

diff --git a/APPL/API2/model_fin_verkoopfactuur.inc b/APPL/API2/model_fin_verkoopfactuur.inc
index 4b27777050..0e1f061e29 100644
--- a/APPL/API2/model_fin_verkoopfactuur.inc
+++ b/APPL/API2/model_fin_verkoopfactuur.inc
@@ -39,6 +39,11 @@ function model_fin_verkoopfactuurregels()
       "label": L("lcl_fin_verkoopfactuur_id"),
       "typ": "varchar"
     },
+    "head": {
+      "dbs": "fin_verkoopfactuurkop_key",
+      "label": L("fin_verkoopfactuurkop"),
+      "typ": "key"
+    },
     "invoicedate": {
       "dbs": "fin_verkoopfactuur_datum",
       "label": L("lcl_fin_verkoopfactuur_datum"),
@@ -329,7 +334,7 @@ model_fin_verkoopfactuur = function()
     };
     this.primary = "fin_verkoopfactuur_id";
     this.includes = { regels: { model: new model_fin_verkoopfactuurregels(),
-                                joinfield: "name" } }
+                                joinfield: "head" } }
     this.list = {
               "columns": [
                   "id",
@@ -337,7 +342,7 @@ model_fin_verkoopfactuur = function()
                   "fin_verkoopfactuur_bedrag",
                   "fin_verkoopfactuur_btw"
               ],
-              default_url : "appl/fin/fin_verkoopfactuur_verzamel.asp?mode=wrap&id="
+              default_url : "appl/fin/fin_verkoopfactuurkop.asp?mode=wrap&id="
             };
     this.search = {
       "filters": [
@@ -355,58 +360,37 @@ model_fin_verkoopfactuur = function()
 }
 
 // Dit is het model voor alle factuurregels van een fin_verkoopfactuur_id
-fin_verkoopfactuur_verzamel = function()
+fin_verkoopfactuurkop = function()
 {
-    this.table = "verzamel"
-    this.tablesql = "(SELECT fin_verkoopfactuur_id"
-                  + "      , MIN(fin_verkoopfactuur_key) min_factuur_key"
-                  + "      , SUM(fin_verkoopfactuur_bedrag) fin_verkoopfactuur_bedrag"
-                  + "      , SUM(fin_verkoopfactuur_btw) fin_verkoopfactuur_btw"
-                  + "      , COUNT(*) aantalregels"
-                  + "   FROM FIN_VERKOOPFACTUUR"
-                  + "  GROUP BY fin_verkoopfactuur_id) verzamel";
+    this.table = "fin_verkoopfactuurkop"
     this.fields = {
         "id": {
-          "dbs": "fin_verkoopfactuur_id",
+          "dbs": "fin_verkoopfactuurkop_key",
+          "label": "Key",
+          "typ": "key"
+        },
+        "name": {
+          "dbs": "fin_verkoopfactuurkop_id",
           "label": L("lcl_fin_verkoopfactuur_id"),
           "typ": "varchar"
-        },
-        "min_factuur_key": {
-          "dbs": "min_factuur_key",
-          "label": L("lcl_fin_verkoopfactuur_id"),
-          "typ": "key",
-          hidden_fld: true
-        },
-        "fin_verkoopfactuur_bedrag": {
-          "dbs": "fin_verkoopfactuur_bedrag",
-          "label": L("lcl_fin_verkoopfactuur_bedrag"),
-          "typ": "float"
-        },
-        "fin_verkoopfactuur_btw": {
-          "dbs": "fin_verkoopfactuur_btw",
-          "label": L("lcl_fin_verkoopfactuur_btw"),
-          "typ": "float",
-          "total": true
         }
     };
-    this.primary = "fin_verkoopfactuur_id";
+    this.primary = "fin_verkoopfactuurkop_key";
     this.includes = { regels: { model: new model_fin_verkoopfactuurregels(),
                                 joinfield: "name" } }
     this.list = {
               "columns": [
-                  "id",
-                  "fin_verkoopfactuur_bedrag",
-                  "fin_verkoopfactuur_btw"
+                  "name"
               ]
             };
     this.search = {
       "filters": [
-        "id"
+        "name"
       ]
     };
     this.print = {
           xmlnode: "verkoopfactuur",
-          key: "min_factuur_key"
+          key: "id"
     };
   this.autfunction = "WEB_VERFAC";
   this.record_title = L("lcl_verkoopfactuur_verzamel"),
diff --git a/APPL/SCF/scaffolding_print.inc b/APPL/SCF/scaffolding_print.inc
index 973cae7357..cb4dc4bbf9 100644
--- a/APPL/SCF/scaffolding_print.inc
+++ b/APPL/SCF/scaffolding_print.inc
@@ -26,7 +26,7 @@ function scaffolding_print(model, scf_params)
             allincludes.push(inc);
     }
 
-    var key = getQParam("id");
+    var key = getQParamInt("id");
     var xxx_params = { filter: { id: key }, include: allincludes };
     var xxx_array = model.REST_GET(xxx_params);
     if (!xxx_array.length)
diff --git a/APPL/SCF/scaffolding_show.inc b/APPL/SCF/scaffolding_show.inc
index 2aa203c9cb..85ce123050 100644
--- a/APPL/SCF/scaffolding_show.inc
+++ b/APPL/SCF/scaffolding_show.inc
@@ -26,7 +26,7 @@ function scaffolding_show(model, scf_params)
     if ("transit" in scf_params)
         transit = "&" + scf_params.transit.name + "=" + safe.url(scf_params.transit.val);
 
-    var key = getQParam("id");
+    var key = getQParamInt("id");
     var xxx_params = { filter: { id: key } };
     var xxx_array = model.REST_GET(xxx_params);
     if (!xxx_array.length)
diff --git a/APPL/SCF/scaffolding_wrap.inc b/APPL/SCF/scaffolding_wrap.inc
index 0fe4c46d49..34f5293eab 100644
--- a/APPL/SCF/scaffolding_wrap.inc
+++ b/APPL/SCF/scaffolding_wrap.inc
@@ -60,7 +60,7 @@ function scaffolding_wrap(model, scf_params)
       </head>
 
       <body>
-    <%  if (key == -1) // Bij fin_verkoopfactuur is de key alfanumeriek. Pas daarvoor op.
+    <%  if (key < 0)
           var page = scf_params.this_fullpath + "?mode=edit" + transit; // Maak een nieuw
         else
           var page = scf_params.this_fullpath + "?mode=show" + transit;
@@ -68,7 +68,7 @@ function scaffolding_wrap(model, scf_params)
 
         IFRAMER("scfFrame", page, { FcltClose: "scfClose" } );
 
-        if (key != -1)
+        if (key > 0)
         {
             for (var inc in model.includes)
             {