Skip to content
This repository has been archived by the owner on May 4, 2020. It is now read-only.

Fix: NPE when trying to restore the module on Windows #4

Open
wants to merge 2 commits into
base: EIPs/EIP-57
Choose a base branch
from

Conversation

Spalek
Copy link

@Spalek Spalek commented Apr 30, 2020

writeExternal() and readExternal() was changed in order to initialize the CRSManager before calling setStandardSRS().

@jechterhoff könntest Du das bei Dir nochmal ausprobieren?
Verhalten vor dem Fix: Beim zweimaligen Ausführen eines Testlaufs auf dem selben Testdatensatz:
Der zweite Testlauf führt zu einem Fehler, wenn man die gespeicherten Ressourcen tmp/*.obj nicht löscht:
[err:XPTY0004] java.lang.NullPointerException. Caused by: ReusableResource:restore(GmlGeoX, String).
C:/Users/spalek/Docume..."

Verhalten nach dem Fix: Beim zweimaligen Ausführen eines Testlaufs auf dem selben Testdatensatz:
Der zweite Testlauf lädt alle Ressourcen aus dem ersten Testlauf und sollte ohne Exception durch laufen.

Falls du auch ein Skript nutzt, welches die Ressourcen löscht: Ich habe bei mir zum Testen die entsprechende Zeile mit REM auskommentiert:
REM del C:\Users\spalek\Documents\tmp\*.obj /s /q

writeExternal() and readExternal() was changed in order to initialize the CRSManager before calling setStandardSRS().
@Spalek Spalek requested a review from jechterhoff April 30, 2020 13:18
Copy link

@jechterhoff jechterhoff left a comment

Choose a reason for hiding this comment

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

Funktioniert bei mir: nachdem per Skript die Testdatenbank angelegt wurde kann man in der BaseX GUI mehrere Testläufe ausführen, ohne dass ein Fehler auftaucht und ohne dass die Datenbank neu angelegt werden müsste (wie zuvor, um den Fehler zu umgehen).

Kurzer Hinweis am Rande: Im Script sollte man das Löschen der .obj-Dateien behalten, da ansonsten bei Anlegen einer neuen Testdatenbank es zu einem Fehler kam.

Insgesamt eine gute Verbesserung der Abläufe für die lokale Testentwicklung.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants