maschhoff / prpdf

preview and rename PDF Explorer
MIT License
11 stars 4 forks source link

Main archiv and unknown folders do not populate subfolders + improve documentation #17

Open tobiasgraeber opened 2 weeks ago

tobiasgraeber commented 2 weeks ago

Hi @maschhoff , vorweg, tolles Projekt!

Ich habe drei potenzielle Fehler gefunden:

1) Wenn man unter v '/mnt/user/Share' weitere Unterordner hat (oder diese anlegt), dann tauchen diese unter dem "Select folder"-Dialog (auch nach reload/restart des containers) - NICHT - auf. Das ist irgendwie kaputt...Die beiden Links hier klappen zwar, auch das Verschieben etc, aber die Unterordner fehlen immer:

grafik

2) In der Readme ist ferner angegeben, dass sich weitere eigene Volumes/Unterordner hinzufügen lassen mit:

-v '/mnt/user/Files/Accounting':'/Archiv/Accounting':'rw'

aber das klappt nicht. Was hingegen klappt ist, wenn man es wie folgt (mit /source/ im container-pfad vorangestellt) angibt:

-v '/mnt/user/Files/Accounting':'/source/archiv/Accounting':'rw'

Die so angegebenen eigenen weiteren Volumes, werden im "Select folder"-Dialog angezeigt. Wenn innerhalb des Host-Pfads dabei Unterodner angelegt werden - klappt hier auch die Auflistung dieser. (So sollte es doch auch unter 1) sein, oder?)

Ggf. die Readme updaten hinsichtlich aller hier angegebener Pfadbeispiele, insb. unter "Run as Docker - Note 2: ...." ?

3) Wenn das /source/static/pdf Verzeichnis (unknown) keine PDFs mehr enthält dann zeigt das "Edit and move" Textfeld "No data available in t" als value an (Vgl. auch https://github.com/maschhoff/prpdf/issues/16#issue-2058494549) -> fix siehe: https://github.com/maschhoff/prpdf/pull/18


Darüber hinaus habe ich folgende Anregungen:

maschhoff commented 1 week ago

Jap, das ist n Bug - ich plan mir mal für nächstes Wochenende ein die Ursache zu finden und das zu fixen. Danke für die guten Ideen das lässt sich alles easy umsetzen - mir fehlt gerade nur die Zeit ;-) Deine Hilfe ist echt willkommen, danke dafür!

maschhoff commented 1 week ago

So that was one line of code fixing 3 errors at once.

Also mit einer Zeile Code konnte ich alle 3 Fehler beheben. Du hast mir den Hinweis so gesehen schon mitgegeben. Er hat das Archiv aus /source/archiv nutzen wollen - da ist aber nichts, da es auch kein Mountpoint im Docker Container ist - diesen habe ich nun auf /Archiv/ geändert, sodass er sofort dort alle Ordner gelesen hat - auch das anlegen von Subdirs über Docker Mounting geht so - und damit sollte zu 3. auch der Fehler behoben sein, dass Dokumente ins Nirvana verschwinden.

Danke!!!

tobiasgraeber commented 1 week ago

Hi @maschhoff dankeschön, toll :-) Freut mich. Habe das die Version gleich mal ausprobiert aber der Fehler lt. meinem Screenshot oben besteht fort. Er zeigt mit im Frontend keine Unterordner von /Archiv an, trotz dessen das ich welche habe z.B. /Archiv/Unterodner1 etc. Das ">" vor dem Ordner dreht sich zwar nach unten aber weiter passiert nichts:

grafik

Neu ist nun das "Unterordner 1" direkt (unten drunter, wie im Screen zu sehen) auftaucht, aber auch hier passiert nichts wenn man diesen "aufklappt". (Unter "Unterordner 1" gibt es auch noch "Unterordner2" der erscheint bislang auch nicht.).

(Das div mit class "nested" im DOM ist/bleibt leer)

Kannst du ggf. die Doku für die Docker-Mounts/Einen Screenshot der korreken Unraid Config die bei dir funktioniert einmal senden? Dann gleiche ich das ab :-)

Ich habe aktuell für files gesetzt:

Container Path: /Archiv/ HostPath: [meinpfad]/Archiv/

Danke!