### Eclipse Workspace Patch 1.0 #P jverein diff --git src/de/jost_net/JVerein/io/AltersgruppenParser.java src/de/jost_net/JVerein/io/AltersgruppenParser.java index ff063b5..44f5ca3 100644 --- src/de/jost_net/JVerein/io/AltersgruppenParser.java +++ src/de/jost_net/JVerein/io/AltersgruppenParser.java @@ -49,10 +49,9 @@ if (stt.countTokens() != 2) { throw new RuntimeException( - String - .format( - "Ungültige Altersgruppe: {0} \nDie Eingaben müssen folgendes Format haben: 1-6,7-13,14-22,23-50", - new Object[] { gruppen.get(i) })); + String.format( + "Ungültige Altersgruppe: %s \nDie Eingaben müssen folgendes Format haben: 1-6,7-13,14-22,23-50", + gruppen.get(i))); } try { diff --git src/de/jost_net/JVerein/io/AltersjubilaeumsExportPDF.java src/de/jost_net/JVerein/io/AltersjubilaeumsExportPDF.java index a61f8c7..0cc5a4e 100644 --- src/de/jost_net/JVerein/io/AltersjubilaeumsExportPDF.java +++ src/de/jost_net/JVerein/io/AltersjubilaeumsExportPDF.java @@ -91,7 +91,7 @@ @Override protected void startJahrgang(int jahrgang) throws DocumentException { - Logger.debug(String.format("Altersjubiläum, Jahrgang=%d}", jahrgang)); + Logger.debug(String.format("Altersjubiläum, Jahrgang=%d", jahrgang)); Paragraph pHeader = new Paragraph("\n" + String.format("%d. Geburtstag", jahrgang), FontFactory.getFont( FontFactory.HELVETICA, 11)); diff --git src/de/jost_net/JVerein/io/CSVFileHelper.java src/de/jost_net/JVerein/io/CSVFileHelper.java index 4746f0b..fd5bcd9 100644 --- src/de/jost_net/JVerein/io/CSVFileHelper.java +++ src/de/jost_net/JVerein/io/CSVFileHelper.java @@ -124,21 +124,17 @@ { if (lastPosition - 1 >= 0 && rCache[lastPosition - 1] == SEPARATOR) { - monitor - .setStatusText(String - .format( - "Leerzeichen nach einem Semikolon in Zeile: {0} und Spalte: {1}", - new Object[] { lineNo + "", columnsPerLine + "" })); + monitor.setStatusText(String.format( + "Leerzeichen nach einem Semikolon in Zeile: %d und Spalte: %d", + new Object[] { lineNo, columnsPerLine })); valid = false; } if (lastPosition + 1 < numberOfChars && rCache[lastPosition + 1] == SEPARATOR) { - monitor - .setStatusText(String - .format( - "Leerzeichen vor einem Semikolon in Zeile: {0} und Spalte: {1}", - new Object[] { lineNo + "", columnsPerLine + "" })); + monitor.setStatusText(String.format( + "Leerzeichen vor einem Semikolon in Zeile: %d und Spalte: %d", + new Object[] { lineNo, columnsPerLine })); valid = false; } } @@ -148,11 +144,9 @@ lineNo++; if (columnsPerLine != numColumns) { - monitor - .setStatusText(String - .format( - "Anzahl der Spalten in Zeile: {0} passt nicht mit der Anzahl Spalten in der Kopfzeile ueberein.", - lineNo + "")); + monitor.setStatusText(String.format( + "Anzahl der Spalten in Zeile: %d passt nicht mit der Anzahl Spalten in der Kopfzeile ueberein.", + lineNo)); valid = false; } columnsPerLine = 0; diff --git src/de/jost_net/JVerein/io/DefaultZusatzbetraegeImport.java src/de/jost_net/JVerein/io/DefaultZusatzbetraegeImport.java index 9277367..0dfaaf6 100644 --- src/de/jost_net/JVerein/io/DefaultZusatzbetraegeImport.java +++ src/de/jost_net/JVerein/io/DefaultZusatzbetraegeImport.java @@ -195,21 +195,17 @@ if (list.size() == 0) { monitor.setStatus(ProgressMonitor.STATUS_ERROR); - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) kein Mitglied in JVerein-Datenbank gefunden. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) kein Mitglied in JVerein-Datenbank gefunden. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } else if (list.size() > 1) { monitor.setStatus(ProgressMonitor.STATUS_ERROR); - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) mehr als ein Mitglied gefunden. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) mehr als ein Mitglied gefunden. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } else @@ -222,22 +218,18 @@ double betrag = results.getDouble("Betrag"); if (betrag == 0) { - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) konnte die Fließkommazahl in der Spalte Betrag nicht verarbeitet werden. Zahl muss größer 0 sein. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) konnte die Fließkommazahl in der Spalte Betrag nicht verarbeitet werden. Zahl muss größer 0 sein. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } zus.setBetrag(betrag); String buchungstext = results.getString("Buchungstext"); if (buchungstext.length() > 140) { - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) konnte der Text in der Spalte Buchungstext nicht verarbeitet werden. Länge des Buchungstextes ist auf 140 Zeichen begrenzt. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) konnte der Text in der Spalte Buchungstext nicht verarbeitet werden. Länge des Buchungstextes ist auf 140 Zeichen begrenzt. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } zus.setBuchungstext(buchungstext); @@ -250,22 +242,18 @@ } catch (ParseException e) { - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) konnte das Datum in der Spalte Fälligkeit nicht verarbeitet werden. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) konnte das Datum in der Spalte Fälligkeit nicht verarbeitet werden. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } int intervall = results.getInt("Intervall"); if (intervall < 0) { - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) konnte die Zahl in der Spalte Intervall nicht verarbeitet werden. Zahl muss größer oder gleich 0 sein. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) konnte die Zahl in der Spalte Intervall nicht verarbeitet werden. Zahl muss größer oder gleich 0 sein. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } zus.setIntervall(intervall); @@ -281,11 +269,9 @@ } catch (ParseException e) { - monitor - .setStatusText(String - .format( - "Für die Importzeile {0} ({1}) konnte das Datum in der Spalte Fälligkeit nicht verarbeitet werden. Abbruch!", - anz + "", mitgliedIdString)); + monitor.setStatusText(String.format( + "Für die Importzeile %d (%s) konnte das Datum in der Spalte Fälligkeit nicht verarbeitet werden. Abbruch!", + anz, mitgliedIdString)); fehlerInDaten = true; } @@ -296,11 +282,9 @@ // überprüfen und parsen der Daten beendet. if (fehlerInDaten == false) { - monitor - .setStatusText(String - .format( - "Überprüfen aller Zusatzbeiträge erfolgreich abschlossen. {0} Zusatzbeiträge werden importiert...", - anz + "")); + monitor.setStatusText(String.format( + "Überprüfen aller Zusatzbeiträge erfolgreich abschlossen. %d Zusatzbeiträge werden importiert...", + anz)); int count = 0; for (Zusatzbetrag zusatzbetrag : zusatzbetraegeList) { @@ -309,7 +293,7 @@ .size())); zusatzbetrag.store(); monitor.setStatusText(String.format( - "Zusatzbeitrag für Mitglied %s erfolgreich importiert. ", + "Zusatzbeitrag für Mitglied %s erfolgreich importiert.", Adressaufbereitung.getNameVorname(zusatzbetrag.getMitglied()))); } monitor.setStatusText("Import komplett abgeschlossen."); diff --git src/de/jost_net/JVerein/io/Import.java src/de/jost_net/JVerein/io/Import.java index 92ac6bb..80f2bf5 100755 --- src/de/jost_net/JVerein/io/Import.java +++ src/de/jost_net/JVerein/io/Import.java @@ -203,12 +203,10 @@ } else if (ba.length() > 30) { - progMonitor - .log(String - .format( - "{0}, {1}: maximale Laenge von 30 Zeichen in Beitragsart_1 ueberschritten, wird automatisch gekuerzt", - this.getResultFrom(results, InternalColumns.NACHNAME), - this.getResultFrom(results, InternalColumns.VORNAME))); + progMonitor.log(String.format( + "%s, %s: maximale Laenge von 30 Zeichen in Beitragsart_1 ueberschritten, wird automatisch gekuerzt", + this.getResultFrom(results, InternalColumns.NACHNAME), + this.getResultFrom(results, InternalColumns.VORNAME))); } if (btr == null || btr.length() == 0) @@ -274,7 +272,7 @@ { parts[2] = "20" + parts[2]; progMonitor.log(String.format( - "Fuer %s wurde das 21 Jahrhundert angenommen", dotDate + "")); + "Fuer %s wurde das 21 Jahrhundert angenommen", dotDate)); } } @@ -555,13 +553,11 @@ } catch (Exception e) { - progMonitor - .log(String - .format( - "Datensatz unvollstaending (Eigenschaften) -> Import wird abgebrochen: ID= {0}, NAME= {1}: {2}", - getResultFrom(results, InternalColumns.MITGLIEDSNR), - getResultFrom(results, InternalColumns.NACHNAME), - e.getMessage())); + progMonitor.log(String.format( + "Datensatz unvollstaending (Eigenschaften) -> Import wird abgebrochen: ID= %s, NAME= %s: %s", + getResultFrom(results, InternalColumns.MITGLIEDSNR), + getResultFrom(results, InternalColumns.NACHNAME), + e.getMessage())); return false; } @@ -724,11 +720,9 @@ if (!altebankverbindung && !neuebankverbindung) { - progMonitor - .log(String - .format( - "Bei {0} ist als Zahlungsart Basislastschrift gesetzt aber weder neue Bankverbindung (IBAN / BIC) noch (BLZ /Konto) vorhanden", - Adressaufbereitung.getNameVorname(m))); + progMonitor.log(String.format( + "Bei %s ist als Zahlungsart Basislastschrift gesetzt aber weder neue Bankverbindung (IBAN/BIC) noch (BLZ/Konto) vorhanden", + Adressaufbereitung.getNameVorname(m))); throw new ApplicationException(); } } @@ -809,11 +803,9 @@ { if (austritt == null) { - progMonitor - .log(String - .format( - "{0}: beim einem definierten Sterbedatum muss es auch ein Austrittsdatum geben, setze Austrittsdatum gleich dem Sterbedatum", - Adressaufbereitung.getNameVorname(m))); + progMonitor.log(String.format( + "%s: beim einem definierten Sterbedatum muss es auch ein Austrittsdatum geben, setze Austrittsdatum gleich dem Sterbedatum", + Adressaufbereitung.getNameVorname(m))); m.setAustritt(sterbeTag); } } @@ -850,11 +842,9 @@ } else { - progMonitor - .log(String - .format( - "Individueller Beitrag fuer {0} enthält keine gültige Formatierung und wird verworfen.", - Adressaufbereitung.getNameVorname(m))); + progMonitor.log(String.format( + "Individueller Beitrag fuer %s enthält keine gültige Formatierung und wird verworfen.", + Adressaufbereitung.getNameVorname(m))); } } @@ -871,11 +861,9 @@ } else { - progMonitor - .log(String - .format( - "Personenart für {0} enthält keine gültige Formatierung. Es dürfen nur Wörter verwendet werden, die mit einem j fuer juristische Personen oder n fuer natürliche Personen beginnen. Bei leerem Inhalt wird der Standardwert n verwendet", - Adressaufbereitung.getNameVorname(m))); + progMonitor.log(String.format( + "Personenart für %s enthält keine gültige Formatierung. Es dürfen nur Wörter verwendet werden, die mit einem j fuer juristische Personen oder n fuer natürliche Personen beginnen. Bei leerem Inhalt wird der Standardwert n verwendet", + Adressaufbereitung.getNameVorname(m))); throw new ApplicationException(); } @@ -899,11 +887,9 @@ } else { - progMonitor - .log(String - .format( - "Zahlungsrythmus bei: {0} ist entweder leer oder besteht nicht nur aus Zahlen, setze auf 12 Monate", - Adressaufbereitung.getNameVorname(m))); + progMonitor.log(String.format( + "Zahlungsrythmus bei: %s ist entweder leer oder besteht nicht nur aus Zahlen, setze auf 12 Monate", + Adressaufbereitung.getNameVorname(m))); m.setZahlungsrhythmus(new Integer(12)); } } @@ -921,11 +907,9 @@ } else { - progMonitor - .log(String - .format( - "Adresstyp bei: {0} ist entweder leer oder besteht nicht nur aus Zahlen, setze auf 1 (Mitglied)", - Adressaufbereitung.getNameVorname(m))); + progMonitor.log(String.format( + "Adresstyp bei: %s ist entweder leer oder besteht nicht nur aus Zahlen, setze auf 1 (Mitglied)", + Adressaufbereitung.getNameVorname(m))); m.setAdresstyp(new Integer(1)); } } diff --git src/de/jost_net/JVerein/server/DBSupportMySqlImpl.java src/de/jost_net/JVerein/server/DBSupportMySqlImpl.java index 07d3ca6..3d04e88 100644 --- src/de/jost_net/JVerein/server/DBSupportMySqlImpl.java +++ src/de/jost_net/JVerein/server/DBSupportMySqlImpl.java @@ -22,7 +22,6 @@ import java.sql.ResultSet; import java.sql.SQLException; import java.sql.Statement; -import java.text.MessageFormat; import de.jost_net.JVerein.rmi.JVereinDBService; import de.willuhn.jameica.system.Application; @@ -122,7 +121,7 @@ @Override public String getSQLTimestamp(String content) { - return MessageFormat.format("(UNIX_TIMESTAMP({0})*1000)", + return String.format("(UNIX_TIMESTAMP(%s)*1000)", new Object[] { content }); } diff --git src/de/jost_net/JVerein/server/FelddefinitionImpl.java src/de/jost_net/JVerein/server/FelddefinitionImpl.java index d6b211f..c219919 100644 --- src/de/jost_net/JVerein/server/FelddefinitionImpl.java +++ src/de/jost_net/JVerein/server/FelddefinitionImpl.java @@ -17,7 +17,6 @@ package de.jost_net.JVerein.server; import java.rmi.RemoteException; -import java.text.MessageFormat; import de.jost_net.JVerein.Einstellungen; import de.jost_net.JVerein.keys.Datentyp; @@ -72,9 +71,8 @@ { char c = testString.charAt(i); if (validChars.indexOf(c) == -1) - throw new ApplicationException(MessageFormat.format( - "Ungültiges Zeichen ({0}) im Feldnamen an Position {1}", c + "", - i)); + throw new ApplicationException(String.format( + "Ungültiges Zeichen (%s) im Feldnamen an Position %d", c, i)); } Mitglied m = (Mitglied) Einstellungen.getDBService().createObject( Mitglied.class, null); @@ -83,8 +81,8 @@ { if (getName().equals(s)) { - throw new ApplicationException(MessageFormat.format( - "{0} ist ein reservierter Name und darf nicht verwendet werden.", + throw new ApplicationException(String.format( + "%s ist ein reservierter Name und darf nicht verwendet werden.", getName())); } } diff --git src/de/jost_net/JVerein/server/JahresabschlussImpl.java src/de/jost_net/JVerein/server/JahresabschlussImpl.java index fae29b4..4bace43 100644 --- src/de/jost_net/JVerein/server/JahresabschlussImpl.java +++ src/de/jost_net/JVerein/server/JahresabschlussImpl.java @@ -17,7 +17,6 @@ package de.jost_net.JVerein.server; import java.rmi.RemoteException; -import java.text.MessageFormat; import java.text.ParseException; import java.util.Date; @@ -106,8 +105,8 @@ new Object[] { gj.getBeginnGeschaeftsjahr()}); if (!anfangsbestaende.hasNext()) { - throw new ApplicationException(MessageFormat.format( - "Für Konto {0} {1} fehlt der Anfangsbestand.", k1.getNummer(), + throw new ApplicationException(String.format( + "Für Konto %s %s fehlt der Anfangsbestand.", k1.getNummer(), k1.getBezeichnung())); } } diff --git src/de/jost_net/JVerein/server/MailAnhangImpl.java src/de/jost_net/JVerein/server/MailAnhangImpl.java index 1e3cb62..53a9b8e 100644 --- src/de/jost_net/JVerein/server/MailAnhangImpl.java +++ src/de/jost_net/JVerein/server/MailAnhangImpl.java @@ -17,7 +17,6 @@ package de.jost_net.JVerein.server; import java.rmi.RemoteException; -import java.text.MessageFormat; import de.jost_net.JVerein.rmi.Mail; import de.jost_net.JVerein.rmi.MailAnhang; @@ -76,9 +75,9 @@ // JVereinUpdateProvider: update0093(Connection) if (dateiname.length() > 50) throw new ApplicationException( - MessageFormat.format( - "Maximale Länge (50) des Dateinamens von Mail-Anhang überschritten. ({0}, {1}...)", - dateiname.length() + "", dateiname.substring(0, 50))); + String.format( + "Maximale Länge (50) des Dateinamens von Mail-Anhang überschritten. (%d, %s...)", + dateiname.length(), dateiname.substring(0, 50))); } @Override diff --git src/de/jost_net/JVerein/server/SpendenbescheinigungImpl.java src/de/jost_net/JVerein/server/SpendenbescheinigungImpl.java index e8e722c..5334172 100644 --- src/de/jost_net/JVerein/server/SpendenbescheinigungImpl.java +++ src/de/jost_net/JVerein/server/SpendenbescheinigungImpl.java @@ -17,7 +17,6 @@ package de.jost_net.JVerein.server; import java.rmi.RemoteException; -import java.text.MessageFormat; import java.util.ArrayList; import java.util.Date; import java.util.GregorianCalendar; @@ -680,7 +679,7 @@ String zeitraumende = new JVDateFormatTTMMJJJJ().format(getZeitraumBis()); map.put(SpendenbescheinigungVar.SPENDEDATUM.getName(), "s. Anlage"); map.put(SpendenbescheinigungVar.SPENDENZEITRAUM.getName(), - MessageFormat.format("{0} bis {1}", spendedatum, zeitraumende)); + String.format("%s bis %s", spendedatum, zeitraumende)); StringBuilder bl = new StringBuilder(); if (gc.get(GregorianCalendar.YEAR) <= 2012) { diff --git src/de/jost_net/JVerein/util/HelpConsumer.java src/de/jost_net/JVerein/util/HelpConsumer.java index 6f0e1ba..2264e35 100644 --- src/de/jost_net/JVerein/util/HelpConsumer.java +++ src/de/jost_net/JVerein/util/HelpConsumer.java @@ -16,8 +16,6 @@ **********************************************************************/ package de.jost_net.JVerein.util; -import java.text.MessageFormat; - import de.willuhn.jameica.gui.AbstractView; import de.willuhn.jameica.gui.GUI; import de.willuhn.jameica.messaging.Message; @@ -35,7 +33,7 @@ AbstractView view = (AbstractView) msg.getData(); Logger.error("help missing for view " + view.getClass()); GUI.getStatusBar().setErrorText( - MessageFormat.format("Hilfe für Ansicht {0} fehlt", + String.format("Hilfe für Ansicht %s fehlt", view.getClass().getName())); } diff --git src/de/jost_net/JVerein/util/LesefeldAuswerter.java src/de/jost_net/JVerein/util/LesefeldAuswerter.java index 1094019..eb20868 100644 --- src/de/jost_net/JVerein/util/LesefeldAuswerter.java +++ src/de/jost_net/JVerein/util/LesefeldAuswerter.java @@ -17,7 +17,6 @@ package de.jost_net.JVerein.util; import java.rmi.RemoteException; -import java.text.MessageFormat; import java.util.ArrayList; import java.util.HashMap; import java.util.List; @@ -192,8 +191,8 @@ map.put(Einstellungen.LESEFELD_PRE + lesefeld.getBezeichnung(), lesefeld.getEvaluatedContent()); } - Logger.debug(MessageFormat.format("Lesefeld-Variablen für Mitglied {0}:", - vornamename)); + Logger.debug(String.format("Lesefeld-Variablen für Mitglied %s:", + vornamename)); for (String key : map.keySet()) { Logger.debug(key + "=" + map.get(key));