Monkeypatch ActiveRecord :: FinderMethods

Question

Ich versuche Affe Patch ActiveRecord :: FinderMethods, um Hash-IDs für meine Modelle zu verwenden. So wird beispielsweise User.find (1) User.find ("FEW"). Leider wird meine überschriebene Methode nicht aufgerufen. Irgendwelche Ideen wie man die find_one Methode überschreibt?

module ActiveRecord 
    module FinderMethods 
    alias_method :orig_find_one, :find_one 
    def find_one(id) 
     if id.is_a?(String) 
     orig_find_one decrypt_id(id) 
     else 
     orig_find_one(id) 
     end 
    end 
    end 
end 

Answer 1

Hier ist ein Artikel, der tatsächlich beschreibt, wie zu tun, was Sie durch Überschreiben der User.primary_key Methode wie wollen:

class User 
    self.primary_key = 'hashed_id' 
end 

Welche Sie User.find zu nennen und die „hashed_id“ passieren würde es erlauben:

http://ruby-journal.com/how-to-override-default-primary-key-id-in-rails/

So ist es möglich.

Das sagte, würde ich empfehlen, dagegen zu tun, und stattdessen etwas wie User.find_by_hashed_id verwenden. Der einzige Unterschied ist, dass diese Methode nil zurückgibt, wenn ein Ergebnis nicht gefunden wird, anstatt eine ActiveRecord::RecordNotFound Ausnahme auszulösen. Sie können dies manuell in Ihrem Controller werfen:

def show 
    @user = User.find_by_hashed_id(hashed_id) 
    raise ActiveRecord::RecordNotFound.new if @user.nil? 
    ... continue processing ... 
end 

schließlich eine andere Notiz dies zu erleichtern, auf Sie - Rails hat auch eine Methode, die Sie in Ihrem Modell außer Kraft setzen können, to_param, ihm zu sagen, welche Eigenschaft zu verwenden beim Generieren von Routen. Standardmäßig verwendet es die ID, aber Sie möchten wahrscheinlich die hashed_id verwenden.

class User 
    def to_param 
    self.hashed_id 
    end 
end 

nun in Ihrem Controller wird params[:id] die hashed_id enthalten anstelle der ID.

def show 
    @user = User.find_by_hashed_id(params[:id]) 
    raise ActiveRecord::RecordNotFound.new if @user.nil? 
    ... continue processing ... 
end 

Answer 2

Ich stimme zu, dass Sie dabei vorsichtig sein sollten, aber es ist möglich.

Wenn Sie eine Methode decode_id haben, die eine Hash-ID zurück in die ursprüngliche ID konvertiert, dann funktioniert das folgende:

In User.rb

# Extend AR find method to allow finding records by an encoded string id: 
def self.find(*ids) 
    return super if ids.length > 1 

    # Note the short-circuiting || to fall-back to default behavior 
    find_by(id: decode_id(ids[0])) || super 
end 

nur sicherstellen, dass decode_id Renditen nil, wenn ein ungültiger Hash übergeben wurde. Auf diese Weise können durch Hash-ID und Standard-ID finden können, so dass, wenn Sie einen Benutzer mit der ID 12345 hatte, dann ist die folgende:

User.find(12345) 
User.find("12345") 
User.find(encode_id(12345)) 

Sollten alle geben den gleichen Benutzer.