discourse/app/models/user_associated_group.rb
David Taylor 3332e9f4c3
DEV: Always pass --force to annotaterb and reorder annotations (#39977)
`.annotaterb.yml` has carried `classified_sort: true` since the project
switched from `annotate` to `annotaterb` (commit 0eab7daea4, July 2025),
but annotaterb's default behaviour is to compare the existing schema
block against what it would generate and skip the rewrite when the
column list matches — even when the *ordering* of those columns differs.
The result is that models which haven't had a schema change since the
config landed never get reordered, and `classified_sort` drift
accumulates indefinitely.

`--force` makes annotaterb always rewrite, so a single `bin/rake
annotate:clean` run brings every model into the canonical format and
keeps them there. Every schema block is now grouped primary-key →
regular columns → timestamps → foreign keys (alphabetical within each
group). Pure annotation comment change — no code modifications.

Also cleans up the rake task to avoid string interpolation for `system`
calls.
2026-05-13 14:12:48 +01:00

51 lines
1.5 KiB
Ruby
Vendored

# frozen_string_literal: true
class UserAssociatedGroup < ActiveRecord::Base
belongs_to :user
belongs_to :associated_group
before_destroy :remove_from_associated_groups
after_commit :add_to_associated_groups, on: %i[create update]
def add_to_associated_groups
associated_group.groups.each do |group|
group.add_automatically(user, subject: associated_group.label)
end
end
def remove_from_associated_groups
Group
.where(
"EXISTS(SELECT 1 FROM group_associated_groups WHERE group_id = groups.id)
AND NOT EXISTS(
SELECT 1
FROM user_associated_groups uag
JOIN group_associated_groups gag
ON gag.associated_group_id = uag.associated_group_id
WHERE uag.user_id = :user_id
AND uag.id != :uag_id
AND gag.group_id = groups.id
)",
uag_id: id,
user_id: user_id,
)
.each { |group| group.remove_automatically(user, subject: associated_group.label) }
end
end
# == Schema Information
#
# Table name: user_associated_groups
#
# id :bigint not null, primary key
# created_at :datetime not null
# updated_at :datetime not null
# associated_group_id :bigint not null
# user_id :bigint not null
#
# Indexes
#
# index_user_associated_groups (user_id,associated_group_id) UNIQUE
# index_user_associated_groups_on_associated_group_id (associated_group_id)
# index_user_associated_groups_on_user_id (user_id)
#