Project

General

Profile

Feature #7538

Move (some) settings from main/install/configuration.dist.php to settings_current table

Added by Yannick Warnier almost 5 years ago. Updated over 4 years ago.

Status:
Feature implemented
Priority:
Normal
Category:
Database & API changes
Target version:
Start date:
19/02/2015
Due date:
% Done:

100%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Move most settings from the config file to the settings_current table.
Some settings will have to remain in the config file though, like the prevent_delete_users, as an admin shouldn't be able to change it.

There is a considerable amount of work to be done here to put each setting in a sensible category and add a title and a comment to language variables.

Associated revisions

Revision fbce1a11 (diff)
Added by Yannick Warnier over 4 years ago

Add helper method to add current settings through migrations - refs #7538

Revision e7e854f2 (diff)
Added by Yannick Warnier over 4 years ago

Partial move of settings from configuration.php to table settings_current - refs #7538

Revision e2328b37 (diff)
Added by Yannick Warnier over 4 years ago

Fix queries of previous commit - refs #7538

Revision e2744569 (diff)
Added by Yannick Warnier over 4 years ago

More fixes on previous migration - refs #7538

Revision 58fe99b5 (diff)
Added by Yannick Warnier over 4 years ago

Disable CAPTCHA by default - refs #7538

Revision 9b53d6a3 (diff)
Added by Yannick Warnier over 4 years ago

Add last_login field to user table - refs #7538

Revision 25c7825e (diff)
Added by Yannick Warnier over 4 years ago

Add settings to settings_current table from configuration.dist.php - refs #7538

Revision b990fc79 (diff)
Added by Yannick Warnier over 4 years ago

Changes to recent current settings - refs #7538

Revision 9acc0209 (diff)
Added by Yannick Warnier over 4 years ago

Update and adjust code to match new database current settings - refs #7538

Revision d8831cd7 (diff)
Added by Yannick Warnier over 4 years ago

Fix duplicate setting insert - refs #7538

Revision da959c36 (diff)
Added by Yannick Warnier over 4 years ago

Fix default values of new settings - refs #7538

History

#1

Updated by Yannick Warnier over 4 years ago

  • % Done changed from 0 to 10
#2

Updated by Yannick Warnier over 4 years ago

Also add the sso subclass as explained in drupal-chamilo module's readme

#3

Updated by Yannick Warnier over 4 years ago

Moved remaining parameters to database. Only missing translations now.

#4

Updated by Yannick Warnier over 4 years ago

The procedure here is as follows:
  • add the setting to an src/Chamilo/CoreBundle/Migration/Schema/V110/Versionxyz.php (using $this->addSettingCurrent()) (thinking deeply about the default value and the language variables names)
  • add the setting (and options) to the settings_current and settings_option tables in main/install/data.sql
  • update the database version in Versionxyz.php, data.sql and database.sql
  • check if maybe some of the settings can be removed by just altering the database (some settings were there only because they required a database change in previous versions)
  • remove the setting from main/install/configuration.dist.php
  • add the Title and Comment variables to translate.chamilo.org and translate to English
  • search for all uses of these variables in the Chamilo code and make sure that all calls to $_configuration['variable'] or api_get_configuration_value() are replaced by api_get_setting() and that the value is checked against the string value of the setting ('true' rather than true)
#5

Updated by Yannick Warnier over 4 years ago

  • Status changed from Assigned to Feature implemented
  • % Done changed from 10 to 100

Moved all necessary settings to migrations.

Also available in: Atom PDF