if ! EmailValidator.allowed?(value)
        record.errors.add(attribute, I18n.t(:'user.email.not_allowed'))
      else
        #good!
      end

так? спасибо

      unless EmailValidator.allowed?(value)
        record.errors.add(attribute, I18n.t(:'user.email.not_allowed'))
      end

А надо?

      if ! EmailValidator.allowed?(value)
        record.errors.add(attribute, I18n.t(:'user.email.not_allowed'))
      else
        #good!
      end

Получается так, но мне первый вариант больше нравится

      if ! EmailValidator.allowed?(value)
        record.errors.add(attribute, I18n.t(:'user.email.not_allowed'))
      else
        #good!
      end

Лучше

if allowed?
do_good
else
do_bad
end

Будет чуть лучше читаться. Но не принципиально

А, или у тебя обработки успешного сценария вообще нет? Но зачем тогда else?

Вообще есть разные воззрения на else ветку.
Кто-то старается её избегать везде где можно. Например давать переменным дефолтные значения, которые были бы в else ветке, а в if их переопределять.
Кто-то говорит, что ветка else должна быть всегда, пусть даже пустая (но с комментарием что так и должно быть) – это будет явно указывать, что этот вариант учтён при написании кода.
Кто-то говорит, что условие в if должно описывать "нормальную" работу, а в elsif/else уже идти обработка ошибок.
Кто-то говорит что в первой ветке должно быть больше кода, чем в else.
Короче, много эстетствующих людей есть

Про пустую ветку впервые слышу, звучит бредово несколько

Ни разу такого не видел

ну это наверное кто то однажды понаписал так элсов случайно

а потом сидел придумывал как оправдаться чтоб не удалять

Ну надо просто clean code по ебалу таким

И збс

Ну, где-то читал, не помню где.

как работает этот метод если в него не передается значение?

    def self.email_regex   
      /\A[a-zA-Z0-9!#\$%&'*+\/=?\^_`{|}~\-]+(?:\.[a-zA-Z0-9!#\$%&'\*+\/=?\^_`{|}~\-]+)*@(?:[a-zA-Z0-9](?:[a-zA-Z0-9\-]*[a-zA-Z0-9])?\.)+[a-zA-Z0-9](?:[a-zA-Z0-9\-]*[a-zA-Z0-9])?$\z/
      
    end

эта строка возвращает true или false ?

он возвращает регулярку

email_regex =~ "my@email"

Так вы не расширяете, вы переопределчяете. Для расширения есть alias_method_chain

А как переопределить контролер?
class ::UsersController
   
   prepend InviteMailerEventExtension  // и сюда писать весь этот код? https://github.com/discourse/discourse/blob/6ab12ed96b66933913da4be4c98f556fea516350/app/controllers/users_controller.rb
 end

@HexJS @Alyask_a
Хелпаните если несложно

каом метод выполняется перед

create

в RoR ?