Ideas for future versions

[cmfcmf]

Cleanup and Minor Features

• Remove deprecated methods and constructor arguments from the OpenWeatherMap class
• Change ° C back to °C (The API should not be HTML aware.)
• Add better error handling
• Make sure to handle invalid data from OWM properly (Invalid argument supplied for foreach() in WeatherForecast.php on line 85 #100)
• getRawWeatherGroupData() #131
• Remove population field from City.
• Remove $appid from all methods except constructor
• Split forecast apis into two methods (no more magic depending on the number of requested days)
• Rename temperature->current to temperature->avg
• Split up Temperature class into separate class for 16 day forecast
• Merge getCurrentWeather() and getCurrentWeatherGroup()

Documentation

• Remove Examples and write a proper documentation instead Cleanup code, examples, readme, add proper docs #98
• Document all the OWM bugs this API fixes

Tests

• Achieve 100% test code coverage
  • Find a tool to create a mock api server
  • Integrate mock api server

New Features

• Support more / all OWM apis
• Support unit conversion Feature request: wind in km/hour instead of m/s #37
• Replace "Fetcher" classes by HTTPlug http://httplug.io/ PSR 18
• Replace custom caching by PSR-6 implementation http://www.php-fig.org/psr/psr-6/
• Add PSR\Logger support (especially useful for debugging nasty OWM errors)

---

You are welcome to propose further (backwards incompatible) changes.

[aseriy]

Achieve 100% test code coverage?

[peter279k]

Hi @aseriy , the code coverage is very closed to the 100%.
Please refer this PR. Thanks.

[peter279k]

The official OpenWeather API service now supports the air pollution service.
Is it possible to add this feature in the next version?

Thanks.

[cmfcmf]

Is it possible to add this feature in the next version?

It certainly is possible to add this feature. However, I currently do not have the time to do so. I'd gladly accept contributions though :)