-
Notifications
You must be signed in to change notification settings - Fork 29
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
Kitsas kaatuu MacOS:llä uutta tositetta tallentaessa #1368
Comments
Kitsas versio 5.1.1 |
Toistuuko tuo myös versiolla 5.2-RC? (https://github.com/petriaarnio/kitupiikki/releases/tag/mac-v5.2-rc) |
Minulla Kitsas kattu vanhaa tositetta avattaessa samalla tavalla. Kitsas 5.1.2, MacOS 13.6, MacBook Pro 2023 M2. |
Ongelma toistui äsken myös versiolla 5.2-RC, muuten sama ympäristö. |
Ongelma ei toistunut Linuxissa (Ubuntu 22.04.3 LTS) Kitsas 5.1:llä. |
Palautteen kautta tulleessa raportissa tositteen valintaan kaatumisesta kaatuminen on tapahtunut samassa kohdassa kuin tässä tallennettaessa tapahtuneessa kaatumisessa. QTableview::currentChanged() kutsuminen vie libqcocoaan ja siellä näytettäisiin yritettävän lukea taulukosta arvoa virheellisellä indeksillä. libqcocoa on vain mac-koneilla oleva kirjasto. mikä selittänee sen että näitä ongelmia on esiintynyt vain mac-koneilla. (Villinä veikkauksena heittäisin että muissa käyttöjärjestelmissä käytetyssä koodissa indeksin oikeellisuus mahdollisesti tarkistettaisiin ennen käyttöä.) Nyt pitäisi vielä selvittää miksi QTableview::currentChanged() on saanut näissä tilanteissa virheellisen / virheelliseksi muuttuvan indeksin.... |
Jos tuosta kaatumisesta on jotain logia mahdollista ottaa talteen, voin sen toki tehdä. (Jos logi saattaa sisältää sensitiivistä tietoa, pitää toki vielä sopia yksityiskohdista.) |
Viiveellä vastaan mutta 5.2RC kaatuu yhtä lailla. Toimitan mielelläni logeja tai vastaavaa jos niistä on apua. |
Tapahtuuko kaatuminen kaikilla liitteillä, sekä uusien tallennuksessa että vanhoja tositteita avatessa? |
@petriaarnio Minulla Kitsas ei ole kaatunut uutta tositetta lisättäessa ainakaan niin usein, että enää sitä muistaisin. Jos ymmärsin oikein, @JJH86 :lla Kitsas taas on kaatunut nimenomaan uutta tositetta luotaessa. Voi siis olla, että kyseessä onkin kaksi eri ongelmaa. Vanhoja tositteita avatatessa Kitsas tuntuu minulla kaatuvan joka kerta, kun tositelaji on Tulo (ML), Meno (OL) tai Kululasku (KL). Toisaalta en saanut Kitsasta kaatumaan avattaessa tositetta, jonka tositelaji oli Muistio (MT), Palkka (P) tai Tiliote (T). Kokeilin kaikkia tositelajeja monta kertaa, ja vika näyttäisi olevan hyvin johdonmukainen. Minulla on kaikissa tositteissa PDF-liitetiedosto, joten sitä en pysty sanomaan, onko liitetiedostolla tai sen puuttumisella merkitystä ongelman kannalta. |
Kokeilin vielä lisää ja lisäsin yhden tulo- ja yhden menotositteen. Tositteen lisääminen onnistui ongelmitta, mutta Kitsas kaatui, kun tositteita tallentamisen jälkeen yritti avata. |
Onko tämän ongelman korjaatumisesta arviota? Nykyisellään Kitsaan Mac-versio ei ole käytettävä. |
Tässä on sellainen ongelma, että en ole saanut softaa kaatumaan omilla tositteilla. Olisiko mahdollista saada sämppeleitä tositteista, joilla kaatuminen tapahtuu? |
Minulla Kitsas kaatuu samalla tavalla kuin @anttipalsola kuvasi: lähes mikä tahansa tosite, jossa on PDF-liite ja tositelajina Meno tai Kululasku, kaataa ohjelman tätä avatessa. Sama ongelma toistuu sekä versiossa 5.1.2 että 5.2-RC ja käyttiksenä on mac OS Sonoma 14.1.2 (23B92), laitteena MBP M1 vm 2021. Tein kuitenkin vielä merkittävän lisähavainnon, josta voi olla apua: jos PDF-liitteen tallentaa ilman vientejä, bugia ei esiinny. Eli tein kokeeksi täysin samasta PDF-tiedostosta kaksi Meno-tyyppistä tositetta, jossa toiseen lisätään joku eurosumma ja menotili ja toisessa tosite tallennetaan ilman vientejä. Viennillisen tositteen klikkaaminen kaataa sovelluksen, viennittömän ei.
@petriaarnio en usko että kyseessä on mikään kyseisiin PDF-liitteisiin liityvä piirre. Ainakin minulla on kirjanpidossa satoja PDF-tositteita kymmenistä eri lähteistä useilla eri tilikausilla, ja yhtäkkiä näistä minkä tahansa (Meno- tai Kululasku-tyyppisen) tositteen avaaminen kaataa Kitsaan. |
@petriaarnio laitetaan nyt kuitenkin vielä esimerkin vuoksi jakoon vaikka Pagesilla luotu tyhjä PDF-tiedosto: Tuo liite kaataa minulla sekä 5.1.2 että 5.2-RC:n tositetta avatessa, jos se on liitteenä Meno-tyyppisellä tositteella, jolla on summa ja menotili, mutta ei kaada silloin jos tositteella ei ole vientiä. Bugi koskee kaikkia muitakin PDF:iä millä kokeilin. |
Olen nyt testannut softaa sekä vanhemmalla Intel- että M1-macillä enkä saa ongelmaa näkyviin. |
Minulla saattaisi olla lähipäivinä aikaa tutkia tätä syvemmin, mutta pitää kyllä aloittaa ihan kehitysympäristön pystyttämisestä, kun edellisestä Kitsas-PR:stä alkaa olla aikaa. Oma lehmä ojassa, kun pitäisi saada kirjanpitoa tehtyä mutta softa kaatuilee. |
Voin vahvistaa @rjoonas 'n havainnon: Kun lisäsin tulotositteen (myyntilasku) PDF-liitteellä mutta ilman vientejä, tositteen pystyi avaamaan ilman, että Kitsas olisi kaatunut. Kun tositteeseen lisäsi viennin, Kitsas kaatui, kun tositteen seuraavan kerran avasi. Minäkin olen välissä päivittänyt käyttöjärjestelmän MacOS 14.2.1 (23C71):een, mutta tämä ei vaikuttanut ongelmaan suuntaan eikä toiseen. Ja koneena on siis ollut kokoajan MacBook Pro 14" 2023 M2 Pro. @petriaarnio : Voisiko olla, että ongelma esiintyisi vain jollain tietyllä Kitsas-konfiguraatiolla, etkä siksi ole pystynyt sitä toistamaan? Minä teen kirjanpidot kahdelle yhdistykselle, ja ongelma esiintyy molemmilla. Alla on joka tapauksessa (ei-täydellinen) kuvaus käyttämistäni asetuksista molemmille yhdistyksille erikseen: Kirjanpitovelvollinen 1
Kirjanpitovelvollinen 2
Käytännössä ero on siis lähinnä siinä, että ensimmäisellä yhdistyksellä on tositesarjat käytössä, toisella ei. Saatko noilla asetuksilla toistettua? Minäkin voisin periaatteessa kokeilla kehitysympäristön pystyttämistä, vaikka oma C++-osaamiseni onkin kyllä Hello world -tasoa. |
Todennäköisesti tosiaan liittyy konffiin tms. Olisiko mahdollista tehdä uusi näennäisfirman kirjanpito ja siihen vienti joka kaataa softan, ja sitten lähettää se minulle? |
Asensin nyt QT Creatorin ja toistin crashin debuggerissa. Se pysäyttää tulomenoapuri.cpp konstruktoriin, tälle riville: kitupiikki/kitsas/apuri/tulomenoapuri.cpp Line 70 in e0d8270
missä kohtaa call stack näyttää tältä:
Ja outputiin tulee Cocoasta poikkeus että QComboBox yrittää käpistellä tyhjän taulun ekaa alkiota:
|
Näillä ohjeilla rytisee ainakin minulla:
|
Eipä Testiyhdistyskään saanut Kitsasta kaatumaan minun koneillani. Myöskään sellaisella koneella, jossa ei ole ulkoisia riippuvuuksia (Qt, ...) asennettuna. Viittaukset alkuperäisiin sijaintipaikkoihin ovat olleet yleisin syy kaatumisiin. |
@petriaarnio Onko mahdollista, että minulla on asennettuna jostain kirjastosta jokin versio, joka ei ole Kitsaan kanssa yhteensopiva. Qt:ta ei pitäisi olla, mutta Homebrew'n kautta kyllä kaikenlaista muuta. |
Kitsaalla ei ole enää (ainakaan suoria) muita kirjastoriippuvuuksia kuin Qt, ja joka tapauksessa kaikki tarvittavat kirjastot on paketoitu applikaation bundleen ja viittaukset niihin sen sisällä, eli en kovin todennäköisenä pidä vääriä lokaaleja kirjastoversioita. |
Pyörin debuggerissa lisää ja tulin siihen tulokseen, että kyseessä taitaa olla Qt:n Mac-komponenttien bugi, joka saattaisi vieläpä olla juuri korjattu – katsokaapa tämä pari viikkoa vanha Qt:n commit, jossa minulla Kitsaan kaatavan kutsun Ongelma on ilmeisesti syntynyt 12.4.2023 commitissa qt/qtbase@52c2b82 ja tosiaan korjattu nyt joulukuussa. Qt 6.6.2, jossa tuo korjaus on mukana, julkaistaan ilmeisesti ihan lähiviikkoina. Kokeilin buildata Kitsaan myös Qt 6.7.0-beta 1:lla, mutta siinä ei valitettavasti ollutkaan vielä tuota committia mukana. |
Vaihdoin Kitsaan 5.3 versioon Qt 6.6.1 koska en huomannut kommenttia 6.6.2:sta. No uusi yritys kun 6.6.2 julkaistaan. |
Tämänhetkisen arvion mukaan Qt 6.6.2 on määrä julkaista 7.2.2024. |
Kuulostaa hyvältä. Tuohon mennessä meiltä pitäisi olla ulkona seuraava päivitys, jossa muutaman korjauksen lisäksi pääsee paremmin tekemään Kitsaan laskelmasta poikkeavan alijäämähuojennuksen. |
Buildasin Kitsaan Qt 6.7.0-beta2:lla, joka julkaistiin toissapäivänä. Sillä sain itselleni jo kasaan Kitsaan, jolla PDF-tositteet aukeavat kaatumatta 👍 |
7.2. meni eikä Qt 6.6.2 näytä vieläkään olevan saatavilla. |
@petriaarnio Varmaankin se kuitenkin ilmestyy lähipäivinä, koska eivät myöskään ole enää siirtäneet "Updated plan" -päivää eteenpäin. Kitsas 5.3 muuten kaatuilee aivan koko ajan, mm. seuravissa tilanteissa:
Kaatuilu ei ole täysin johdonmukaista: välillä esim. tositteiden selausikkunan avaaminen onnistuu, välillä ei. Viimeistään kaatuminen tapahtuu kuitenkin 4–5 klikkauksen jälkeen, joten nykyisellään Kitsas ei ainaakn minulla ole Macbookissa käytettävä. Oletettavasti kyse on kuitenkin samasta ongelmasta, joka Qt 6.6.2 korjaa, joten en tee näistä erillistä bugiraporttia ainakaan vielä. Käyttämäni käyttöjärjestelmäversio on nykyisin MacOS Sonoma 14.3 (23D56). |
Qt 6.6.2 julkaisu on siirtynyt taas eteenpäin. Uusi arvio on 15.2.2024. |
Qt 6.6.2 on julkaistu tänään. @petriaarnio, milloin saisitte ulos Kitsaan version, jossa tuo olisi mukana? (Ja jos @rjoonas ehtii testaamaan, niin aina parempi.) |
Teen uuden version tämän viikon lauantaina. |
Nyt olisi versio 5.4.1 Qt 6.6.2:lla tarjolla |
Latasin version 5.4.1 enkä enää saanut ohjelmaa kaatumaan, vaikka parhaani yritin. :-) Näyttää siis siltä, että ongelma korjaantui. Kiitos @rjoonas 'lle perimmäisen syyn selvittämisestä ja testaamisesta sekä @petriaarnio lle ongelman korjaamisesta. Minun puolestani tämän tiketin voi sulkea. |
Versio 5.5 kaatuu tositetta tallennettaessa, CSV-tuotua tiliotetta läpi käydessä ja tilitapahtumia kohdistaessa. Consolesta crash report tallessa, jos sieltä jotain hyödyllistä pystyy antamaan. Olen tästä bugista kärsinyt jo useiden versioiden ajan. Jätän tuoreimman kaatumisen tuottamat käyttökokemukset nyt tästä tarkemmin selostamatta. edit. kaatui uudestaan 25 min myöhemmin |
Ohjelma tallentaa tiedot oikein mutta kaatuu samalla. Toistettu kahdella eri koneella ja erillisellä asennuksella/kirjanpidolla.
Liitteenä virhe:
kitsas_error.log
The text was updated successfully, but these errors were encountered: