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

tasks/setup.php copyDatabase lädt SQL dump nicht aus release Ordner hoch #79

Closed
ynamite opened this issue Jan 12, 2024 · 3 comments · Fixed by #86
Closed

tasks/setup.php copyDatabase lädt SQL dump nicht aus release Ordner hoch #79

ynamite opened this issue Jan 12, 2024 · 3 comments · Fixed by #86

Comments

@ynamite
Copy link

ynamite commented Jan 12, 2024

Damit der Task tasks/setup.php bei mir durchläuft, musst ich die Methode copyDatabase leicht anpassen und zwar musste ich auf Zeile 211 folgende Zeile entfernen:

cd('{{current_path}}');

Das Problem ist, dass mit dieser Zeile der Export in /.build/release/ landet, die Funktion upload weiter unten aber als Quelle vom Projekt-Root ausgehnt /:

upload($path, "{{release_path}}/$path");

Wenn man die angegebene Zeile 211 entfernt, landet der Dump ebenfalls im korrekten Ordner, aber vom Root ausgehend (nicht im Release).

Man könnte auch umgekehrt die Zeile hier auf upload("{{current_path}}/$path", "{{release_path}}/$path"); ändern, also den current_path mitgeben, aber da bin ich unsicher, ob der SQL dump besser im Release ist oder doch im Root (Entwicklungsumgebung). Jedenfalls hat das mein Problem gelöst.

@ynamite
Copy link
Author

ynamite commented Mar 13, 2024

Nach weiteren Tests denke ich, dass die zweite Variante mehr Sinn ergibt.
Oder was meint ihr?

edit: allerdings krieg ich es mit weder mit der Original-Variante noch mit der zweiten Variante (oben) zum laufen ... hmpf

@gharlan
Copy link
Member

gharlan commented May 21, 2024

Von mir angedacht war eigentlich Variante 1.
Die nutze ich nun per #86.

(Die Zeile wird aber benötigt, wenn man einen anderen Server als Quelle nimmt, nicht Lokal. Insbesondere für das lokale Setup wichtig. Dementsprechend ist die Zeile nun in einem if.)

@ynamite
Copy link
Author

ynamite commented May 21, 2024

Alles klar, super, vielen Dank!

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

Successfully merging a pull request may close this issue.

2 participants