configDB - Bug beim Import von Dateien in DB

[Benni]

Hallo betateilchen,

nach dem ich inzwischen einige meiner Dateien, zwecks von dir angeregter Dateisicherung mittels configDB, teils mit fileimport und teils mit filemove in meine Datenbank-Konfig importiert habe (sqlite), musste ich feststellen, dass bei einigen das letzte Zeichen nicht mitimportiert wurde.

Ich habe das eben mal schnell mit einer Testdatei durchexerziert: Das Problem tritt bei beiden Varianten des Dateiimports gleichermaßen auf und anscheinend nur dann, wenn nach dem letzten Zeichen nichts weiteres mehr in der Datei drinsteht, sprich noch weitere Leerzeilen folgen somit das letzte Zeichen auch tatsächlich das letzte Zeichen in der letzten Zeile der Datei ist.

Ob das bei binary files auch ein Problem ist, habe ich allerdings nicht geprüft.

Grüsse Benni.

[betateilchen]

anscheinend nur dann, wenn nach dem letzten Zeichen nichts weiteres mehr in der Datei drinsteht

gibts hier im Forum eigentlich irgendwo einen Stilblüten-Thread?

Nach dem letzten Zeichen einer Datei kommt von Haus aus nix mehr, das liegt in der Natur eines "letzten Zeichens".

[Benni]

Ich gebe zu, das war unglücklich formuliert!

[betateilchen]

Aber Du hast die Ursache richtig erkannt:

Wenn die letzte Zeile einer Textdatei nicht mit einem Zeilenende abgeschlossen ist, passiert genau das von Dir Beschriebene. Ich bin schon dran - irgendwas muss ich mir ja seinerzeit dabei gedacht haben, das letzte Zeichen einer Zeile beim Import zu entfernen.

[betateilchen]

Kannst Du mal bitte mit der angehängten Datei testen?

[Benni]

Mach ich gerne, danke! Aber erst morgen früh. Der Rechner ist heute schon aus und ich bin eben nur noch kurz mobil online.

[betateilchen]

Vermutlich werde ich in absehbarer Zeit die Trennung in text und bin files ohnehin aufgeben. Die Unterscheidung stört mich noch an anderen Stellen.

[Benni]

Guten Morgen!

Ich habe eben die oben angehängte configDB.pm erfolgreich getestet.
Alle Zeichen meiner Testdatei wurden vollständig importiert.

Danke dir!

[betateilchen]

Danke für die Info. Das deckt sich mit meinen Tests gestern abend. Die Änderung wird heute bereits mit dem normalen fhem-Update ausgeliefert.