- Fixed font icons and biomarkers can be shared #58
Conversation
| only_successful = self.request.GET.get('onlySuccessful') == 'true' | ||
| biomarkers = Biomarker.objects.filter(user=self.request.user) | ||
| biomarkers = Biomarker.objects.filter( | ||
| Q(is_public=True) | Q(user=user) | Q(shared_institutions__institutionadministration__user=user)).distinct() |
There was a problem hiding this comment.
Acá la condición me parece que está mal. Está devolviendo los que son públicos, ó los que pertenecen al usuario, o los que están compartidos con una institución de la cual el usuario es admin. Me parece que tendría que compartir con las instituciones en las que está el usuario, independientementes de que sea admin o no!
There was a problem hiding this comment.
en la condicion 'Q(shared_institutions__institutionadministration__user=user)' no significa que trae los que estan compartidos con la institucion a la que el usuario pertenece?.
Porque hice las pruebas y me traia si el usuario pertenecia a esa institucion por mas que no le pertenezca el biomarker o no sea admin. Tambien devolvia cuando era admin por mas que no le pertenezca. la condicion de si era admin la borre porque era lo que me devolvia mal la ultima vez. No se si me estoy confundiendo
| @@ -60,14 +62,15 @@ class Biomarker(models.Model): | |||
| cnas: QuerySet['CNAIdentifier'] | |||
| mirnas: QuerySet['MiRNAIdentifier'] | |||
| mrnas: QuerySet['MRNAIdentifier'] | |||
|
|
|||
| is_public = models.BooleanField(blank=False, null=False, default=False) | |||
There was a problem hiding this comment.
Agregar una línea vacía arriba para distinguir rápido entre los tipos y los campos de esta clase
No description provided.