Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix erreur admin #994

Merged
merged 3 commits into from
Jul 8, 2020
Merged

Conversation

macintoshplus
Copy link
Member

  • Correction des erreurs 404 sur les pages de gestion des événements quand aucun identifiant d’événement n'est présent dans l'URL.
  • Vérification de la possibilité de supprimer un forum avant la suppression.
  • Correction du lien de suppression d'un forum.

@kuraobi
Copy link
Contributor

kuraobi commented Jul 8, 2020

Merci pour cette PR, notamment pour les 2 correctifs sur la suppression.
Cependant je n'arrive pas à produire le cas d'un ID absent dans les pages événements, aurais-tu un exemple de lien ? Quand je vais dans l'admin sur l'une de ces pages, par défaut il n'y a pas d'ID et c'est donc le dernier événement en date qui est sélectionné, mais du coup il devrait toujours y avoir un événement de sélectionné non ?
C'est juste que je trouve ça étrange de créer des pages vides 😅

@macintoshplus
Copy link
Member Author

Conditions : tous les événements doivent être terminé.

Clique dans le menu de gauche sur l'un des liens. Par exemple 'Speakers'.

Voici un aperçu du résultat:

image

@macintoshplus
Copy link
Member Author

Cependant je n'arrive pas à produire le cas d'un ID absent dans les pages événements, aurais-tu un exemple de lien ? Quand je vais dans l'admin sur l'une de ces pages, par défaut il n'y a pas d'ID et c'est donc le dernier événement en date qui est sélectionné, mais du coup il devrait toujours y avoir un événement de sélectionné non ?

La méthode AppBundle\Event\Model\Repository\EventRepository::getNextEvent() est déprécié. De plus elle ne retourne rien si aucun événement n'est à venir.

J'ai le cas avec ma base de données actuelles. Ce cas peux se produit après le Forum PHP où l'on n'a pas encore ajouté l'AFUP Day.

@kuraobi
Copy link
Contributor

kuraobi commented Jul 8, 2020

Elle est dépréciée mais n'a pas de remplaçante 😅 . Effectivement c'est logique qu'elle ne renvoie rien s'il n'y a pas de next event. Peut-être qu'on devrait avoir un getDefaultEvent() pour récupérer l'événement par défaut, ou simplement utiliser la méthode qui récupère la liste des events dispos et prendre le premier de la liste ?
Sinon ça fonctionne aussi avec des pages vides comme proposé ici,

@macintoshplus
Copy link
Member Author

Je préféré ne pas tenter de deviner l'intention de l'utilisateur et lui laisser le choix en le prévenant qu'il faut sélectionner quelque chose.

Lors de la MEP ça ne changera pas grans chose.

@macintoshplus macintoshplus merged commit 7af7a53 into afup:master Jul 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants