No Code Attached Yet J4 Issue
avatar coolcat-creations
coolcat-creations
23 Nov 2017

In the Database Options we don't have the possibility anymore to backup old tables.
Do we really want to remove this feature or move it in something like an advanced installer or bring it back?

avatar coolcat-creations coolcat-creations - open - 23 Nov 2017
avatar joomla-cms-bot joomla-cms-bot - change - 23 Nov 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Nov 2017
avatar ciar4n
ciar4n - comment - 23 Nov 2017

I'd say remove it. There is plenty other ways of keeping a copy of the old database.

Having said that we should problem warn users that the database will be over written.

avatar dgrammatiko
dgrammatiko - comment - 23 Nov 2017

@ciar4n it's not (me thinks)

avatar franz-wohlkoenig franz-wohlkoenig - change - 24 Nov 2017
Category Administration
avatar franz-wohlkoenig franz-wohlkoenig - change - 24 Nov 2017
Status New Discussion
avatar coolcat-creations
coolcat-creations - comment - 24 Nov 2017

Can we detect if tables with the same prefix exist, and if so asking if the user wants to keep them or not?

avatar dgrammatiko
dgrammatiko - comment - 24 Nov 2017

Can we detect if tables with the same prefix exist, and if so asking if the user wants to keep them or not?

Yes, once we promise-fy the client side, this would be easier (and is kinda our target) to achieve

avatar infograf768
infograf768 - comment - 26 Nov 2017

I have explained here why we do need this feature back as it does not only concerns Advanced users.
#18797 (comment)

avatar brianteeman brianteeman - change - 25 Mar 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 25 Mar 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Aug 2018
Title
[4.0 Installer] DB Options - Advanced Mode?
[4.0] [Installer] DB Options - Advanced Mode?
avatar joomla-cms-bot joomla-cms-bot - edited - 21 Aug 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Apr 2019
Title
[4.0] [Installer] DB Options - Advanced Mode?
[4.0] DB Options - Advanced Mode?
avatar franz-wohlkoenig franz-wohlkoenig - edited - 19 Apr 2019
avatar brianteeman
brianteeman - comment - 14 Sep 2019

I can only assume that in the intervening time the code has been changed and is no longer relevant.

If you try to install joomla on an existing database with an existing database prefix then the existing tables are renamed to bak_prefix_tablename

@franz-wohlkoenig this can be closed

avatar coolcat-creations
coolcat-creations - comment - 14 Sep 2019

But if the tables belong to another installation they should not be renamed.

avatar brianteeman
brianteeman - comment - 14 Sep 2019

no different to today with joomla 3

The only way that would happen would be if

  1. you manually chose the same prefix - backup created - no change to today
  2. the random prefix used for site 1 is the same random prefix as used for site 2. Almost never going to happen but as a safety measure a backup is created.

So what is the problem you want to solve?

avatar HLeithner HLeithner - change - 15 Sep 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-09-15 12:16:58
Closed_By HLeithner
avatar HLeithner
HLeithner - comment - 15 Sep 2019

Wasn't the plan to make the installer more simple? Normally you should never install 2 or more joomla installations into one database. The only reason why we have prefixes is because mysql does not support schemas and you often only get one database from the hoster... so in the end it's a workaround for a infrastructure problem out of scope of joomla...

Anyway if we make auto backups if there is a installation it would be fine for me, hopefully the user get an information that his old database get backuped.

Can be reopened if needed.

avatar HLeithner HLeithner - close - 15 Sep 2019
avatar coolcat-creations
coolcat-creations - comment - 15 Sep 2019

Should this not be leaved open to at least make sure we inform the user what happened

Normally you should never install 2 or more joomla installations into one database.

That's true for experienced users, but the plan is also to increase usability, so your recommendation to inform the user is for me mandatory.

Something like

  • "We found another Joomla database tables in this database, for security reasons we recommend to... (...) "
  • "We found another Joomla database with the same prefix (...) if you click on continue the old tables will be backed up"
    Kind of...
avatar brianteeman
brianteeman - comment - 15 Sep 2019

Again - there is no change in behaviour to joomla 3 - its just that the backup is automatic now and not left to the user

avatar coolcat-creations
coolcat-creations - comment - 15 Sep 2019

Yes, I got you. Anyway we can improve the User Experience and lead them and inform them what is happening there.

avatar HLeithner HLeithner - change - 15 Sep 2019
Status Closed New
Closed_Date 2019-09-15 12:16:58
Closed_By HLeithner
avatar HLeithner
HLeithner - comment - 15 Sep 2019

Notifying the user should really be done. Silently moving an old database would be a bad experience. If someone writes a PR which include a decision before it get moved it would be ok to but only if needed and not always shown.

avatar HLeithner HLeithner - reopen - 15 Sep 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Sep 2019
Status New Discussion
avatar brianteeman
brianteeman - comment - 17 Sep 2019

the code to remove old database or backup old database is still there. just a ui not present so easy to add it.

avatar Hackwar Hackwar - change - 14 Nov 2022
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2022-11-14 11:58:16
Closed_By Hackwar
Labels Added: No Code Attached Yet
Removed: ?
avatar Hackwar Hackwar - close - 14 Nov 2022
avatar Hackwar
Hackwar - comment - 14 Nov 2022

Yes, we could introduce an Ajax call to check for existing tables and then display a warning message that the tables exist and will be renamed, requiring further action from the user and... Or we introduce a message below the prefix field and thus close this issue, since we have a PR with #39215. If someone feels the need to change this, they can do this in the future, but we don't have to keep an issue open for 5 years without any solution. Anyway, thanks for the report and the discussion, I hope this solves the issue sufficiently.

avatar coolcat-creations
coolcat-creations - comment - 14 Nov 2022

Please do not close issues because there is no solution. An issue that is 5 years old should even get higher priority when there is still no solution.

avatar Hackwar
Hackwar - comment - 14 Nov 2022

I created a solution in the PR I linked.

avatar coolcat-creations
coolcat-creations - comment - 14 Nov 2022

Ok, why did you write "but we don't have to keep an issue open for 5 years without any solution."

avatar Hackwar
Hackwar - comment - 14 Nov 2022

Because if no one wants to do the totally proper solution with the AJAX call and all that stuff, then I'd rather provide the simple, not perfect solution and close a 5 year old issue, than keeping it open for yet another 5 years.

avatar coolcat-creations
coolcat-creations - comment - 14 Nov 2022

Fair enough ok!

Add a Comment

Login with GitHub to post a comment