IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: Incorrect admin sidebar setting migration (#25192)

Browse Source 
Fixes the migration in 8c6144d116
which was unconditionally enabling the new admin sidebar for admins
if they ever changed the old setting `enable_admin_sidebar_navigation`,
even if they changed it to false.
This commit is contained in:
Martin Brennan 2024-01-10 07:24:23 +10:00 committed by GitHub
parent cef6aca6e5
commit f2105727c2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
db/post_migrate/20240108022138_change_enable_admin_sidebar_to_group_post_migration.rb
View File

@ -7,7 +7,7 @@ class ChangeEnableAdminSidebarToGroupPostMigration < ActiveRecord::Migration[7.0
"SELECT value FROM site_settings WHERE name = 'enable_admin_sidebar_navigation'", "SELECT value FROM site_settings WHERE name = 'enable_admin_sidebar_navigation'",
).first ).first
if enable_admin_sidebar_navigation_raw.present? if enable_admin_sidebar_navigation_raw.present? && enable_admin_sidebar_navigation_raw == "t"
DB.exec( DB.exec(
"INSERT INTO site_settings(name, value, data_type, created_at, updated_at) "INSERT INTO site_settings(name, value, data_type, created_at, updated_at)
VALUES('admin_sidebar_enabled_groups', :setting, '20', NOW(), NOW())", VALUES('admin_sidebar_enabled_groups', :setting, '20', NOW(), NOW())",