Merge remote-tracking branch 'upstream/devel' into devel

petriaarnio · May 7, 2024 · 61623b8 · 61623b8
2 parents 0f3ad65 + 9150b5e
commit 61623b8
Show file tree

Hide file tree

Showing 79 changed files with 740 additions and 5,827 deletions.
diff --git a/kitsas/alv/alvkaudet.cpp b/kitsas/alv/alvkaudet.cpp
@@ -15,6 +15,8 @@ void AlvKaudet::haeKaudet()
 {
     kaudet_.clear();
     errorCode_.clear();
+    varmenneTila = EIKAYTOSSA;
+
     QDate pvm = kp()->tilitpaatetty();
     while( pvm.year() <= kp()->paivamaara().year() ) {
         haussa_++;

diff --git a/kitsas/alv/varmennedialog.cpp b/kitsas/alv/varmennedialog.cpp
@@ -4,10 +4,7 @@
 #include <QRegularExpressionValidator>
 #include <QPushButton>
 
-#include "toimisto/groupdata.h"
 #include "db/kirjanpito.h"
-#include "db/asetusmodel.h"
-#include "pilvi/pilvimodel.h"
 
 VarmenneDialog::VarmenneDialog(QWidget *parent) :
     QDialog(parent),
@@ -28,15 +25,6 @@ VarmenneDialog::~VarmenneDialog()
     delete ui;
 }
 
-bool VarmenneDialog::toimistoVarmenne(GroupData *group)
-{
-    connect( ui->buttonBox, &QDialogButtonBox::helpRequested, [] { kp()->ohje("toimisto/varmenne/"); });
-    if( exec() == QDialog::Accepted) {
-        group->lisaaVarmenne(ui->tunnusEdit->text(), ui->salaEdit->text());
-        return true;
-    }
-    return false;
-}
 
 int VarmenneDialog::pilviVarmenne()
 {

diff --git a/kitsas/alv/varmennedialog.h b/kitsas/alv/varmennedialog.h
@@ -17,7 +17,6 @@ class VarmenneDialog : public QDialog
     explicit VarmenneDialog(QWidget *parent = nullptr);
     ~VarmenneDialog();
 
-    bool toimistoVarmenne(GroupData* group);
     int pilviVarmenne();
 
     QString tunnus() const;

diff --git a/kitsas/apuri/apuririvi.cpp b/kitsas/apuri/apuririvi.cpp
@@ -28,7 +28,7 @@ ApuriRivi::ApuriRivi(int tili)
         bool alv = kp()->asetukset()->onko(AsetusModel::AlvVelvollinen);
         if( tilini ) {
             alvkoodi_ = alv ? tilini->luku("alvlaji") : AlvKoodi::EIALV;
-            veroprosentti_ = alv ? tilini->luku("alvprosentti") : 0;
+            veroprosentti_ = alv ? tilini->alvprosentti() : 0;
             poistoaika_ = tilini->luku("menojaannospoisto");
             kohdennus_ = tilini->luku("kohdennus");
         }
@@ -71,14 +71,30 @@ ApuriRivi::ApuriRivi(const TositeVienti &vienti, bool plusOnKredit)
 
 }
 
+int ApuriRivi::alvkoodi() const
+{
+    return alvkoodi_;
+}
+
 void ApuriRivi::setEra(const EraMap &era)
 {
     era_ = era;
 }
 
 void ApuriRivi::setAlvkoodi(int koodi)
 {
-    alvkoodi_ = koodi;
+    if( kp()->asetukset()->onko(AsetusModel::AlvVelvollinen) )
+        alvkoodi_ = koodi;
+    else
+        alvkoodi_ = AlvKoodi::EIALV;
+}
+
+void ApuriRivi::setAlvprosentti(double prosentti)
+{
+    if(kp()->asetukset()->onko(AsetusModel::AlvVelvollinen))
+        veroprosentti_ = prosentti;
+    else
+        veroprosentti_ = 0.0;
 }
 
 void ApuriRivi::setAlvvahennys(bool vahennys, int vahennysVientiId)

diff --git a/kitsas/apuri/apuririvi.h b/kitsas/apuri/apuririvi.h
@@ -37,7 +37,7 @@ class ApuriRivi
     ApuriRivi(const TositeVienti& vienti, bool plusOnKredit = true);
 
     int tilinumero() const { return tilinumero_; }
-    int alvkoodi() const { return alvkoodi_; }
+    int alvkoodi() const;
     double alvprosentti() const { return veroprosentti_; }
     bool alvvahennys() const { return alvvahennys_; }
     QString selite() const { return selite_;}
@@ -56,7 +56,7 @@ class ApuriRivi
 
     void setTili(int numero) { tilinumero_=numero;}
     void setAlvkoodi(int koodi);
-    void setAlvprosentti(double prosentti) { veroprosentti_ = prosentti;}
+    void setAlvprosentti(double prosentti);
     void setAlvvahennys(bool vahennys, int vahennysvientiId = 0);
     void setSelite(const QString& selite) { selite_ = selite;}
     void setKohdennus(int kohdennus) { kohdennus_ = kohdennus;}

diff --git a/kitsas/apuri/tiliote/tiliotekirjaaja.cpp b/kitsas/apuri/tiliote/tiliotekirjaaja.cpp
@@ -44,6 +44,7 @@ TilioteKirjaaja::TilioteKirjaaja(TilioteApuri *apuri) :
 
     resize(800,600);
     restoreGeometry( kp()->settings()->value("TilioteKirjaaja").toByteArray());
+    ui->viennitView->setColumnHidden(ApuriRivit::ALV, !kp()->asetukset()->onko(AsetusModel::AlvVelvollinen));
 
 }
 
