Skip to content
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

Fix Bugs in Backup und BuchungImpl #672

Merged

Conversation

JohannMaierhofer
Copy link

@JohannMaierhofer JohannMaierhofer commented Feb 14, 2025

Es wurden Rechnungen nicht im Diagnose Backup exportiert und dann war noch ein Typo in BuchungImpl.

Es gibt auch noch eine Fehler beim Import der Diagnosedaten bei den Buchungen.
"1 fehlerhaft class java.lang.Integer cannot be cast to class java.lang.Long (java.lang.Integer and java.lang.Long are in module java.base of loader 'bootstrap'), überspringe"
Das passiert wenn eine Spendenbescheinigung referenziert ist. Ich verstehe aber nicht woher das kommt.
Es passiert in Zeile 174 bei getBuchungsart()

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Feb 14, 2025
@JohannMaierhofer
Copy link
Author

Muss nach #666 übernommen werden wegen dem Plugin.xml

@JohannMaierhofer
Copy link
Author

Ich ahbe das Problem mit getProjektID gefunden. Projekt ist als foreign Objekt definiert. Da kann man dann wohl kein (Long) super.getAttribute() machen. Ein setAttribute mit Long geht anscheinend schon.

Dann habe ich festgestellt, dass beim Diagnose Backup Import die plausi Checks aufgerufen werden. Ich ahbe darum das limit bei Buchungen auf 50 Jahre gesetzt. Ich konnte sonst nicht alle Buchungen importieren.

@JohannMaierhofer
Copy link
Author

Jetzt habe ich auch das Problem mit dem Import von Buchungen gelöst. Man muss Long in den Gettern durch Object ersetzten. Integer wäre vermutlich auch gegangen.

@JohannMaierhofer
Copy link
Author

Die Sollbuchungpositionen wurden auch nicht im Diagnose backup exportiert.

if (cal1.before(cal2))
{
throw new ApplicationException("Buchungsdatum liegt mehr als 10 Jahre zur�ck");
throw new ApplicationException(
"Buchungsdatum liegt mehr als 50 Jahre zur�ck");

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das sind dann aber 40 Jahre. Erst wird 10 Jahr in die Zukunft gegangen und dann 50 Jahre von da in die Vergangeneheit.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Habe oben auf 60 geändert

@JohannMaierhofer JohannMaierhofer merged commit 13f5433 into openjverein:master Feb 18, 2025
2 checks passed
@JohannMaierhofer JohannMaierhofer deleted the fix_backup_und_buchung branch February 18, 2025 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix This will fix a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants