JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources

Remove to many issue using mongoid

Aperta
#724 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Type: Enhancement
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

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. 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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.