feat: Aloituskuulutuksen lataaminen kansalaisen käyttöliittymästä (#135)

backend/integrationtest/api/__snapshots__/api.test.ts.snap

@@ -0,0 +1,66 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Api should search, load and save a project 1`] = `
+Object {
+  "__typename": "ProjektiJulkinen",
+  "aloitusKuulutusJulkaisut": Array [
+    Object {
+      "__typename": "AloitusKuulutusJulkaisuJulkinen",
+      "aloituskuulutusPDFt": Object {
+        "SUOMI": Object {
+          "aloituskuulutusIlmoitusPDFPath": "/tiedostot/suunnitelma/1.2.246.578.5.1.2978288874.2711575506/aloituskuulutus/ILMOITUS TOIMIVALTAISEN VIRANOMAISEN KUULUTUKSESTA HASSU AUTOMAATTITESTIPROJEKTI1.pdf",
+          "aloituskuulutusPDFPath": "/tiedostot/suunnitelma/1.2.246.578.5.1.2978288874.2711575506/aloituskuulutus/KUULUTUS SUUNNITTELUN ALOITTAMISESTA HASSU AUTOMAATTITESTIPROJEKTI1.pdf",
+        },
+        "__typename": "AloitusKuulutusPDFt",
+      },
+      "elyKeskus": "Pirkanmaa",
+      "hankkeenKuvaus": Object {
+        "RUOTSI": "På Svenska",
+        "SAAME": "Saameksi",
+        "SUOMI": "Lorem Ipsum",
+        "__typename": "HankkeenKuvaukset",
+      },
+      "kielitiedot": Object {
+        "__typename": "Kielitiedot",
+        "ensisijainenKieli": "SUOMI",
+      },
+      "kuulutusPaiva": "2022-01-02",
+      "siirtyySuunnitteluVaiheeseen": "2022-01-01",
+      "suunnitteluSopimus": undefined,
+      "velho": Object {
+        "__typename": "Velho",
+        "kunnat": Array [
+          "Helsinki",
+          " Vantaa",
+        ],
+        "nimi": "HASSU AUTOMAATTITESTIPROJEKTI1",
+        "tilaajaOrganisaatio": "Väylävirasto",
+        "tyyppi": "TIE",
+        "vaylamuoto": Array [
+          "tie",
+        ],
+      },
+      "yhteystiedot": Array [
+        Object {
+          "__typename": "Yhteystieto",
+          "etunimi": "A-tunnus1",
+          "organisaatio": "CGI Suomi Oy",
+          "puhelinnumero": "123",
+          "sahkoposti": "mikko.haapamki@cgi.com",
+          "sukunimi": "Hassu",
+        },
+        Object {
+          "__typename": "Yhteystieto",
+          "etunimi": "Marko",
+          "organisaatio": "Kajaani",
+          "puhelinnumero": "0293121213",
+          "sahkoposti": "markku.koi@koi.com",
+          "sukunimi": "Koi",
+        },
+      ],
+    },
+  ],
+  "euRahoitus": false,
+  "oid": "1.2.246.578.5.1.2978288874.2711575506",
+}
+`;

backend/integrationtest/api/api.test.ts

@@ -11,6 +11,8 @@ import {
   ProjektiRooli,
   SuunnitteluSopimus,
   SuunnitteluSopimusInput,
+  TilasiirtymaToiminto,
+  TilasiirtymaTyyppi,
 } from "../../../common/graphql/apiModel";
 import fs from "fs";
 import axios from "axios";
@@ -63,11 +65,10 @@ describe("Api", () => {
     const { oid, projekti } = await readProjektiFromVelho();
 
     // Expect that projektipaallikko is found
-    expect(
-      projekti.kayttoOikeudet?.filter(
-        (kayttaja) => kayttaja.rooli === ProjektiRooli.PROJEKTIPAALLIKKO && kayttaja.email
-      )
-    ).is.not.empty;
+    const projektiPaallikko = projekti.kayttoOikeudet
+      ?.filter((kayttaja) => kayttaja.rooli === ProjektiRooli.PROJEKTIPAALLIKKO && kayttaja.email)
+      .pop();
+    expect(projektiPaallikko).is.not.empty;
 
     const kayttoOikeudet = projekti.kayttoOikeudet?.map((value) => ({
       rooli: value.rooli,
@@ -172,6 +173,18 @@ describe("Api", () => {
     expect(updatedProjekti2.suunnitteluSopimus).to.be.undefined;
     expect(updatedProjekti2.kielitiedot).eql(kielitiedot);
 
+    userFixture.loginAsProjektiKayttaja(projektiPaallikko);
+    await api.siirraTila({
+      oid,
+      tyyppi: TilasiirtymaTyyppi.ALOITUSKUULUTUS,
+      toiminto: TilasiirtymaToiminto.LAHETA_HYVAKSYTTAVAKSI,
+    });
+    await api.siirraTila({ oid, tyyppi: TilasiirtymaTyyppi.ALOITUSKUULUTUS, toiminto: TilasiirtymaToiminto.HYVAKSY });
+
+    userFixture.logout();
+    const publicProjekti = await loadProjektiFromDatabase(oid);
+    expect(publicProjekti).toMatchSnapshot();
+
     // Finally delete the projekti
     const archiveResult = await projektiArchive.archiveProjekti(oid);
     expect(archiveResult.oid).to.be.equal(oid);
@@ -188,12 +201,12 @@ describe("Api", () => {
 
   async function loadProjektiFromDatabase(oid: string) {
     const savedProjekti = await api.lataaProjekti(oid);
-    expect(savedProjekti.tallennettu).to.be.true;
+    expect(!savedProjekti.tallennettu || savedProjekti.tallennettu).to.be.true;
     return savedProjekti;
   }
 
   async function searchProjectsFromVelhoAndPickFirst(): Promise<string> {
-    const searchResult = await api.getVelhoSuunnitelmasByName("HASSUTESTIPROJEKTI");
+    const searchResult = await api.getVelhoSuunnitelmasByName("HASSU AUTOMAATTITESTIPROJEKTI1");
     // tslint:disable-next-line:no-unused-expression
     expect(searchResult).not.to.be.empty;
 

backend/src/asiakirja/abstractPdf.ts

@@ -1,4 +1,4 @@
-import { deburr } from "lodash";
+import deburr from "lodash/deburr";
 import log from "loglevel";
 import { PDF } from "../../../common/graphql/apiModel";
 

backend/src/config.ts

@@ -35,6 +35,7 @@ const config = {
 
   emailsOn: process.env.EMAILS_ON,
   emailsTo: process.env.EMAILS_TO,
+  isProd: (): boolean => process.env.ENVIRONMENT == "prod",
 };
 
 process.env.AWS_XRAY_CONTEXT_MISSING = "LOG_ERROR";

backend/src/email/emailTemplates.ts

@@ -1,4 +1,4 @@
-import { get } from "lodash";
+import get from "lodash/get";
 import { config } from "../config";
 import { DBProjekti } from "../database/model/projekti";
 import { EmailOptions } from "./email";

backend/src/endDateCalculator/bankHolidays.ts

@@ -9,7 +9,8 @@ export class BankHolidays {
   }
 
   isBankHoliday(date: Dayjs) {
-    const isWeekened = date.day() === 0 || date.day() === 6;
-    return isWeekened || !!this.bankHolidays.find((bankHoliday) => bankHoliday.isSame(date));
+    const pureDate = date.set("hours", 0).set("minutes", 0);
+    const isWeekened = pureDate.day() === 0 || pureDate.day() === 6;
+    return isWeekened || !!this.bankHolidays.find((bankHoliday) => bankHoliday.isSame(pureDate));
   }
 }

backend/src/endDateCalculator/endDateCalculatorHandler.ts

@@ -1,18 +1,32 @@
 import { LaskePaattymisPaivaQueryVariables } from "../../../common/graphql/apiModel";
-import { dateToString, parseDate } from "../util/dateUtil";
+import { dateTimeToString, dateToString, ISO_DATE_FORMAT, parseDate } from "../util/dateUtil";
 import { Dayjs } from "dayjs";
 import { bankHolidaysClient } from "./bankHolidaysClient";
+import { config } from "../config";
 
 export async function calculateEndDate({ alkupaiva }: LaskePaattymisPaivaQueryVariables) {
-  const start = parseDate(alkupaiva);
-  if (start.isValid()) {
-    let endDate: Dayjs = start.add(1 + 30, "day");
-    const bankHolidays = await bankHolidaysClient.getBankHolidays();
-    while (bankHolidays.isBankHoliday(endDate)) {
-      endDate = endDate.add(1, "day");
+  let start: Dayjs;
+  // Only accept dates in prod, but allow datetimes in other environments
+  const isDateOnly = config.isProd() || alkupaiva.length == ISO_DATE_FORMAT.length;
+  if (isDateOnly) {
+    start = parseDate(alkupaiva);
+    if (!start.isValid()) {
+      new Error("Alkupäivän pitää olla muotoa YYYY-MM-DD tai YYYY-MM-DDTHH:mm");
     }
+  } else {
+    start = parseDate(alkupaiva);
+    if (!start.isValid()) {
+      new Error("Alkupäivän pitää olla muotoa YYYY-MM-DDTHH:mm");
+    }
+  }
+  let endDate: Dayjs = start.add(1 + 30, "day");
+  const bankHolidays = await bankHolidaysClient.getBankHolidays();
+  while (bankHolidays.isBankHoliday(endDate)) {
+    endDate = endDate.add(1, "day");
+  }
+  if (isDateOnly) {
     return dateToString(endDate);
   } else {
-    throw new Error("Alkupäivän pitää olla muotoa YYYY-MM-DD");
+    return dateTimeToString(endDate);
   }
 }

backend/src/files/fileService.ts

@@ -52,19 +52,19 @@ export class FileService {
     const sourceFileProperties = await this.getUploadedSourceFileInformation(filePath);
 
     const fileNameFromUpload = FileService.getFileNameFromPath(filePath);
-    const targetPath =
-      FileService.getProjektiDirectory(param.oid) + `/${param.targetFilePathInProjekti}/${fileNameFromUpload}`;
+    const targetPath = `/${param.targetFilePathInProjekti}/${fileNameFromUpload}`;
+    const targetBucketPath = FileService.getProjektiDirectory(param.oid) + targetPath;
     try {
       await getS3Client().send(
         new CopyObjectCommand({
           ...sourceFileProperties,
           Bucket: config.yllapitoBucketName,
-          Key: targetPath,
+          Key: targetBucketPath,
           MetadataDirective: "REPLACE",
         })
       );
       log.info(
-        `Copied uploaded file (${sourceFileProperties.ContentType}) ${sourceFileProperties.CopySource} to ${targetPath}`
+        `Copied uploaded file (${sourceFileProperties.ContentType}) ${sourceFileProperties.CopySource} to ${targetBucketPath}`
       );
       return targetPath;
     } catch (e) {
@@ -99,6 +99,10 @@ export class FileService {
     return `yllapito/tiedostot/projekti/${oid}`;
   }
 
+  private static getPublicProjektiDirectory(oid: string) {
+    return `tiedostot/suunnitelma/${oid}`;
+  }
+
   async getUploadedSourceFileInformation(
     uploadedFileSource: string
   ): Promise<{ ContentType: string; CopySource: string }> {
@@ -177,6 +181,10 @@ export class FileService {
   getYllapitoPathForProjektiFile(oid: string, path: string): string | undefined {
     return path ? `/${FileService.getProjektiDirectory(oid)}${path}` : undefined;
   }
+
+  getPublicPathForProjektiFile(oid: string, path: string): string | undefined {
+    return path ? `/${FileService.getPublicProjektiDirectory(oid)}${path}` : undefined;
+  }
 }
 
 export const fileService = new FileService();

backend/src/handler/projektiAdapter.ts

@@ -44,7 +44,7 @@ export class ProjektiAdapter {
       kayttoOikeudet: KayttoOikeudetManager.adaptAPIKayttoOikeudet(kayttoOikeudet),
       tyyppi: velho?.tyyppi || dbProjekti.tyyppi, // remove usage of projekti.tyyppi after all data has been migrated to new format
       aloitusKuulutus: adaptAloitusKuulutus(aloitusKuulutus),
-      suunnitteluSopimus: adaptSuunnitteluSopimus(suunnitteluSopimus),
+      suunnitteluSopimus: adaptSuunnitteluSopimus(dbProjekti.oid, suunnitteluSopimus),
       liittyvatSuunnitelmat: adaptLiittyvatSuunnitelmat(liittyvatSuunnitelmat),
       aloitusKuulutusJulkaisut: adaptAloitusKuulutusJulkaisut(dbProjekti.oid, aloitusKuulutusJulkaisut),
       velho: {
@@ -104,7 +104,7 @@ function adaptLiittyvatSuunnitelmat(suunnitelmat?: Suunnitelma[] | null): API.Su
   return suunnitelmat as undefined | null;
 }
 
-function adaptKielitiedot(kielitiedot?: Kielitiedot | null): API.Kielitiedot | undefined | null {
+export function adaptKielitiedot(kielitiedot?: Kielitiedot | null): API.Kielitiedot | undefined | null {
   if (kielitiedot) {
     return {
       ...kielitiedot,
@@ -181,10 +181,15 @@ function adaptAloitusKuulutus(kuulutus?: AloitusKuulutus | null): API.AloitusKuu
 }
 
 function adaptSuunnitteluSopimus(
+  oid: string,
   suunnitteluSopimus?: SuunnitteluSopimus | null
 ): API.SuunnitteluSopimus | undefined | null {
   if (suunnitteluSopimus) {
-    return { __typename: "SuunnitteluSopimus", ...suunnitteluSopimus };
+    return {
+      __typename: "SuunnitteluSopimus",
+      ...suunnitteluSopimus,
+      logo: fileService.getYllapitoPathForProjektiFile(oid, suunnitteluSopimus.logo),
+    };
   }
   return suunnitteluSopimus as undefined | null;
 }
@@ -193,14 +198,14 @@ function removeUndefinedFields(object: API.Projekti): Partial<API.Projekti> {
   return pickBy(object, (value) => value !== undefined);
 }
 
-function adaptYhteystiedot(yhteystiedot: Yhteystieto[]): API.Yhteystieto[] {
+export function adaptYhteystiedot(yhteystiedot: Yhteystieto[]): API.Yhteystieto[] {
   if (yhteystiedot) {
     return yhteystiedot.map((yt) => ({ __typename: "Yhteystieto", ...yt }));
   }
   return [];
 }
 
-function adaptJulkaisuPDFPaths(
+export function adaptJulkaisuPDFPaths(
   oid: string,
   aloitusKuulutusPDFS: LocalizedMap<AloitusKuulutusPDF>
 ): AloitusKuulutusPDFt | undefined {
@@ -224,15 +229,15 @@ function adaptJulkaisuPDFPaths(
   return { __typename: "AloitusKuulutusPDFt", SUOMI: result[Kieli.SUOMI], ...result };
 }
 
-function adaptHankkeenKuvaus(hankkeenKuvaus: LocalizedMap<string>): HankkeenKuvaukset {
+export function adaptHankkeenKuvaus(hankkeenKuvaus: LocalizedMap<string>): HankkeenKuvaukset {
   return {
     __typename: "HankkeenKuvaukset",
     SUOMI: hankkeenKuvaus.SUOMI,
     ...hankkeenKuvaus,
   };
 }
 
-function adaptAloitusKuulutusJulkaisut(
+export function adaptAloitusKuulutusJulkaisut(
   oid: string,
   aloitusKuulutusJulkaisut?: AloitusKuulutusJulkaisu[] | null
 ): API.AloitusKuulutusJulkaisu[] | undefined {
@@ -246,7 +251,7 @@ function adaptAloitusKuulutusJulkaisut(
         hankkeenKuvaus: adaptHankkeenKuvaus(julkaisu.hankkeenKuvaus),
         yhteystiedot: adaptYhteystiedot(yhteystiedot),
         velho: adaptVelho(velho),
-        suunnitteluSopimus: adaptSuunnitteluSopimus(suunnitteluSopimus),
+        suunnitteluSopimus: adaptSuunnitteluSopimus(oid, suunnitteluSopimus),
         kielitiedot: adaptKielitiedot(kielitiedot),
         aloituskuulutusPDFt: adaptJulkaisuPDFPaths(oid, julkaisu.aloituskuulutusPDFt),
       };
@@ -255,7 +260,7 @@ function adaptAloitusKuulutusJulkaisut(
   return undefined;
 }
 
-function adaptVelho(velho: Velho): API.Velho {
+export function adaptVelho(velho: Velho): API.Velho {
   return { __typename: "Velho", ...velho };
 }