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

Update scripts deletes files with no warning #475

Open
pgorod opened this issue May 7, 2024 · 1 comment
Open

Update scripts deletes files with no warning #475

pgorod opened this issue May 7, 2024 · 1 comment
Labels
Area: Upgrade/Migration Issues & PRs related to all things regarding Suite8 Upgrades & Migrations Priority:Important Issues & PRs that are important; broken functions; errors; there are workarounds Severity: Major Significant impact/severe disruption Type:Suggestion Issue containing a suggestion of functionality, process or UI. Associated PRs are called enhancement

Comments

@pgorod
Copy link
Contributor

pgorod commented May 7, 2024

Issue

Currently it seems that the update scripts use a diff to make changes to the SuiteCRM directory, causing any extraneous files and directories to just get deleted without any warning.

In the Forums some people have mentioned this. One person had backups created in the SuiteCRM directory, and they disappeared.

https://community.suitecrm.com/t/filter-using-advanced-filter-date-between-start-date-and-end-date-failed-with-7-14-and-8-5/92644/4?u=pgr

Expected Behavior

  • Destructive action should be avoided, if possible.
  • If not, the installation should at least refuse to proceed if it detects that things will get deleted. It could have a message something like this:

"Installation cannot continue because there are unknown files or folders that would be deleted. Please move them and try again".

  • if that is also not possible, then at least a big flashing red warning should be displayed.

  • another possible kind of warning is to use a mandatory switch in the command, something like this:

./bin/console suitecrm:app:upgrade -t SuiteCRM-8.6.0 -deleteAllExtraneousFiles

If that is the only way to proceed and get things installed, then people will necessarily become aware.

@johnM2401 johnM2401 added Type:Suggestion Issue containing a suggestion of functionality, process or UI. Associated PRs are called enhancement Priority:Important Issues & PRs that are important; broken functions; errors; there are workarounds Area: Upgrade/Migration Issues & PRs related to all things regarding Suite8 Upgrades & Migrations Severity: Major Significant impact/severe disruption labels May 7, 2024
@chris001
Copy link
Contributor

chris001 commented May 8, 2024

Silent 🤭 loss ♻️ 🗑️ of user generated files is bad. 😞

@pgorod pgorod changed the title Update scripts delete files with no warning Update scripts deletes files with no warning May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Upgrade/Migration Issues & PRs related to all things regarding Suite8 Upgrades & Migrations Priority:Important Issues & PRs that are important; broken functions; errors; there are workarounds Severity: Major Significant impact/severe disruption Type:Suggestion Issue containing a suggestion of functionality, process or UI. Associated PRs are called enhancement
Projects
None yet
Development

No branches or pull requests

3 participants