MayaLabs Discussion Area

Suomeksi (Finnish only) => MLInvoiceen liittyvä keskustelu => Topic started by: Mihail on 24.08.11 - klo:14:52

Title: Bugiraportit
Post by: Mihail on 24.08.11 - klo:14:52
Ehdotan kaikkien vikaraporttien kirjaamista tähän alueeseen.
Title: RE: Bugiraportit
Post by: Mihail on 24.08.11 - klo:14:54
Vllasku 1.3.2 versiolla käytettäessä 'Lisää viivästysmaksut' toimintoa laskurivien otsikot tulostuvat väärin. Viivästyskorko ja maksukehoitus vaihtavat paikkaa keskenään. Ainoastaan otsikot eli Nimike-kentät vaihtavat paikkaansa, tietokantaan tallennettu data on oikein, samoin laskurivien muut tiedot.
Title: RE: Bugiraportit
Post by: Mihail on 24.08.11 - klo:15:09
Korjausehdotuksena edelliseen:

invoice_printer_base.php rivi 398:

Alkuperäinen koodi
switch ($row['reminder_row'])
     {
     case 1:
       $description = $GLOBALS['locPDFReminderFeeDesc'];
       break;
     case 2:
       $description = $GLOBALS['locPDFPenaltyInterestDesc'];
       break;


Korjattu versio:
switch ($row['reminder_row'])
     {
     case 2:
       $description = $GLOBALS['locPDFReminderFeeDesc'];
       break;
     case 1:
       $description = $GLOBALS['locPDFPenaltyInterestDesc'];
       break;

Title: RE: Bugiraportit
Post by: Ere Maijala on 24.08.11 - klo:19:41
Parempi olisi tehdä vikaraporteista aina uusi viestiketju. Näin siihen saa kuvaavan otsikon, eikä mahdollinen keskustelu eri virheistä sekoitu yhteen soppaan. Jos bugiraportteja tulee paljon, voin tehdä niille oman keskustelualueen, mutta tähän mennessä siihen ei ole ollut tarvetta.

Kiitos bugiraportista, virhe taitaa olla ollut siellä siitä lähtien kun tuon tein...

Korjauksesta sen verran, että parempi tehdä se toisin päin, niin menevät numerojärjestyksessä eikä tartte ihmetellä tuota myöhemmin:


     switch ($row['reminder_row'])
     {
     case 1:
       $description = $GLOBALS['locPDFPenaltyInterestDesc'];
       break;
     case 2:
       $description = $GLOBALS['locPDFReminderFeeDesc'];
       break;