thedevdojo/voyager

No Belongs-To-Many relation to current model in BREAD

Ouverte

#2 193 ouverte le 8 nov. 2017

Voir sur GitHub
 (2 commentaires) (5 réactions) (0 personne assignée)PHP (2 652 forks)batch import
help wantedopen for suggestionstype:enhancement

Métriques du dépôt

Stars
 (11 817 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

  • Laravel Version: 5.5.20
  • Voyager Version: 1.0
  • PHP Version: 7.1.8
  • Database Driver & Version: MySQL 5.6.35

Description:

I would like to create a "belongs-to-many" relation from article to article which is called "related articles".

When I try to do this I can choose a pivot table but the system automatically chooses the keys for the relation. In this case it tries to use twice article_id as keys which is obviously not possible.

The fix would be to allow users to customize the key names which should be used inside of the pivot.

Steps To Reproduce:

  • Create a database table "test"
  • Create a database table "related_tests" with test_id and related_id as columns
  • Create a bread for this table
  • Add relationship
  • Choose belongs to many with test as target
  • Choose related_tests as pivot

Guide contributeur

Direction de recherche
Examinez le code de gestion des relations BREAD pour permettre la personnalisation des noms de clés pivot pour les relations BelongsToMany auto référentielles.
Stack technique
phplaravel
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHPLaravel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55