finnishtransportagency · Jarzka · Apr 18, 2017 · Apr 18, 2017 · Apr 18, 2017 · Apr 18, 2017
diff --git a/resources/api/documentation/harja-api-versiohistoria.md b/resources/api/documentation/harja-api-versiohistoria.md
@@ -1,7 +1,9 @@
-<b>Nykyinen versio: 1.0.22</b>  
-<b>Julkaistu: 18.4.2017</b>
+<b>Nykyinen versio: 1.0.23</b>  
+<b>Julkaistu: 19.4.2017</b>
 
 <b>Versiohistoria:</b>
+- Versionumero: 1.0.23 Julkaistu: 19.4.2017:
+    - Ylläpidon urakoiden aikataulun kirjauksesta poistettu kenttä tiemerkinta-takaraja. Vain tilaaja voi muuttaa kyseistä arvoa, joten tätä ei ole tarkoitus muokata urakoitsijan rajapinnasta.
 - Versionumero: 1.0.22 Julkaistu: 18.4.2017:
     - Päällystyksen aikataulun kirjaus ei enää huomioi kellonaikaa päällystyksen aloitus- ja lopetuskentälle. Aika on syytä antaa jatkossa pelkästään päivämääränä.
 - Versionumero: 1.0.21 Julkaistu: 13.4.2017:

diff --git a/resources/api/examples/paallystyksen-aikataulun-kirjaus-request.json b/resources/api/examples/paallystyksen-aikataulun-kirjaus-request.json
@@ -17,7 +17,6 @@
     "paallystys-aloitettu": "2015-01-10",
     "paallystys-valmis": "2015-01-12",
     "valmis-tiemerkintaan": "2015-01-12",
-    "tiemerkinta-takaraja": "2015-01-26",
     "kohde-valmis": "2015-01-16",
     "paallystysilmoitus": {
       "takuupvm": "2015-02-15"

diff --git a/resources/api/examples/tiemerkinnan-aikataulun-kirjaus-request.json b/resources/api/examples/tiemerkinnan-aikataulun-kirjaus-request.json
@@ -13,7 +13,6 @@
     "lahetysaika": "2015-01-30T12:00:00+02:00"
   },
   "aikataulu": {
-    "tiemerkinta-takaraja": "2015-01-26",
     "tiemerkinta-aloitettu": "2015-01-12",
     "tiemerkinta-valmis": "2015-01-15"
   }

diff --git a/resources/api/schemas/entities/aikataulu.schema.json b/resources/api/schemas/entities/aikataulu.schema.json
@@ -24,11 +24,6 @@
       "type": "string",
       "format": "date-time"
     },
