Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Tukipyyntö] Laskutusliitännäinen: Tarvitaan tieto laskutettujen niteiden määrästä ja laskutetun aineiston kokonaissumma #2

Closed
piiaseme opened this issue Oct 18, 2022 · 20 comments
Assignees
Labels
local enhancement New feature or request

Comments

@piiaseme
Copy link

Tukipyyntö

Aiemmin käytetty Kohan laskutustyökalu antoi laskutetuista kokonaissumman sekä laskutettujen niteiden määrän. Laskutusliitännäinen ei tuota tietoa anna, joten olisiko mahdollista saada ominaisuus suoraan laskutusliitännäiseen? Jos se ei onnistu, niin saisiko valmiit SQL-loitsut/raportit, jolla nuo tiedot saa?

Nämä tiedot tarvitaan täsmäytystä ja tilastointia varten.

Laskujen lukumäärä näkyy laskutusliitännäisessä, mutta ei niiden yhteenlaskettu summa eikä laskutetun aineiston määrä.

Jos on mahdollista saada nuo tiedot irti suoraan liitännäisestä, mutta sen työstämisessä voi mennä aikaa, niin saammeko siksi aikaa käytettäväksi raportin :)

Kohan osio

Laskutus

Jotain muuta?

No response

@piiaseme piiaseme added the help wanted Extra attention is needed label Oct 18, 2022
@piiaseme piiaseme self-assigned this Oct 18, 2022
@piiaseme piiaseme changed the title Laskutusliitännäinen: Tarvitaan tieto laskutettujen niteiden määrästä ja laskutetun aineiston kokonaissumma [Tukipyyntö] Laskutusliitännäinen: Tarvitaan tieto laskutettujen niteiden määrästä ja laskutetun aineiston kokonaissumma Oct 18, 2022
@piiaseme piiaseme assigned piiaseme and pirliis and unassigned piiaseme and pirliis Nov 7, 2022
@CodoDerDritte CodoDerDritte transferred this issue from KohaSuomi/Koha Nov 8, 2022
@lmstrand
Copy link

Tässä raportti lisättäväksi Kohaan:

select 
ExtractValue(content, 'Finvoice/MessageTransmissionDetails/MessageReceiverDetails/ToIntermediator') as 'Asiakas-ID',
ExtractValue(content, 'Finvoice/InvoiceDetails/InvoiceTotalVatIncludedAmount') as 'laskutettu yhteensä',
ExtractValue(content, 'Finvoice/InvoiceRow/ArticleName') as 'niteet',
ExtractValue(content, 'count(Finvoice/InvoiceRow/ArticleName)') as 'niteiden lkm'

from message_queue where message_transport_type = 'finvoice'
and (CAST(time_queued as date) between <<Alkupvm|date>> and <<Loppupvm|date>>)

Raportti ei laske yhteenlaskettuja summia, summien laskemisessa olevaa ongelmaa tutkitaan. Käyttäkää sillä välin sormia, laskimia, helmitaulua ja exceliä apuna, kunnes löydetään ratkaisu miten ne saisi mukaan raporttiin.

@emta001
Copy link
Contributor

emta001 commented Nov 14, 2022

Siitä miten laskutettujen yhteissumma ja niteiden määrä näkyisi liitännäisessä, oliko ajatus siis se, että kun painetaan "Näytä laskutetut" niin nämä näkyisivät esimerkiksi listan alussa (Laskutettu yhteensä kohdan jälkeen)?

@emta001
Copy link
Contributor

emta001 commented Nov 14, 2022

Tässä SQL-rapsa, jolla saa myös yhteenlasketun summan oikein:

