Skip to content
This repository was 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
Spalek wants to merge 2 commits into
interactive-instruments:EIPs/EIP-57from
Spalek:fix-readExternal
Open

Fix: NPE when trying to restore the module on Windows#4
Spalek wants to merge 2 commits into
interactive-instruments:EIPs/EIP-57from
Spalek:fix-readExternal

Conversation

@Spalek

@Spalek Spalek commented Apr 30, 2020

Copy link
Copy Markdown

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

@jechterhoff jechterhoff left a comment

Copy link
Copy Markdown

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