How bad of practice is it to do something like this?

gistfile1.rb

require 'rubygems'
require 'active_support'

module CustomName
  def self.included(base)
    base.alias_method_chain :name, :update
  end

  def name_with_update
    ['custom', name_without_update].join(' ')
  end
end

class Person
  def name
    'person'
  end
end

Person.send(:include, CustomName)

p = Person.new
puts p.name #=> 'custom person'