-    "tiemerkinta-takaraja": {
-      "id": "urn:harja/urakka/sopimukset/0/sopimus/aikataulu-tiemerkinta-takaraja",
-      "type": "string",
-      "format": "date-time"
-    },
     "tiemerkinta-aloitettu": {
       "id": "urn:harja/urakka/sopimukset/0/sopimus/tiemerkinta-aloitettu",
       "type": "string",

diff --git a/resources/api/schemas/entities/tiemerkinnan-aikataulu.schema.json b/resources/api/schemas/entities/tiemerkinnan-aikataulu.schema.json
@@ -4,11 +4,6 @@
   "type": "object",
   "additionalProperties": false,
   "properties": {
-    "tiemerkinta-takaraja": {
-      "id": "urn:harja/urakka/sopimukset/0/sopimus/aikataulu-tiemerkinta-takaraja",
-      "type": "string",
-      "format": "date-time"
-    },
     "tiemerkinta-aloitettu": {
       "id": "urn:harja/urakka/sopimukset/0/sopimus/tiemerkinta-aloitettu",
       "type": "string",

diff --git a/src/clj/harja/kyselyt/paallystys.sql b/src/clj/harja/kyselyt/paallystys.sql
@@ -247,7 +247,7 @@ VALUES (:yllapitokohde, :tyon_tyyppi :: MAARAMUUTOS_TYON_TYYPPI, :tyo, :yksikko,
                         :yksikkohinta, :luoja, :ulkoinen_id, :jarjestelma);
 
 -- name: paivita-yllapitokohteen-maaramuutos<!
-UPDATE yllapitokohteen_maaramuutos
+UPDATE yllapitokohteen_maaramuutos ym
 SET
   tyon_tyyppi      = :tyon_tyyppi :: MAARAMUUTOS_TYON_TYYPPI,
   tyo              = :tyo,
@@ -262,7 +262,7 @@ SET
 WHERE id = :id
       AND (SELECT urakka
            FROM yllapitokohde
-           WHERE id = :id) = :urakka;
+           WHERE id = ym.yllapitokohde) = :urakka;
 
 -- name: yllapitokohteella-paallystysilmoitus
 SELECT EXISTS(SELECT id

diff --git a/src/clj/harja/kyselyt/yllapitokohteet.sql b/src/clj/harja/kyselyt/yllapitokohteet.sql
@@ -734,7 +734,6 @@ SET
   paallystys_loppu     = :paallystys_loppu,
   kohde_valmis         = :kohde_valmis,
   valmis_tiemerkintaan = :valmis_tiemerkintaan,
-  tiemerkinta_takaraja = :aikataulu_tiemerkinta_takaraja,
   muokattu             = NOW(),
   muokkaaja            = :muokkaaja
 WHERE yllapitokohde = :id;
@@ -753,7 +752,6 @@ UPDATE yllapitokohteen_aikataulu
 SET
   tiemerkinta_alku     = :tiemerkinta_alku,
   tiemerkinta_loppu    = :tiemerkinta_loppu,
-  tiemerkinta_takaraja = :aikataulu_tiemerkinta_takaraja,
   muokattu             = NOW(),
   muokkaaja            = :muokkaaja
 WHERE yllapitokohde = :id;

diff --git a/src/clj/harja/palvelin/integraatiot/api/yllapitokohteet.clj b/src/clj/harja/palvelin/integraatiot/api/yllapitokohteet.clj
@@ -165,7 +165,6 @@
        :paallystys_alku (json/aika-string->java-sql-date (:paallystys-aloitettu aikataulu))
        :paallystys_loppu (json/aika-string->java-sql-date (:paallystys-valmis aikataulu))
        :valmis_tiemerkintaan (json/pvm-string->java-sql-date (:valmis-tiemerkintaan aikataulu))
-       :aikataulu_tiemerkinta_takaraja (json/pvm-string->java-sql-date (:tiemerkinta-takaraja aikataulu))
        :kohde_valmis (json/pvm-string->java-sql-date (:kohde-valmis aikataulu))
        :muokkaaja (:id kayttaja)
        :id kohde-id})
@@ -210,7 +209,6 @@
       db
       {:tiemerkinta_alku (json/pvm-string->java-sql-date (:tiemerkinta-aloitettu aikataulu))
        :tiemerkinta_loppu (json/pvm-string->java-sql-date (:tiemerkinta-valmis aikataulu))
-       :aikataulu_tiemerkinta_takaraja (json/pvm-string->java-sql-date (:tiemerkinta-takaraja aikataulu))
        :muokkaaja (:id kayttaja)
        :id kohde-id})
 

diff --git a/test/clj/harja/palvelin/integraatiot/api/yllapitokohteet_test.clj b/test/clj/harja/palvelin/integraatiot/api/yllapitokohteet_test.clj
@@ -53,7 +53,6 @@
     (is (= 200 (:status vastaus)))
     (is (= 6 (count yllapitokohteet)))
     (is (some? leppajarven-ramppi))
-    (is (some? (:tiemerkinta-takaraja (:aikataulu leppajarven-ramppi))))
     (is (some? (:paallystys-aloitettu (:aikataulu leppajarven-ramppi))))
     (is (some? (:paallystys-valmis (:aikataulu leppajarven-ramppi))))
     (is (some? (:valmis-tiemerkintaan (:aikataulu leppajarven-ramppi))))

diff --git a/test/clj/harja/palvelin/palvelut/yllapitokohteet/maaramuutokset_test.clj b/test/clj/harja/palvelin/palvelut/yllapitokohteet/maaramuutokset_test.clj
@@ -106,4 +106,65 @@
                                       :yllapitokohde-id yllapitokohde-id
                                       :maaramuutokset testipayload
                                       :sopimus-id @muhoksen-paallystysurakan-paasopimuksen-id
