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

AbstractPackageReader Code überarbeiten #49

Open
j3nsch opened this issue Oct 28, 2024 · 0 comments
Open

AbstractPackageReader Code überarbeiten #49

j3nsch opened this issue Oct 28, 2024 · 0 comments
Assignees

Comments

@j3nsch
Copy link
Member

j3nsch commented Oct 28, 2024

Der AbstractPackageReader hat zwei abgeleitete Klassen, TarPackageReader und ZipPackageReader. Diese Klassen kümmern sich um das Auspacken entsprechender Dateien in ein temporäres Verzeichnis. Sie kümmern sich aber auch um die Verarbeitung des Inhalts des Datei. Damit ist die Verwendbarkeit auf die OPUS 4 Sword-Pakete beschränkt.

Die Tar/Zip-Funktionalität soll unabhängig von der Weiterverarbeitung nutzbar sein. Es möglich sein den weiter verarbeitenden Code mit einem Verzeichnis zu verwenden, egal ob es vorher ausgepackt wurde oder schon da war.

Es sollte eine Funktion für die Bereinigung der temporären Dateien geben, deren Verwendung durch den übergeordneten Workflow-Code gesteuert wird.

Die Tar/Zip-Klassen sollten wissen welche MIME-Types sie unterstützen. Klassen, die weitere MIME-Types unterstützen sollten konfigurierbar sein.

Der eigentliche Code für die Verarbeitung von Imports sollte nicht direkt von den Klassen abhängig sein. Dafür muss ein Interface eingeführt werden.

Die Namen der Dateien sollten nicht fest im Code stehen.

@j3nsch j3nsch self-assigned this Oct 28, 2024
j3nsch added a commit that referenced this issue Oct 28, 2024
j3nsch added a commit that referenced this issue Oct 29, 2024
j3nsch added a commit that referenced this issue Oct 30, 2024
j3nsch added a commit that referenced this issue Oct 30, 2024
j3nsch added a commit that referenced this issue Oct 30, 2024
j3nsch added a commit that referenced this issue Oct 30, 2024
j3nsch added a commit that referenced this issue Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Status: No status
Development

No branches or pull requests

1 participant