select  
ExtractValue(content, 'Finvoice/MessageTransmissionDetails/MessageReceiverDetails/ToIntermediator') as 'Asiakas-ID', 
ExtractValue(content, 'Finvoice/InvoiceDetails/InvoiceTotalVatIncludedAmount') as 'laskutettu yhteensä', ExtractValue(content, 'Finvoice/InvoiceRow/ArticleName') as 'niteet', 
ExtractValue(content, 'count(Finvoice/InvoiceRow/ArticleName)') as 'niteiden lkm'  
from message_queue 
where message_transport_type = 'finvoice'  
UNION ALL 
select 'Total', 
SUM(CONVERT(REPLACE(ExtractValue(content, 'Finvoice/InvoiceDetails/InvoiceTotalVatIncludedAmount'), ',', '.' ), DECIMAL(16,2))), 
NULL, 
SUM(convert((ExtractValue(content, 'count(Finvoice/InvoiceRow/ArticleName)')), SIGNED)) from message_queue where message_transport_type = 'finvoice';

Sen verran pitää vielä säätää, että saadaan päivämäärällä rajaus toimimaan.

Liitännäiseen noiden arvojen saamiseen voi tosiaan mennä hetki, ensimmäisenä olevan laskun arvot menee ihan ok, mutta yhteenlasku ei ihan toimi vielä.

@emta001
Copy link
Contributor

emta001 commented Nov 14, 2022

Raportti löytyy OUTIn testiltä, id 526.

@emta001
Copy link
Contributor

emta001 commented Nov 16, 2022

Testillä on nyt kokeiltavana noiden laskutettujen kokonaissumman sekä laskutettujen niteiden määrän tarkastelu. Ne näkyy kun menee "Näytä lasketut" näkymään, "Tulokset" kohdan jälkeen. Vielä pitää hieman kai säätää, plugari herjaa nyt puuttuvista price -arvoista. Mutta muuten tuon pitäisi toimia kuten pitääkin.

@piiaseme
Copy link
Author

Laskutettujen määrä ja yhteenlaskettu summa täsmäävät. Korvaushinnan puute ei näytä vaikuttavan yhteenlaskettujen summaan eikä laskutettujen niteiden määrään.

Raportti 526: en ihan hoksaa kuin saan sillä oikean tuloksen, mutta johtunee raportin yhteenlaskun toimimattomuudesta?

@emta001
Copy link
Contributor

emta001 commented Nov 16, 2022

Laskutettujen määrä ja yhteenlaskettu summa täsmäävät. Korvaushinnan puute ei näytä vaikuttavan yhteenlaskettujen summaan eikä laskutettujen niteiden määrään.

Juu se laskee korvaushinnat vaikka ne ei jostain syystä näkyisikään muuten. MUtta niiden puute aiheuttaa nyt josatin syystä sen, että selaimen konsolissa näkyy virhe "price.replace is not a function". Tuo ei kuitenkaan näyttäisi vaikuttavan liitännäisen toimintaan muuten, mutta pitänee sekin yritää korjata.

Raportti 526: en ihan hoksaa kuin saan sillä oikean tuloksen, mutta johtunee raportin yhteenlaskun toimimattomuudesta?

Hmm, sen pitäisi toimia samaan tapaan kuin liitännäisen eli se hakee tietylle aikavälille laskutetut. Mutta enpä saa sieltä nyt itsekään mitään ulos.

@emta001
Copy link
Contributor

emta001 commented Nov 16, 2022

Aaa niin aivan, raportti perustuu siihen milloin laskutettu aineisto on lisätty message_queue-tauluun, ei laskutettujen niteiden eräpäiviin kuten liitännäisessä. OUTIn testillä kun rapsaan laittaa aikaväliksi esim. 1.10.-31.10.2022 niin pitäisi löytyä aineistoa.

@piiaseme
Copy link
Author

Okei kiitti, testaan uudelleen :)

@emta001
Copy link
Contributor

emta001 commented Nov 16, 2022

Nyt ei enää herjaa ja korvaushintojen pitäisi näkyä kaikkien niteiden kohdalla oikein.

@piiaseme
Copy link
Author

Laskutettujen niteiden korvaushinnat näkyvät nyt testin laskutusliitännäisessä. Eivät näkyneet ensin, mutta Ctrl+F5 auttoi siinä.

@piiaseme
Copy link
Author

Raportti toimii myös (nyt kun osasin sitä käyttää oikein..).

