Skip to content

Commit

Permalink
Fix individueller Beitrag (#291)
Browse files Browse the repository at this point in the history
* Fix individueller Beitrag

* 0 Werte auf NULL setzen

* Individueller Beitrag nicht negativ
  • Loading branch information
JohannMaierhofer authored Aug 14, 2024
1 parent 1aa4836 commit c8a7546
Show file tree
Hide file tree
Showing 6 changed files with 51 additions and 10 deletions.
7 changes: 7 additions & 0 deletions src/de/jost_net/JVerein/Variable/MitgliedMap.java
Original file line number Diff line number Diff line change
Expand Up @@ -180,8 +180,15 @@ public Map<String, Object> getMap(Mitglied m, Map<String, Object> inma,
VarTools.maskieren(m.getIban()));
map.put(MitgliedVar.IBAN.getName(), m.getIban());
map.put(MitgliedVar.ID.getName(), m.getID());
if (m.getIndividuellerBeitrag() != null)
{
map.put(MitgliedVar.INDIVIDUELLERBEITRAG.getName(),
Einstellungen.DECIMALFORMAT.format(m.getIndividuellerBeitrag()));
}
else
{
map.put(MitgliedVar.INDIVIDUELLERBEITRAG.getName(), null);
}
map.put(MitgliedVar.BANKNAME.getName(), getBankname(m));
map.put(MitgliedVar.KONTOINHABER_ADRESSIERUNGSZUSATZ.getName(),
m.getKtoiAdressierungszusatz());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -310,7 +310,7 @@ private void generiereMitglied(Reporter rpt, Mitglied m)
if (Einstellungen.getEinstellung().getIndividuelleBeitraege())
{
rpt.addColumn("Individueller Beitrag", Element.ALIGN_LEFT);
if (m.getIndividuellerBeitrag() > 0)
if (m.getIndividuellerBeitrag() != null)
{
rpt.addColumn(
Einstellungen.DECIMALFORMAT.format(m.getIndividuellerBeitrag())
Expand Down
2 changes: 1 addition & 1 deletion src/de/jost_net/JVerein/gui/control/MitgliedControl.java
Original file line number Diff line number Diff line change
Expand Up @@ -1193,7 +1193,7 @@ public DecimalInput getIndividuellerBeitrag() throws RemoteException
}
individuellerbeitrag = new DecimalInput(
getMitglied().getIndividuellerBeitrag(), Einstellungen.DECIMALFORMAT);
individuellerbeitrag.setName("individueller Beitrag");
individuellerbeitrag.setName("Individueller Beitrag");
return individuellerbeitrag;
}

Expand Down
3 changes: 1 addition & 2 deletions src/de/jost_net/JVerein/io/AbrechnungSEPA.java
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@
import de.jost_net.OBanToo.SEPA.Basislastschrift.Zahler;
import de.jost_net.OBanToo.StringLatin.Zeichen;
import de.willuhn.datasource.rmi.DBIterator;
import de.willuhn.datasource.rmi.DBObject;
import de.willuhn.jameica.gui.GUI;
import de.willuhn.jameica.gui.internal.action.Program;
import de.willuhn.jameica.hbci.HBCIProperties;
Expand Down Expand Up @@ -413,7 +412,7 @@ private JVereinZahler abrechnungMitgliederSub(AbrechnungSEPAParam param, Progres
if (primaer)
{
if (Einstellungen.getEinstellung().getIndividuelleBeitraege()
&& m.getIndividuellerBeitrag() > 0)
&& m.getIndividuellerBeitrag() != null)
{
betr = m.getIndividuellerBeitrag();
}
Expand Down
34 changes: 34 additions & 0 deletions src/de/jost_net/JVerein/server/DDLTool/Updates/Update0442.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
/**********************************************************************
* This program is free software: you can redistribute it and/or modify it under the terms of the
* GNU General Public License as published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without
* even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
* the GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License along with this program. If not,
* see <http://www.gnu.org/licenses/>.
*
**********************************************************************/
package de.jost_net.JVerein.server.DDLTool.Updates;

import de.jost_net.JVerein.server.DDLTool.AbstractDDLUpdate;
import de.willuhn.util.ApplicationException;
import de.willuhn.util.ProgressMonitor;

import java.sql.Connection;

public class Update0442 extends AbstractDDLUpdate
{
public Update0442(String driver, ProgressMonitor monitor, Connection conn)
{
super(driver, monitor, conn);
}

@Override
public void run() throws ApplicationException
{
execute("update mitglied set individuellerbeitrag = NULL where individuellerbeitrag <= 0");
}
}
13 changes: 7 additions & 6 deletions src/de/jost_net/JVerein/server/MitgliedImpl.java
Original file line number Diff line number Diff line change
Expand Up @@ -287,6 +287,12 @@ && getZahlerID() == null)
{
throw new ApplicationException("Bitte Zahler auswählen!");
}

// Individueller Beitrag darf nicht kleiner als 0 sein
if (getIndividuellerBeitrag() != null && getIndividuellerBeitrag() < 0)
{
throw new ApplicationException("Individueller Beitrag darf nicht negativ sein!");
}
}

/***
Expand Down Expand Up @@ -1047,12 +1053,7 @@ public void setBeitragsgruppe(Integer beitragsgruppe) throws RemoteException
@Override
public Double getIndividuellerBeitrag() throws RemoteException
{
Double d = (Double) getAttribute("individuellerbeitrag");
if (d == null)
{
return Double.valueOf(0);
}
return d;
return (Double) getAttribute("individuellerbeitrag");
}

@Override
Expand Down

0 comments on commit c8a7546

Please sign in to comment.