Meteor-Community-Packages/meteor-roles

Internal function _getInheritedRoleNames() should return role object

Open

#304 aperta il 10 dic 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (161 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (919 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

https://github.com/Meteor-Community-Packages/meteor-roles/blob/32c230ded9cb92b27a3485d143cabf2732fef186/roles/roles_common.js#L543-L557

The function _getInheritedRoleNames() is currently used to construct the array inheritedRoles on a role-assignment, where it only takes over the parameter _id. If you have data stored on the role, wouldn't it be better to have it copied along?

If one of these parameter changes, the programmer extending this package would then be responsible for also keeping this data updated there. If he doesn't need it there, he can extend this package wrapping this function to remove the additional properties.

Guida contributor

Direzione di ricerca
Esamina la funzione ` getInheritedRoleNames()` in `roles/roles common.js` (righe 543–557). Comprendi come attualmente restituisca solo i nomi dei ruoli e come viene costruito l'array `inheritedRoles`. Considera di modificare la funzione per restituire l'oggetto ruolo completo in modo che le proprietà aggiuntive vengano preservate. Valuta l'impatto sui chiamanti e assicura la compatibilità con le versioni precedenti o fornisci un'opzione.
Tech stack
javascriptnodejsmongodb
Dominio
authorization
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptMeteor
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70