-                                      :vuosi 2017})))))
+                                      :vuosi 2017})))))
+
+(deftest maaramuutosten-paivitystoimii
+  (let [yllapitokohde-id (hae-yllapitokohde-leppajarven-ramppi-jolla-paallystysilmoitus)
+        uusi-maaramuutos {:yllapitokohde yllapitokohde-id
+                          :tyyppi :ajoradan-paallyste
+                          :tyo "Testissä luotu määrämuutos"
+                          :yksikko "kg"
+                          :tilattu-maara 100
+                          :toteutunut-maara 120
+                          :yksikkohinta 3}
+        muokattu-maaramuutos {:yllapitokohde yllapitokohde-id
+                              :tyyppi :ajoradan-paallyste
+                              :tyo "Testissä luotu määrämuutos"
+                              :yksikko "kg"
+                              :tilattu-maara 123
+                              :toteutunut-maara 666
+                              :yksikkohinta 3}
+        poistettu-maaramuutos {:yllapitokohde yllapitokohde-id
+                               :tyyppi :ajoradan-paallyste
+                               :tyo "Testissä luotu määrämuutos"
+                               :yksikko "kg"
+                               :tilattu-maara 123
+                               :toteutunut-maara 666
+                               :yksikkohinta 3
+                               :poistettu true}
+        tallenna-maaramuutokset (fn [maaramuutokset]
+                                  (kutsu-palvelua
+                                    (:http-palvelin jarjestelma)
+                                    :tallenna-maaramuutokset +kayttaja-jvh+
+                                    {:urakka-id @muhoksen-paallystysurakan-id
+                                     :yllapitokohde-id yllapitokohde-id
+                                     :maaramuutokset maaramuutokset
+                                     :sopimus-id @muhoksen-paallystysurakan-paasopimuksen-id
+                                     :vuosi 2017}))
+        hae-maaramuutokset #(kutsu-palvelua
+                              (:http-palvelin jarjestelma)
+                              :hae-maaramuutokset +kayttaja-jvh+
+                              {:urakka-id @muhoksen-paallystysurakan-id
+                               :yllapitokohde-id yllapitokohde-id})
+        hae-tarkasteltava-maaramuutos (fn [maaramuutokset]
+                                        (first (filter #(= "Testissä luotu määrämuutos" (:tyo %)) maaramuutokset)))
+        maaramuutokset-ennen-testia (count (hae-maaramuutokset))]
+
+    (tallenna-maaramuutokset [uusi-maaramuutos])
+
+    (let [maaramuutokset (hae-maaramuutokset)
+          maaramuutos-id (:id (hae-tarkasteltava-maaramuutos maaramuutokset))]
+      (is (= (+ 1 maaramuutokset-ennen-testia) (count maaramuutokset)) "Kirjauksen jälkeen löytyi vain 1 uusi määrämuutos")
+      (is maaramuutos-id "Id on saatu haettua")
+
+      (tallenna-maaramuutokset [(assoc muokattu-maaramuutos :id maaramuutos-id)])
+      (let [maaramuutokset (hae-maaramuutokset)
+            maaramuutos (hae-tarkasteltava-maaramuutos maaramuutokset)]
+        (is (= (+ 1 maaramuutokset-ennen-testia) (count maaramuutokset)) "Muokkauksen jälkeen löytyi sama määrä määrämuutoksia")
+        (is (= 123M (:tilattu-maara maaramuutos)) "Tilattu määrä on päivitetty oikein")
+        (is (= 666M (:toteutunut-maara maaramuutos)) "Toteutunut määrä on päivitetty oikein"))
+
+      (tallenna-maaramuutokset [(assoc poistettu-maaramuutos :id maaramuutos-id)])
+      (is (= maaramuutokset-ennen-testia (count (hae-maaramuutokset)))))
+
+    (u "DELETE FROM yllapitokohteen_maaramuutos WHERE tyo = 'Testissä luotu määrämuutos';")))
diff --git a/...allystyksen_aikataulun_kirjaus_viallinen_paallystys_valmis_ilman_paallystyksen_alkua.json b/...allystyksen_aikataulun_kirjaus_viallinen_paallystys_valmis_ilman_paallystyksen_alkua.json
@@ -17,7 +17,6 @@
     "paallystys-aloitettu": null,
     "paallystys-valmis": "2015-01-12T12:00:00Z",
     "valmis-tiemerkintaan": "2015-01-12",
-    "tiemerkinta-takaraja": "2015-01-26",
     "kohde-valmis": "2015-01-16",
     "paallystysilmoitus": {
       "takuupvm": "2015-02-15"

diff --git a/...paallystyksen_aikataulun_kirjaus_viallinen_tiemerkintapvm_ilman_paallystyksen_loppua.json b/...paallystyksen_aikataulun_kirjaus_viallinen_tiemerkintapvm_ilman_paallystyksen_loppua.json
@@ -17,7 +17,6 @@
     "paallystys-aloitettu": "2015-01-10T12:00:00Z",
     "paallystys-valmis": null,
     "valmis-tiemerkintaan": "2015-01-12",
-    "tiemerkinta-takaraja": "2015-01-26",
     "kohde-valmis": "2015-01-16",
     "paallystysilmoitus": {
       "takuupvm": "2015-02-15"

diff --git a/...surssit/api/tiemerkinnan_aikataulun_kirjaus_viallinen_tiemerkinta_valmis_ilman_alkua.json b/...surssit/api/tiemerkinnan_aikataulun_kirjaus_viallinen_tiemerkinta_valmis_ilman_alkua.json
@@ -13,7 +13,6 @@
     "lahetysaika": "2015-01-30T12:00:00Z"
   },
   "aikataulu": {
-    "tiemerkinta-takaraja": "2015-01-26",
     "tiemerkinta-aloitettu": null,
     "tiemerkinta-valmis": "2015-01-15"
   }