JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources
Remove to many issue using mongoid
Nessuno ha ancora preso questa issue.
- Lingua principale
- Ruby
- Stelle
- 2.3k
- Fork
- 546
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
I ran into an issue when trying to remove a has_many relationship using mongoid backed models.
The issues comes down to this code:
def _remove_to_many_link(relationship_type, key)
relation_name = self.class._relationships[relationship_type].relation_name(context: @context)
@model.public_send(relation_name).delete(key)
:completed
end
Mongoid defines the delete method on a relationship as follow https://github.com/mongodb/mongoid/blob/v5.1.2/lib/mongoid/relations/referenced/many.rb#L103
I had to override the method definition in my resource like this:
def _remove_to_many_link(relationship_type, key)
relation_name = self.class._relationships[relationship_type].relation_name(context: @context)
relationship = @model.public_send(relation_name)
document = relationship.find(id: key)
relationship.delete(document)
:completed
end
Not a huge deal but it would interesting to find a way to abstract how we delete relationships.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia da _remove_to_many_link e confronta la sua chiamata a relationship.delete(key) con il contratto di eliminazione delle relazioni referenced-many di Mongoid. Identifica il confine di astrazione per l'eliminazione delle relazioni; il lavoro è completato quando la rimozione di has_many funziona con Mongoid senza un override della risorsa, mantenendo invariato il comportamento esistente delle relazioni.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- ruby
- Ambito
- api, backend
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100