@piiaseme
Copy link
Author

Saadaanko liitännäisen muutos tuotantoon ensi viikolla vai tarviiko vielä testata? Raportin voin itse siirtää.

@emta001
Copy link
Contributor

emta001 commented Nov 18, 2022

Miepä hoksautan Pääkäyttäjät- kanavalla, josko joku kerkeis vielä testaamaan tänään tai maanantaina.

@CodoDerDritte CodoDerDritte added local enhancement New feature or request and removed help wanted Extra attention is needed labels Nov 21, 2022
@piiaseme
Copy link
Author

Hei, kopioin ja siirsin raportin #526 OUTIn testiltä tuotantoon. Jostain syystä en saa järkevää tulosta, kun ajan raportin. Onko siihen jokin jippo, jota en ole hoksannut?

Laskutusliitännäinen laskee ainoan tehdyn laskutuserän oikein. Ensimmäisessä tuotannon testierässä lähetettiin viisi laskua, Nyt liitännäinen hakee neljä laskua, sillä yksi niistä on jo hoidettu (maksettu tai nide on palautettu). Eli mielestäni se toimii oikein tuotannossakin.

@emta001
Copy link
Contributor

emta001 commented Nov 22, 2022

Hei, kopioin ja siirsin raportin #526 OUTIn testiltä tuotantoon. Jostain syystä en saa järkevää tulosta, kun ajan raportin. Onko siihen jokin jippo, jota en ole hoksannut?

Ei pitäisi olla jippoa, millä tavalla se tulos ei oo järkevä? Ainakin itse sain ajalle 1.10.-31.10. tulokseksi 10 riviä, niteiden yhteenlaskettu määrä 50, laskutetun aineiston kokonaissumma 1457.40.

@piiaseme
Copy link
Author

piiaseme commented Nov 22, 2022

Pirkko-Liisa huomasi, että siinä on rivit (=laskutetut) tuplana. Eli esim. asiakkaan 26459 lasku on raportin tuloksessa kahteen kertaan vaikka pitäisi olla yhdesti.

Kaivelin sähköposteja ja löysin viestiketjun, jossa ratkottiin ensimmäisen lähetyksen ongelmia eli onko voinut käydä niin, että laskut on lähteneet kahteen kertaan tuotanto-Kohasta eteenpäin, mutta ajona asiakkaille vain sen yhden kerran.

En ainakaan huomannut mitään tuplaan viittaavaa raportin koodista.

@emta001
Copy link
Contributor

emta001 commented Nov 23, 2022

Kannassa on tosiaan tuplana nuo laskut. Esimerkiksi asiakkaalle 26459 on mennyt ilmeisesti sama lasku 2022-10-17 12:40:02 ja uudestaan 2022-10-17 14:10:24. Eli rpitäisikö raportista saada metsästettyä nämä tuplat pois?

@piiaseme
Copy link
Author

Voiko se tuplaantuminen johtua siitä, että ne laskut on oikeasti lähteneet meiltä kahdesti, kun tuotanossa lähettämisessä oli pientä ongelmaa vai onko vika jossain muualla?

Oulun laskuttajat ovat jo saaneet ensimmäisen Finnvoice-laskutuserän tiedot ilmoitettua kaupungille, joten ei ehkä tarvitse käyttää siihen aikaa, että metsästät ne sieltä pois?

Olisiko viisainta katsoa seuraava laskutuserä raportilla, kun se laitetaan tuotannossa liikkeelle, sillä on varmaan myös pieni määrältään. Ja jos tuo tuplaantuminen toistuu silloinkin, niin sitten tutkitaan tarkemmin mistä tuplaantuminen raportilla johtuu?

Tämmöstä ehdottaisin, onko järkevä ajatus?

@emta001
Copy link
Contributor

emta001 commented Nov 24, 2022

Juu kuulostaa järkevälle. Avataan sitten myös asiasta uusi tiketti.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
local enhancement New feature or request
Projects
Status: Suljettu
Development

No branches or pull requests

5 participants