@@ -179,7 +180,7 @@ void TilioteKirjaaja::alaTabMuuttui(int tab)
     } else if ( tab == SIIRTO ) {
         ui->tiliLabel->setText( menoa_ ? tr("Tilille") : tr("Tililtä")  );
         ui->asiakasLabel->setText( menoa_ ? tr("Saaja") : tr("Maksaja"));
-        ui->tiliEdit->suodataTyypilla( ".*");
+        ui->tiliEdit->suodataTyypilla( "[ABCD].*");
 
     } else if( tab == VAKIOVIITE) {
         ui->maksuView->setModel( kp()->vakioViitteet() );
@@ -189,7 +190,7 @@ void TilioteKirjaaja::alaTabMuuttui(int tab)
     tiliMuuttuu();
     ui->viennitView->horizontalHeader()->setSectionResizeMode(TilioteViennit::TILI, QHeaderView::Stretch);
 
-    bool const voiLisataVienteja = tab != MAKSU && ui->alaTabs->currentIndex() != VAKIOVIITE && tab != MAKSU;
+    bool const voiLisataVienteja = tab != MAKSU && ui->alaTabs->currentIndex() != VAKIOVIITE;
     ui->lisaaVientiNappi->setVisible( voiLisataVienteja );
     ui->poistaVientiNappi->setVisible( voiLisataVienteja && rivit_->rowCount() > 1);
     ui->viennitView->setVisible( voiLisataVienteja && rivit_->rowCount() > 1);
@@ -223,6 +224,7 @@ void TilioteKirjaaja::verotonMuuttuu()
 
 void TilioteKirjaaja::alvMuuttuu()
 {
+
    int alvkoodi = ui->alvCombo->currentData( VerotyyppiModel::KoodiRooli ).toInt();
    if( !ladataan_) {
         rivi()->setAlvkoodi( alvkoodi );
@@ -280,19 +282,19 @@ void TilioteKirjaaja::ylaTabMuuttui(int tab)
    menoa_ = (tab == TILILTA);
 
    if( menoa_ ) {
-        ui->alaTabs->setTabText(MAKSU, tr("Maksettu lasku"));
+        ui->alaTabs->setTabText(MAKSU, tr("Maksett&u lasku"));
         ui->alaTabs->setTabIcon(TULOMENO, QIcon(":/pic/poista.png") ) ;
         ui->alaTabs->setTabIcon(SIIRTO, QIcon(":/pic/tililta.png"));
-        ui->alaTabs->setTabText(TULOMENO, tr("Meno"));
+        ui->alaTabs->setTabText(TULOMENO, tr("&Meno"));
         if( ui->alaTabs->count() > VAKIOVIITE)
             ui->alaTabs->removeTab(VAKIOVIITE);
     } else {
-        ui->alaTabs->setTabText(MAKSU, tr("Saapuva maksu"));
+        ui->alaTabs->setTabText(MAKSU, tr("Saap&uva maksu"));
         ui->alaTabs->setTabIcon(TULOMENO, QIcon(":/pic/lisaa.png") ) ;
         ui->alaTabs->setTabIcon(SIIRTO, QIcon(":/pic/tilille.png"));
-        ui->alaTabs->setTabText(TULOMENO, tr("Tulo"));
+        ui->alaTabs->setTabText(TULOMENO, tr("&Tulo"));
         if( ui->alaTabs->count() == VAKIOVIITE )
-            ui->alaTabs->addTab(QIcon(":/pic/viivakoodi.png"), tr("Vakioviite"));
+            ui->alaTabs->addTab(QIcon(":/pic/viivakoodi.png"), tr("&Vakioviite"));
     }
 
     alaTabMuuttui( ui->alaTabs->currentIndex() );
@@ -466,12 +468,14 @@ void TilioteKirjaaja::lisaaVienti()
     const int uusiRivi = rivit_->lisaaRivi( oletustili() );
     ui->viennitView->setVisible(true);
     ui->viennitView->selectRow(uusiRivi);
+    ui->poistaVientiNappi->setVisible( true );
     QTimer::singleShot(50, this, &TilioteKirjaaja::tiliMuuttuu);
 }
 
 void TilioteKirjaaja::poistaVienti()
 {
     ui->viennitView->setVisible( rivit_->rowCount() > 2 );
+    ui->poistaVientiNappi->setVisible( rivit_->rowCount() > 2);
     rivit_->poistaRivi(nykyAliRiviIndeksi_);
     ui->viennitView->selectRow( qMin(nykyAliRiviIndeksi_, rivit_->rowCount()-1) );
 }
@@ -483,12 +487,12 @@ void TilioteKirjaaja::alusta()
     ui->setupUi(this);
     ui->viennitView->setModel(rivit_);
 
-    ui->ylaTab->addTab(QIcon(":/pic/lisaa.png"), tr("Tilille"));
-    ui->ylaTab->addTab(QIcon(":/pic/poista.png"), tr("Tililtä"));
+    ui->ylaTab->addTab(QIcon(":/pic/lisaa.png"), tr("T&ilille"));
+    ui->ylaTab->addTab(QIcon(":/pic/poista.png"), tr("Tililt&ä"));
 
-    ui->alaTabs->addTab(QIcon(":/pic/lasku.png"), tr("Laskun maksu"));
-    ui->alaTabs->addTab(QIcon(":/pic/lisaa.png"), tr("Tulo"));
-    ui->alaTabs->addTab(QIcon(":/pic/tilille.png"), tr("Siirto"));
+    ui->alaTabs->addTab(QIcon(":/pic/lasku.png"), tr("Lask&un maksu"));
+    ui->alaTabs->addTab(QIcon(":/pic/lisaa.png"), tr("&Tulo"));
+    ui->alaTabs->addTab(QIcon(":/pic/tilille.png"), tr("&Siirto"));
 
 
     veroFiltteri_ = new QSortFilterProxyModel(this);
@@ -704,10 +708,14 @@ void TilioteKirjaaja::tallennaRivi()
         return;
 
     ApuriRivi* aliRivi = rivit_->rivi(rivi);
+    Tili tili = ui->tiliEdit->valittuTili();
 
     aliRivi->setTili( ui->tiliEdit->valittuTilinumero());
     aliRivi->setKohdennus( ui->kohdennusCombo->kohdennus());
-    aliRivi->setEra(ui->eraCombo->eraMap());
+    if( tili.eritellaankoTase())
+        aliRivi->setEra(ui->eraCombo->eraMap());
+    else
+        aliRivi->setEra(EraMap());
     aliRivi->setMerkkaukset( ui->merkkausCC->selectedDatas());
     aliRivi->setJaksoalkaa( ui->jaksoAlkaaEdit->date());
     aliRivi->setJaksopaattyy( ui->jaksoLoppuuEdit->date());
@@ -745,6 +753,9 @@ TilioteKirjausRivi TilioteKirjaaja::tallennettava()
     rivi.asetaKumppani( ui->asiakastoimittaja->map());
     rivi.asetaTyyppi(tyyppi());
     rivit_->asetaTyyppi(tyyppi(), !menoa_ );
+    if( rivit_->rowCount() == 1) {
+        rivi.asetaOtsikko( ui->seliteEdit->toPlainText() );
+    }
 
     if( ui->alaTabs->currentIndex() == MAKSU) {
         QModelIndex index = ui->maksuView->currentIndex();

diff --git a/kitsas/apuri/tiliote/tiliotekirjaaja.ui b/kitsas/apuri/tiliote/tiliotekirjaaja.ui
@@ -56,7 +56,7 @@
        <item row="0" column="0">
         <widget class="QLabel" name="label">
          <property name="text">
-          <string>Päivämäärä</string>
+          <string>&amp;Päivämäärä</string>
          </property>
          <property name="buddy">
           <cstring>pvmEdit</cstring>
@@ -79,7 +79,7 @@
        <item row="4" column="0">
         <widget class="QLabel" name="euroLabel">
          <property name="text">
-          <string>Euroa</string>
+          <string>&amp;Euroa</string>
          </property>
          <property name="buddy">
           <cstring>euroEdit</cstring>
@@ -135,7 +135,10 @@
        <item row="12" column="0">
         <widget class="QLabel" name="seliteLabel">
          <property name="text">
-          <string>Selite</string>
+          <string>Se&amp;lite</string>
+         </property>
+         <property name="buddy">
+          <cstring>seliteEdit</cstring>
          </property>
         </widget>
        </item>
@@ -158,7 +161,10 @@
        <item row="13" column="0">
         <widget class="QLabel" name="jaksotusLabel">
          <property name="text">
-          <string>Jaksotus</string>
+          <string>&amp;Jaksotus</string>
+         </property>
+         <property name="buddy">
+          <cstring>jaksoAlkaaEdit</cstring>
          </property>
         </widget>
        </item>
@@ -210,6 +216,9 @@
             <iconset resource="../../pic/pic.qrc">
              <normaloff>:/pic/lisaarivi.png</normaloff>:/pic/lisaarivi.png</iconset>
            </property>
+           <property name="shortcut">
+            <string>Ins</string>
+           </property>
           </widget>
          </item>
          <item>
@@ -356,6 +365,9 @@
         <iconset resource="../../pic/pic.qrc">
          <normaloff>:/pic/ohje.png</normaloff>:/pic/ohje.png</iconset>
        </property>
+       <property name="shortcut">
+        <string>F1</string>
+       </property>
       </widget>
      </item>
      <item>

diff --git a/kitsas/apuri/tiliote/tiliotekirjausrivi.cpp b/kitsas/apuri/tiliote/tiliotekirjausrivi.cpp
@@ -76,8 +76,13 @@ TilioteKirjausRivi::TilioteKirjausRivi(const QVariantMap &tuonti, TilioteModel *
 
     rivi.setTili(tiliNumero);
 
+
     Tili tili = model->kitsas()->tilit()->tiliNumerolla(tiliNumero);
-    if( tili.onko(TiliLaji::TULO) || tili.onko(TiliLaji::MENO) ) {
+
+    // Alv-velvollisuuden huomioiminen tuonnissa
+    bool onkoAlv = model->kitsas()->asetukset()->onko(AsetusModel::AlvVelvollinen);
+
+    if( onkoAlv && (tili.onko(TiliLaji::TULO) || tili.onko(TiliLaji::MENO)) ) {
         rivi.setAlvkoodi(tili.alvlaji());
         rivi.setAlvprosentti(tili.alvprosentti());
     } else {

diff --git a/kitsas/db/tilimodel.cpp b/kitsas/db/tilimodel.cpp
@@ -125,9 +125,9 @@ QVariant TiliModel::data(const QModelIndex &index, int role) const
                 return QString();
             return QVariant( kp()->tiliTyypit()->tyyppiKoodilla( tili->tyyppiKoodi() ).kuvaus() );
         case ALV: {
-            int vero = tili->luku("alvprosentti");
-            if(vero)
-                return QVariant( QString("%1 %").arg(vero));
+            double vero = tili->alvprosentti();
+            if(vero > 1e-5)
+                return QVariant( QString("%1 %").arg(vero,0,'f',1));
             return QVariant();
         }
         case SALDO: