Neuer Modus FORCE_FILES #66
Conversation
lib/command.php
Outdated
| @@ -10,6 +12,8 @@ protected function configure() | |||
| $this | |||
| ->setName('developer:sync') | |||
| ->setDescription('Synchronizes the developer files') | |||
| ->addOption('force-db', null, InputOption::VALUE_NONE, 'Force the current status in db') | |||
lib/command.php
Outdated
| @@ -10,6 +12,8 @@ protected function configure() | |||
| $this | |||
| ->setName('developer:sync') | |||
| ->setDescription('Synchronizes the developer files') | |||
| ->addOption('force-db', null, InputOption::VALUE_NONE, 'Force the current status in db') | |||
| ->addOption('force-files', null, InputOption::VALUE_NONE, 'Force the current status in file system') | |||
| /** | ||
| * The method is called, when an existing item is deleted by the file system (`FORCE_FILES` is activated) | ||
| * | ||
| * Use the method to delete the item in the base system |
There was a problem hiding this comment.
Kommt per copy/paste von den anderen Methoden.
Das Ding ist, das developer-Addon ist nicht nur darauf ausgelegt, in die DB zu synchronisieren, sondern theoretisch sonst wo hin. Also die eine Seite sind immer Dateien (Developer-Dateien). Die andere Seite bestimmen die Synchronizer. Die drei default Synchronizer (Templates, Module, Actions) syncen halt in die DB. Diese Basis-Synchronizer-Klasse lässt das aber offen.
| */ | ||
| public static function start() | ||
| public static function start($force = false) |
There was a problem hiding this comment.
willste nicht lieber nur noch die konstanten unterstützen statt dem bool?
There was a problem hiding this comment.
Ich will keinen BC-Break..
false == kein force
1 oder true == force_db (so wie auch vorher bei true)
2 == force_files
|
Ggf wäre es sinnvoll diesen mode generell für eine redaxo instanz festzulegen statt nur für einen commando aufruf...? Aktuell stehst du in meinen augen vor dem problem dass du ggf. Jmd anderem seine arbeit überschreibst, die via webinterface gemacht wurde (oder umgekehrt). Wenn man den mode in der instanz einstellt könnte man in der redaxo oberfläche das editieren sperren/eine warnung ausgeben, wenn via files gearbeiten werden soll/muss etc |
Konkret in meinem/unseren Use case besteht das Problem nicht. Auf den Servern ist das Syncing im Backend und Frontend komplett ausgeschaltet, wird immer nur einmalig beim Deployen gesynct. Ganz allgemein magst du eventuell Recht haben. Finde es aber trotzdem gut, im Kommando diese Force-Options zu haben, und würde es daher erst mal so belassen, da uns das so erstmal reicht. |
vllt würde es sinn machen ein initiales setup commando mit ydeploy zu haben, was einem diese einstellung vorschlägt? |
Ja, ist geplant. |
Bisher gibt es zwei Modi für die Synchronisation.
Der normale Modus, wo je nach Zeitstempel der DB-Datensatz oder die Datei gewinnt.
Und dann gibt es den force-Mode, wo immer die DB gewinnt. Dieser wird zum Beispiel nach einem Backup-Import ausgeführt.
Für YDeploy benötigt ich aber das Gegenteil, einen force-Mode, wo die Dateien gewinnen: yakamara/ydeploy#7
Beim Deployen soll der per Dateien aufgespielte Stand garantiert werden.
Daher schlage ich vor, den force-Mode zu trennen in
FORCE_DBundFORCE_FILES.YDeploy würde dann
redaxo/bin/console developer:sync --force-filesnutzen.