Commit 7733e8c3 authored by Felipe Artur's avatar Felipe Artur

Add migration to fix corrupted projects

parent 05cb1816
class ChangeProjectIdToNotNullInProjectFeatures < ActiveRecord::Migration
DOWNTIME = false
def up
# Deletes corrupted project features
delete_project_features_sql = "DELETE FROM project_features WHERE project_id IS NULL"
execute(delete_project_features_sql)
change_column_null :project_features, :project_id, false
end
def down
change_column_null :project_features, :project_id, true
end
end
class FixProjectsWithoutProjectFeature < ActiveRecord::Migration
DOWNTIME = false
def up
# Creates missing project features with private visibility
sql =
%Q{
INSERT INTO project_features(project_id, repository_access_level, issues_access_level, merge_requests_access_level, wiki_access_level,
builds_access_level, snippets_access_level, created_at, updated_at)
SELECT projects.id as project_id,
10 as repository_access_level,
10 as issues_access_level,
10 as merge_requests_access_level,
10 as wiki_access_level,
10 as builds_access_level ,
10 as snippets_access_level,
projects.created_at,
projects.updated_at
FROM "projects"
LEFT OUTER JOIN project_features ON project_features.project_id = projects.id
WHERE (project_features.id IS NULL)
}
execute(sql)
end
def down
end
end
...@@ -11,7 +11,7 @@ ...@@ -11,7 +11,7 @@
# #
# It's strongly recommended that you check this file into your version control system. # It's strongly recommended that you check this file into your version control system.
ActiveRecord::Schema.define(version: 20170824162758) do ActiveRecord::Schema.define(version: 20170913180600) do
# These are extensions that must be enabled in order to support this database # These are extensions that must be enabled in order to support this database
enable_extension "plpgsql" enable_extension "plpgsql"
...@@ -1113,7 +1113,7 @@ ActiveRecord::Schema.define(version: 20170824162758) do ...@@ -1113,7 +1113,7 @@ ActiveRecord::Schema.define(version: 20170824162758) do
add_index "project_authorizations", ["user_id", "project_id", "access_level"], name: "index_project_authorizations_on_user_id_project_id_access_level", unique: true, using: :btree add_index "project_authorizations", ["user_id", "project_id", "access_level"], name: "index_project_authorizations_on_user_id_project_id_access_level", unique: true, using: :btree
create_table "project_features", force: :cascade do |t| create_table "project_features", force: :cascade do |t|
t.integer "project_id" t.integer "project_id", null: false
t.integer "merge_requests_access_level" t.integer "merge_requests_access_level"
t.integer "issues_access_level" t.integer "issues_access_level"
t.integer "wiki_access_level" t.integer "wiki_access_level"
......
...@@ -58,7 +58,7 @@ describe Projects::UpdateService, '#execute' do ...@@ -58,7 +58,7 @@ describe Projects::UpdateService, '#execute' do
end end
end end
context 'When project visibility is higher than parent group' do context 'when project visibility is higher than parent group' do
let(:group) { create(:group, visibility_level: Gitlab::VisibilityLevel::INTERNAL) } let(:group) { create(:group, visibility_level: Gitlab::VisibilityLevel::INTERNAL) }
before do before do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment