apis-is / apis

Making data readily available to anyone interested
https://apis.is
MIT License
162 stars 64 forks source link

The flights data is messed up #456

Open maxrahder opened 6 years ago

maxrahder commented 6 years ago

Thanks for all the great data feeds!

But the flights data is all mixed up. It looks like you may be extracting your data from an HTML page, which has since been modified and no longer matches your expectations.

Steps to reproduce

Paste this into a browser URL: http://apis.is/flight?language=en&type=departures

You'll see data like this.

{
      "date": "08:40",
      "flightNumber": "<span class=\"cutoff\" title=\"Copenhagen\">Copenhagen</span>",
      "airline": "FI208",
      "to": "<span class=\"cutoff\" title=\"Icelandair\">Icelandair</span>",
      "plannedArrival": "Departed 08:29",
      "realArrival": "\n",
      "status": "\n\t\t\t\t\t\t\t\t"
},{
      "date": "21:10",
      "flightNumber": "<span class=\"cutoff\" title=\"New York JFK\">New York JFK</span>",
      "airline": "WW109",
      "to": "<span class=\"cutoff\" title=\"WOW air\">WOW air</span>",
      "plannedArrival": "",
      "realArrival": "\n<a href=\"#\" data-flight=\"2018/05/16 21:10 GMT+00:00|WW109\" class=\"bt-messenger\" aria-label=\"Notifications to Facebook Messenger\"><i class=\"icon icon-messenger\"></i></a>\n<a href=\"#\" data-flight=\"2018/05/16 21:10 GMT+00:00|WW109\" class=\"bt-twitter\" aria-label=\"Notifications to Twitter\"><i class=\"icon icon-twitter\"></i></a>\n<a href=\"#\" class=\"email-notifications\" aria-label=\"Notifictions by e-mail\"><i class=\"icon icon-mail\"></i></a>\n<a href=\"#\" class=\"pin-flight\" aria-label=\"Pin\"><i class=\"icon icon-pin hidden-sm hidden-md hidden-lg\"></i></a>\n",
      "status": "\n\t\t\t\t\t\t\t\t\t<a href=\"#\" class=\"btn btn-rounded btn-blue btn-sm hidden-sm hidden-md pin-flight\"><i class=\"icon icon-planeup\"></i> Pin</a>\n\t\t\t\t\t\t\t\t\t<a href=\"#\" class=\"visible-sm visible-md pin-flight\" aria-label=\"Pin\"><i class=\"icon icon-pin\"></i></a>\n\t\t\t\t\t\t\t\t"
}

Just about every field is messed up:

MiniGod commented 6 years ago

Thank you @maxrahder, this is a great reminder. There is already a PR to fix this: #443.

stale[bot] commented 6 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

ycaK commented 5 years ago

https://github.com/apis-is/apis/pull/474