Conversation
FJiskra
force-pushed
the
PES-2788-v31_uninstall-update
branch
from
49e2860 to
af08d4f
Compare
FJiskra
force-pushed
the
PES-2788-v31_uninstall-update
branch
from
af08d4f to
d2af880
Compare
| { | ||
| $sql = []; | ||
| // DB migrations do not work with PACKETERY_REMOVE_ALL_DATA set to false | ||
| $dropTables = !defined('PACKETERY_REMOVE_ALL_DATA') || PACKETERY_REMOVE_ALL_DATA; |
There was a problem hiding this comment.
Neodpovídá zadání - pokud není konstanta definována, nemají se tabulky odstraňovat. I když v tomto případě by se to tak mohlo řešit.
There was a problem hiding this comment.
Měnit dosavadní výchozí chování je blbost. Připravte se na to, že to fungovat nebude.
| private function installDatabase() | ||
| { | ||
| $sql = []; | ||
| // DB migrations do not work with PACKETERY_REMOVE_ALL_DATA set to false |
There was a problem hiding this comment.
Co ten komentář znamená? Pokud je nastavena konstanta, tak to přece neovlivní spuštění skriptů ze složky upgrade, ne?
Pokud jde o situaci, že si eshopista odinstaluje starší verzi a nainstaluje novější, tak tu musí sám vyřešit. Mazání tabulek by neodpovídalo nastavení konstanty.
There was a problem hiding this comment.
Upgrade soubory se nepoužijí při čerstvé instalaci. Původní řešení to řešilo dropováním tabulek.
| */ | ||
| private function uninstallDatabase() | ||
| { | ||
| if (defined('PACKETERY_REMOVE_ALL_DATA') && !PACKETERY_REMOVE_ALL_DATA) { |
There was a problem hiding this comment.
Neodpovídá zadání - pokud není konstanta definována, nemají se tabulky odstraňovat.
| */ | ||
| private function deleteConfiguration() | ||
| { | ||
| if (defined('PACKETERY_REMOVE_ALL_DATA') && !PACKETERY_REMOVE_ALL_DATA) { |
There was a problem hiding this comment.
Neodpovídá zadání - pokud není konstanta definována, nemá se nastavení odstraňovat.
3 participants
No description provided.