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

Afficher les alertes de trafic #40

Open
AymenFarrah opened this issue Nov 15, 2021 · 0 comments
Open

Afficher les alertes de trafic #40

AymenFarrah opened this issue Nov 15, 2021 · 0 comments

Comments

@AymenFarrah
Copy link

AymenFarrah commented Nov 15, 2021

Bonjour

Je voudrais bien faire une pull request mais j'ai pas réussi a en créer.

L'idée étant d'afficher l'alerte de trafic la plus récente pour être plus conforme à ce qu'affiche la RATP comme information.

Pour y arriver je propose ce code dans le service RatpTrafficService.php

// Collect specific errors : Travaux
if (isset($errors['Travaux'])) {
    array_push($events, current($errors['Travaux']));                         
} 
// Collect specific errors : Incident
if (isset($errors['Incidents techniques'])) {
    array_push($events, current($errors['Incidents techniques']));
} 
// Collect specific errors : Grève
if (isset($errors['Mouvement social'])) {
    array_push($events, current($errors['Mouvement social']));                        
} 
// If no specific errors ! Collect all available errors
if (empty($events)) {
    $events = current($errors);
}
// Sort events to get the recent event on top
usort($events, function ($event1, $event2) {
    return $event2['startDate'] <=> $event1['startDate'];
}
// Take the first event on the events list (the recent one)
$event = reset($events);
  • L' information donnée par la RATP en capture ici :

Capture d’écran 2021-11-15 à 21 10 28

  • La version d'API actuelle donne ça :

Capture d’écran 2021-11-15 à 21 02 23

- La version avec mon fix donne ça :

Capture d’écran 2021-11-15 à 21 02 31

Merci !

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

No branches or pull requests

1 participant