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

[#28003] Multidb install #122

Closed
wants to merge 10 commits into from
Closed

Conversation

elkuku
Copy link
Contributor

@elkuku elkuku commented Feb 5, 2012

This modifies the installation process to allow database specific checks, like version etc.

@infograf768
Copy link
Member

Do we have a tracker?

@elkuku
Copy link
Contributor Author

elkuku commented Feb 8, 2012

Apologies: It's #28003

* @package Joomla.Libraries
* @since ¿
*/
class JDatabaseInstallerException extends Exception
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The platform is moving away from these non-semantic exceptions. I don't think the CMS should start with them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have any documentation or guidelines on this ?

@realityking
Copy link
Contributor

Could you explain why you made this a library? I see no usefulness beyond the installation for this kinda stuff. I'd just put it into the installation folder.

@elkuku
Copy link
Contributor Author

elkuku commented Feb 8, 2012

I can imagine the possibility to migrate the database from one engine to another. So the code would be used to check and setup the database for the new engine.
As a side effect the classes will be picked up by the autoloader.
Oh, and they fit in the concept that I am trying here:
https://github.com/elkuku/joomla-cms/tree/multidb/libraries/cms/database
missing packages for updater, driver and query, which may or not be part of the libraries/joomla package..

@elkuku
Copy link
Contributor Author

elkuku commented Jul 29, 2012

So, this might be closed after six month without attention, or is there anything that is missing ?
Yes I would believe that is currently not mergeable..

@nicksavov
Copy link
Contributor

Thanks for coding this, Nikolai! Could you update the pull request since it's out of sync? Once it's updated, please leave a message on here and I'll get a reviewer to review it for you and provide feedback on your questions.

Thanks! :)

@phproberto
Copy link
Contributor

I'm closing this as it's not mergeable and 2 years old. Feel free to open if you update it and are still interested in get it merged.

Thanks for contributing.

@phproberto phproberto closed this Oct 3, 2013
@elkuku elkuku deleted the multidb-install branch April 19, 2017 05:04
wilsonge pushed a commit that referenced this pull request Jun 4, 2017
* Use JForm to render the action plugins

* Only determine the with and height when it is a file

* Updates

* Add demo crop plugin

* Introduce an event bus to activate and deactivate a plugin n the js side

* Not needed

* Valid HTML

* Localizing the plugins

* Move to plugin layouts

* New lines

* Add rotate plugin

* Add rotate plugin

* Remove not needed code
yvesh pushed a commit to yvesh/joomla-cms that referenced this pull request Jun 18, 2017
Fix in the RoboFile in the env parameter
bembelimen referenced this pull request in bembelimen/joomla-cms Apr 26, 2018
obuisard added a commit to obuisard/joomla-cms that referenced this pull request Feb 21, 2023
* Replaced tour with guidedtours in plugin test

* Renamed onAjax function

* Error assigning step id

* Steps never get to complete button
Error progress indicator current step

* Added aria-label on step buttons

* attachTo unnecessary for start tour step
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 this pull request may close these issues.

None yet

5 participants